-
-
Notifications
You must be signed in to change notification settings - Fork 443
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
POTEL 59 - Bump OpenTelemetry #3935
Conversation
|
Performance metrics 🚀
|
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.
from what i can tell it's fine
I guess the move of the dependencyManagement
block to the noagent gradle file is wanted, right? 😅
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 👍
@adinauer The system tests seem to be failing though |
Yes, some tests were failing because "no agent" didn't work with the new OTel version. It's fixed in the POTEL 60 PR. |
Yes, this is intentional as OTel suggests to add this config when using the spring boot otel starter. |
📜 Description
Bump OpenTelemetry SDK to
1.44.1
, OpenTelemetry instrumentation to2.10.0
and OpenTelemetry semantic conventions to1.28.0
💡 Motivation and Context
💚 How did you test it?
📝 Checklist
sendDefaultPII
is enabled.🔮 Next steps