-
Notifications
You must be signed in to change notification settings - Fork 12
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 all references to 0.103.0 and 0.24.0 #86
Conversation
fix path reference increment app version bump version to reflect security update refactor: Remove uncommented concurrent batch processor and make note in readme docs: Add in readme to show repro steps for updating versions or testing locally chore: recomment lines in the arrow builder config switch to batch processor like the vm example wanted uncoment angry attributes processor uncomment receivers
9a95bcc
to
debd28a
Compare
drop all changes to collector-k8s chart fix: Bump v for collector-k8s and fix operator distro chore: Use k8s distro for operator
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.
This looks good to me, but I don't have much experience updating these charts..
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, but I don't have a lot of awareness of the changes... I scanned https://github.com/open-telemetry/opentelemetry-collector-contrib/releases and the non-contrib side for obvious breaking changes, and didn't see anything immediately jump out
I'm mildly concerned that the otelarrowexporter and otelarrowreceiver are not actually included in the v0.103 release, and possibly not in the v0.104 release, because open-telemetry/opentelemetry-collector-releases#595 merged after v0.104 was cut. (I would |
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.
The components are OK at v0.103, it was only the contrib-col docker image that does not have them. Thanks @amoreena.
Description
Testing: