-
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 #20400 to 7.x: New agent structure (symlinks) #20960
Conversation
[Ingest Manager] New agent structure (symlinks) (elastic#20400)
Pinging @elastic/ingest-management (Team:Ingest Management) |
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.
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
Cherry picking #20986 for rpm/deb fixes |
[Ingest Manager] Support for UPGRADE_ACTION (elastic#21002)
Cherry-pick of PR #21002: Support for UPGRADE_ACTION
Cherry-pick of PR #20986 which fixes DEB packages
Cherry-pick of PR #20400 to 7.x branch. Original message:
What does this PR do?
Different approach to #20307
Working with symlinks turned out to be a bit tricky due to how OSes handles Working Directory and executable names.
Windows on top of that requires that service name which is used to be registered is in Abs form and it needs to match the one used to reguiter the service (hence the magic with os.Args[0] replacements in the code,
os.Args[0]
is used in lib as a service name)Due to the approach of determining WD when running a binary using a symlink
paths.yml
is either on the symlink level (windows
) or on executable level (darwin
andlinux
).These will get regenerated during future Upgrade/Rollback.
Why is it important?
For future upgrade work
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Same as with previous PR this is tested on linux/darwin and windws (service and direct run)