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

chore: Update all references to 0.103.0 and 0.24.0 #86

Merged
merged 6 commits into from
Jul 16, 2024
Merged

Conversation

amoreena
Copy link
Contributor

@amoreena amoreena commented Jun 27, 2024

Description

  • updated to the latest stable version 0.103.0
  • added in readme for future users
  • pin version of the collector from the recent operator upgrade

Testing:

  1. tested this locally with the updated readme steps
Screenshot 2024-07-01 at 8 52 22 AM

@amoreena amoreena requested a review from taniyourstruly July 1, 2024 15:38
@amoreena amoreena marked this pull request as ready for review July 1, 2024 15:38
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
@amoreena amoreena requested a review from a team July 8, 2024 20:54
@amoreena amoreena force-pushed the LS-61382 branch 2 times, most recently from 9a95bcc to debd28a Compare July 8, 2024 22:22
amoreena added 2 commits July 8, 2024 15:22
drop all changes to collector-k8s chart

fix: Bump v for collector-k8s and fix operator distro

chore: Use k8s distro for operator
@amoreena amoreena requested review from a team July 9, 2024 20:47
@carolynblumberg
Copy link

I can shepherd this PR since @amoreena is OOO through EOW. @lightstep/team-telemetry-pipeline / @gdvalle want to take a last look at this?

@amoreena amoreena requested a review from carolynblumberg July 15, 2024 18:14
Copy link

@carolynblumberg carolynblumberg left a 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..

@amoreena amoreena requested a review from gdvalle July 15, 2024 21:14
Copy link
Member

@gdvalle gdvalle left a 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

@jmacd
Copy link
Contributor

jmacd commented Jul 15, 2024

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 docker pull to test this, but I have to re-install Docker a.t.m.)

Copy link
Contributor

@jmacd jmacd left a 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.

@amoreena amoreena merged commit ed882e9 into main Jul 16, 2024
1 check passed
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.

4 participants