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

Update indirect dependency opentelemetry-lib to 0.9.0 #14002

Closed
tskorupka opened this issue Sep 6, 2024 · 5 comments
Closed

Update indirect dependency opentelemetry-lib to 0.9.0 #14002

tskorupka opened this issue Sep 6, 2024 · 5 comments

Comments

@tskorupka
Copy link

tskorupka commented Sep 6, 2024

Hey,

apm-server in 8.15.x uses opentelemetry-lib in version 0.7.x

opentelemetry-lib 0.9.0 introduces changes in hostmetric scrapers interface changes, see https://github.com/elastic/opentelemetry-lib/releases/tag/v0.9.0

When we can expect newer version of apm-server with the updates?

This prevents us from using opentelemetry collectors in v0.108.0 :(

Kind Regards,
Tomasz

@kruskall
Copy link
Member

kruskall commented Sep 6, 2024

Hey 👋

Thanks for raising an issue!

apm-server in 8.15.x uses opentelemetry-lib in version 0.8.1

afaics apm-server 8.15.0 uses opentelemetry-lib 0.7.2:

github.com/elastic/opentelemetry-lib v0.7.2 // indirect

and apm-server 8.15.1 uses opentelemetry-lib 0.7.3:

github.com/elastic/opentelemetry-lib v0.7.3 // indirect

Can you double check the version you are using ?

This prevents us from using opentelemetry collectors in v0.108.0 :(

Could you clarify this ? what's the impact on the collector ?

@tskorupka
Copy link
Author

Hey,

Sorry, I meant it uses 0.7.x,

I would like to raise a request to update it to 0.8.x

Tomasz

@tskorupka
Copy link
Author

Could you clarify this ? what's the impact on the collector ?

hostmetrics scraper labels/ids changed in otel 0.107.0 (format), when we have updated otel collector to newest version, 0.108.0 we no longers see host metrics in elastic apm.

Tomasz

@lahsivjar
Copy link
Contributor

The opentelemetry-lib dependency is via apm-data. The dependencies are auto-upgraded and I can see an upgrade PR created: elastic/apm-data#357

Also, main already uses 0.8.1 now:

github.com/elastic/opentelemetry-lib v0.8.1 // indirect

The changes should be released with apm-server v8.16.0

@lahsivjar
Copy link
Contributor

The apm-data PR has been merged. The changes, as mentioned above, would be released with apm-server v8.16.0.

Closing this issue, feel free to reopen if needed.

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

3 participants