Skip to content
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

[fleet] Bundle the java agent attacher #4824

Closed
jalvz opened this issue Feb 19, 2021 · 8 comments
Closed

[fleet] Bundle the java agent attacher #4824

jalvz opened this issue Feb 19, 2021 · 8 comments
Assignees
Milestone

Comments

@jalvz
Copy link
Contributor

jalvz commented Feb 19, 2021

We can do it manually or automate as part of the build / package process.

As we don't expect that the attacher will be updated often, it is probably easier do it manually. The attacher won't necessarily follow stack versioning.

See elastic/apm-agent-java#1667

@jalvz jalvz added the fleet label Feb 19, 2021
@simitt simitt added the v7.14.0 label Jun 1, 2021
@simitt simitt added this to the 7.14 milestone Jun 1, 2021
@simitt
Copy link
Contributor

simitt commented Jun 1, 2021

@felixbarny and @jalvz let's figure out the details how we want to bundle this so we can work on it for 7.14.

@simitt
Copy link
Contributor

simitt commented Jun 17, 2021

@felixbarny can you provide details from where the java agent attacher can be pulled when bundling?

We can do it manually or automate as part of the build / package process.

@jalvz maybe I am misunderstandind, but I'd like to avoid adding any manual steps. The java agent attacher should be part of the packaging process, used in the release process.

@jalvz
Copy link
Contributor Author

jalvz commented Jun 17, 2021

The attacher lives in the java agent repo: https://github.com/elastic/apm-agent-java/tree/master/apm-agent-attach

Maybe just copying it to some folder in apm-server is enough for start, wdyt?

But we need to get "notified" if there are important (or breaking) changes in the attacher that need to be aware of...

@felixbarny
Copy link
Member

Two options: you can either download the attacher from maven central, or we just commit the attacher binaries to the APM Server GitHub repo.

@zube zube bot modified the milestones: 7.14, 7.15 Jul 1, 2021
@zube zube bot added the v7.15.0 label Jul 1, 2021
@zube zube bot modified the milestones: 7.15, 7.16 Aug 20, 2021
@stuartnelson3
Copy link
Contributor

@simitt @axw preference on how to manage this? I'd be inclined to add it as a manual release step, similar to updating beats.

@axw
Copy link
Member

axw commented Aug 24, 2021

Updating manually sounds fine to me.

I'd prefer not to commit the jar to the repo. We could instead inject them into the packages, fetching them at package build time:

func customizePackaging() {

WDYT @stuartnelson3?

@stuartnelson3
Copy link
Contributor

That sounds fine to me, then we just update the fetch url in the magefile

@stuartnelson3
Copy link
Contributor

closed in #6593

@zube zube bot added [zube]: Done and removed [zube]: Ready labels Nov 16, 2021
@zube zube bot removed the [zube]: Done label Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants