-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[chore] Update core dependencies #37447
Conversation
Signed-off-by: opentelemetrybot <[email protected]>
Signed-off-by: opentelemetrybot <[email protected]>
Signed-off-by: opentelemetrybot <[email protected]>
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.
Looks like more checks need addressing
Filed #37471 to fix a few more tests |
The promotion of |
the |
@bogdandrutu @sfc-gh-sili could you help with this? |
Fixes some more issues with the OTel update
Sorry, should have given an update. I think this should be taken care of with #37476. I believe it's because the feature gate enabled queuing that breaks context propagation. |
Signed-off-by: Alex Boten <[email protected]>
I pushed a change run tidy to address some of the failing checks |
You need to run |
Note sure what to do about these failures:
|
Sounds vaguely related to #37447 (comment) ? |
I've got a fix for those. Not sure how to push directly to this PR, I'll open up a PR pointed at the branch. Edit: I guess you just open up a PR and merge it right away, should be good to go now. |
i'll get the versions syncd |
I couldn't merge main into this PR, so I replayed the change to fix the check-codeowners check. Edit: That apparently didn't fix all the issues. It's not a required check anyway, maybe we just allow it to fail. |
Signed-off-by: Alex Boten <[email protected]>
This PR updates the opentelemetry-collector dependency to the latest release