-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Updates to OTel-Arrow v0.24.0 deps #33518
Conversation
(The test failures look unrelated.) |
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.
Right, they're related to #33520
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
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.
Thanks @jmacd. Will there be much more work on the receiver/exporter that requires it to be done in multiple repositories or could most of the work be done in this repo?
Reason I'm asking is that it's difficult to catch up on all the changes in a single PR review. And I'm guessing doing the work in multiple places sucks too :)
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.
@codeboten Thanks! Re: #33518 (review), yes ... yes.
I filed open-telemetry/otel-arrow#225 to record this intention. We will remove the exporter and receiver components from that repository as soon as they are merged in this PR.
Description:
This is a copy of the otel-arrow repository exporter and receiver components at
open-telemetry/otel-arrow@dd6e224
Updates both go.mods to OTel-Arrow v0.24.0 dependencies, the current release.
Applies repo-specific
make gci
logic.Link to tracking Issue: #26491
Testing: Passing
Documentation: n/a