-
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
[CI] Jenkins package pipeline for agentbeat #38951
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
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!
/package |
This pull request is now in conflicts. Could you fix it? 🙏
|
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.
LGTM provided that CI is green and the corresponding packaging step on Jenkins works as expected.
Regarding the use of a script (under .buildkite/
) to install msitools
, I also had the same initial thought, but on a second thought, we could simply pre-install it on our base image; this is tracked in https://github.com/elastic/ingest-dev/issues/3194
@v1v Moved the install script to |
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
…ic/beats into ci_jenkins_packaging_for_agentbeat
/package |
/test |
…ic/beats into ci_jenkins_packaging_for_agentbeat
Now packaging works. The PR is ready to merge |
Force merging to fix the 8.14 SNAPSHOT build |
I think something is still broken in the DRA packaging logic:
|
@pazone see #38951 (comment). This PR will also need to be backported I guess. |
Proposed commit message
Adds agentbeat to Jenkins packaging pipeline to fix the legacy DRA publishing
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues