Skip to content
Discussion options

You must be logged in to vote

Thanks for the rc.37 results, both of these are useful.

The "Check for updates" button applying updates outside the maintenance window is a real bug, and you found a gap in my rc.36 fix. I traced it. rc.36 added the window gate to the scheduled scan path (the cron tick checks the window before it runs). But the manual "Check for updates" button doesn't go through that path. It calls the watcher's watch() directly, which skips the scan-level gate entirely, and your auto-apply action trigger is subscribed to every container report (cron or manual), so it picks up the result and applies the update regardless of the window. So rc.36 closed the scheduled path but left the manual one open. That…

Replies: 10 comments 6 replies

Comment options

You must be logged in to vote
2 replies
@lrnd1
Comment options

@lrnd1
Comment options

Comment options

You must be logged in to vote
1 reply
@lrnd1
Comment options

Comment options

You must be logged in to vote
1 reply
@lrnd1
Comment options

Comment options

You must be logged in to vote
1 reply
@lrnd1
Comment options

Comment options

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
1 reply
@lrnd1
Comment options

Comment options

You must be logged in to vote
0 replies
Answer selected by scttbnsn
Comment options

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
0 replies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Q&A
Labels
None yet
2 participants