-
Notifications
You must be signed in to change notification settings - Fork 525
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
package java-attacher alongside apm-server #6593
package java-attacher alongside apm-server #6593
Conversation
This pull request does not have a backport label. Could you fix it @stuartnelson3? 🙏
NOTE: |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
Note: in the
|
This pull request is now in conflicts. Could you fix it @stuartnelson3? 🙏
|
This pull request is now in conflicts. Could you fix it @stuartnelson3? 🙏
|
@axw @felixbarny let me know if the gpg stuff aligns with what you want/expect, or if I should be storing the pubkey somewhere else. |
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
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 great, thank you :)
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.
Please wait until 1.27.0 is out before merging. Otherwise, just some minor suggestions.
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.
I took the liberty to update the attacher version to the just released version 1.27.0
/test |
(cherry picked from commit 5a8e0d2)
(cherry picked from commit 5a8e0d2) Co-authored-by: stuart nelson <[email protected]>
Verified by downloading the |
Motivation/summary
download and package the
java-attacher
when executingmake release
Checklist
How to test these changes
run some different flavors of
mage package
, eg.DEV=true SNAPSHOT=true PLATFORMS=linux/amd64 TYPES=docker mage package
DEV=true SNAPSHOT=true PLATFORMS=linux/amd64 TYPES=tar.gz mage package
DEV=true SNAPSHOT=true PLATFORMS=linux/amd64 TYPES=deb mage package
and verify that
java-attacher.jar
is stored in the artifact alongside theapm-server
binaryRelated issues
#4824