Skip to content

update apm server urls env var#64071

Closed
graphaelli wants to merge 1 commit intoelastic:masterfrom
graphaelli:update-apm-java-instructions
Closed

update apm server urls env var#64071
graphaelli wants to merge 1 commit intoelastic:masterfrom
graphaelli:update-apm-java-instructions

Conversation

@graphaelli
Copy link
Member

@graphaelli graphaelli commented Apr 21, 2020

Summary

fixes elastic/apm#258

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@graphaelli graphaelli added the release_note:skip Skip the PR/issue when compiling release notes label Apr 21, 2020
@graphaelli graphaelli requested review from a team and bmorelli25 April 21, 2020 13:34
Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bmorelli25
Copy link
Member

bmorelli25 commented Apr 21, 2020

I think we can close this PR. elastic/apm#258 was fixed in master #60790 but wasn't backported for some reason.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@graphaelli
Copy link
Member Author

Ah,. I didn't even notice it was the go agent section. I do expect this env var to be consistent across agents but the public spreadsheet does indeed reflect this discrepancy. I'll close this - is there an issue to align the agents on this var already?

@graphaelli graphaelli closed this Apr 21, 2020
@graphaelli graphaelli deleted the update-apm-java-instructions branch April 21, 2020 18:46
@bmorelli25
Copy link
Member

is there an issue to align the agents on this var already?

No, I don't think there is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes v7.7.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Kibana Home > Add APM > Java uses deprecated ELASTIC_APM_SERVER_URL

3 participants