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 usage of ABI 2 to match breaking changes in opentelemetry-cpp v1.16.0 release #466

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

justin-mccann
Copy link
Contributor

See https://github.com/open-telemetry/opentelemetry-cpp/blob/v1.16.0/CHANGELOG.md#1160-2024-06-21
for details on the v1.16.0 changes.

Fixes #465.

@justin-mccann justin-mccann requested a review from a team July 13, 2024 13:25
@justin-mccann justin-mccann marked this pull request as draft July 15, 2024 13:40
@justin-mccann
Copy link
Contributor Author

Apologies for the churn; I was following code from the relevant v1.16.0 changes and am also seeing some discrepancies between build environments.

@justin-mccann justin-mccann marked this pull request as ready for review July 15, 2024 15:57
@justin-mccann
Copy link
Contributor Author

The Centos7 check is failing because of issue #461. Maybe we should disable it until a new platform target is chosen.

@lalitb
Copy link
Member

lalitb commented Jul 17, 2024

@kpratyus, @DebajitDas #461 is blocker for now. Should we disable the relevant centos7 CI test to let other PRs get merged?

@lalitb
Copy link
Member

lalitb commented Jul 30, 2024

@svrnm The CI for webserver instrumentation has been failing for a long time now. Can you help check with @kpratyus / @DebajitDas if they can fix it soon :) Thanks.

@svrnm
Copy link
Member

svrnm commented Jul 30, 2024

@aryanishan1001 please take a look!

@justin-mccann
Copy link
Contributor Author

@marcalff You may want to take a look at this one; it overlaps a bit with your PR #441

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants