-
Notifications
You must be signed in to change notification settings - Fork 178
Pass rollback window duration to upgrade watcher command #8177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pass rollback window duration to upgrade watcher command #8177
Conversation
This pull request does not have a backport label. Could you fix it @ycombinator? 🙏
|
b491de6
to
625abbb
Compare
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
dbb9df9
to
625abbb
Compare
Some upgrade integration tests in CI are failing with errors that look related to the changes in this PR:
Moving PR back to draft while I investigate and fix. |
da545a0
to
0711763
Compare
@pchila @kaanyalti Thank you for your patience with this PR as I shifted my focus to other, more time-critical work (FIPS). This PR is ready for re-review now, when you get a chance. |
|
@ycombinator can we put this one on hold for the moment? The discussion around locking for the upgrade marker may have impacts on the distributions of responsibilities (we may even want the elastic-agent main process to handle the writing of the rollback window, in which case we don't need to pass an extra parameter to the watcher) |
Superseded by #8767 |
What does this PR do?
This PR enhances the upgrade process to invoke the Upgrade Watcher with an additional CLI option,
--rollback-window
. This option accepts any value that can be parsed bytime.ParseDuration
. However, the code that invokes the Upgrade Watcher passes this value in seconds, e.g.180s
.Why is it important?
The Upgrade Watcher will use the value of the
--rollback-window
CLI option to help ensure that the upgraded Agent can be rolled back manually within the specified rollback window duration. This functionality will be implemented in future PRs.Checklist
I have made corresponding changes to the documentationI have made corresponding change to the default configuration filesI have added an entry in./changelog/fragments
using the changelog toolI have added an integration test or an E2E testDisruptive User Impact
None; the Upgrade Watcher is invoked internally as part of the Agent upgrade process.
How to test this PR locally
Related issues
rollback_window
is set #6880Questions to ask yourself