Skip to content

Commit

Permalink
Removing unnecessary call to setMeta
Browse files Browse the repository at this point in the history
  • Loading branch information
100mik committed Feb 7, 2022
1 parent 5038d99 commit 13b8b42
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions pkg/kapp/app/recorded_app.go
Original file line number Diff line number Diff line change
Expand Up @@ -361,16 +361,12 @@ func (a *RecordedApp) update(doFunc func(*Meta)) error {

change.Data = meta.AsData()

updatedMeta, err := a.coreClient.CoreV1().ConfigMaps(a.nsName).Update(context.TODO(), change, metav1.UpdateOptions{})
_, err = a.coreClient.CoreV1().ConfigMaps(a.nsName).Update(context.TODO(), change, metav1.UpdateOptions{})
if err != nil {
return fmt.Errorf("Updating app: %s", err)
}

// Update memoized meta
_, err = a.setMeta(*updatedMeta)
if err != nil {
return err
}
a.memoizedMeta = &meta

return nil
}
Expand Down

0 comments on commit 13b8b42

Please sign in to comment.