-
Notifications
You must be signed in to change notification settings - Fork 176
Adding new configuration setting: agent.upgrade.rollback.window
#8065
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
Conversation
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
This pull request does not have a backport label. Could you fix it @ycombinator? 🙏
|
a95b2b0
to
624d614
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. All unit tests and integration tests pass. Only failure is related to check-ci and then change to elastic-agent.reference.yml
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a couple of nitpicks about the new settings description, not blocking. Feel free to resolve them if you think those are unnecessary.
LGTM
Co-authored-by: Paolo Chilà <[email protected]>
Co-authored-by: Paolo Chilà <[email protected]>
40ffcbe
to
98e3528
Compare
|
💛 Build succeeded, but was flaky
Failed CI StepsHistory
cc @ycombinator |
* upstream/main: Guard against `nil` pointer dereference (elastic#8107) Generate NOTICE.txt with only modules used by binaries (elastic#8053) Retry enrollment requests when an error is returned, add enrollment timeout (elastic#8056) Changelog for 8.17.6 version (elastic#8062) (elastic#8106) [main][Automation] Update versions (elastic#8098) Allow using beats receivers for self-monitoring (elastic#8031) Adding new configuration setting: `agent.upgrade.rollback.window` (elastic#8065) [Integration Testing] Allow tests to declare themselves as needing a FIPS environment (elastic#8083) fix(agentless): overcome SIGPIPE in agentless promotion pipeline (elastic#8094) ksm autosharing integration configuration update (elastic#8086)
What does this PR do?
This PR adds a new optional Agent configuration setting,
agent.upgrade.rollback.window
. This setting takes a string parseable bytime.Duration()
. The default value of this setting is168h
(== 7 days). The value of this setting will determine how long after users upgrade an Agent will they be permitted to request a rollback to the previous version of the Agent on disk.Why is it important?
To allow users to perform manual rollbacks of upgraded Elastic Agent within a certain time window.
Checklist
I have made corresponding changes to the documentation./changelog/fragments
using the changelog toolDisruptive User Impact
None; this PR adds a new, optional setting. Further, older versions of Elastic Agent will ignore this setting if they encounter it.
How to test this PR locally
Related issues
rollback_window
to elastic agent configuration with a default value of 7d #6881Questions to ask yourself