-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Cherry-pick #24253 to 7.x: Make installer atomic on windows #24297
Conversation
[Ingest Manager] Make installer atomic on windows (elastic#24253)
Pinging @elastic/ingest-management (Team:Ingest Management) |
Pinging @elastic/agent (Team:Agent) |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
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.
Backport looks good, other than the changelog entry.
@@ -36,6 +36,8 @@ | |||
- Windows agent doesn't uninstall with a lowercase `c:` drive in the path {pull}23998[23998] | |||
- Fix reloading of log level for services {pull}[24055]24055 | |||
- Fix: Successfully installed and enrolled agent running standalone{pull}[24128]24128 | |||
- Make installer atomic on windows {pull}[24253]24253 | |||
- Remove installed services on agent uninstall {pull}[24151]24151 |
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.
Need to fix this changelog entry.
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.
changelogs 😢
Cherry-pick of PR #24253 to 7.x branch. Original message:
What does this PR do?
PR fixes issue on windows when on restart while installing beats we end up with partial data.
awaitable installler was introduced which forces app wait for installer finish its job
and
sync is forced for windows after rename is called during install.
Why is it important?
Fixes #24180
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.