Skip to content

Commit

Permalink
fixup! o/configstate/configcore: tweak comments around managed refres…
Browse files Browse the repository at this point in the history
…h schedule

Signed-off-by: Maciej Borzecki <[email protected]>
  • Loading branch information
bboozzoo committed Jun 20, 2024
1 parent 338b29d commit 7c24ca1
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions overlord/configstate/configcore/refresh.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,10 @@ func init() {
}

func reportOrIgnoreInvalidManageRefreshes(tr RunTransaction, optName string) error {
// check if the option is set as part of transaction changes; if not than
// it's already set in the config state and we shouldn't error out about it
// now. refreshScheduleManaged will do the right thing when refresh cannot
// be managed anymore.
// check if the option is set as part of transaction changes; if not
// than it's already set in the config state and we shouldn't error out
// about it now since the required conditions were met at the time it
// got set
for _, k := range tr.Changes() {
if k == "core."+optName {
return fmt.Errorf("cannot set schedule to managed")
Expand Down

0 comments on commit 7c24ca1

Please sign in to comment.