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] add note that remoteobserverprocessor ships with contrib distro #24211

Merged

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Jul 11, 2023

Description:
See open-telemetry/opentelemetry-collector-releases#370 for addition

Link to tracking Issue:
#19633

@atoulme atoulme force-pushed the update_remoteobserverprocessor_distros branch from 69f91f8 to ebd4eee Compare July 11, 2023 15:34
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a big deal, but it would be great to know when the right time to update this readme is, since the component is not yet released in the contrib image, but i don't think we have a set process yet

@atoulme
Copy link
Contributor Author

atoulme commented Jul 11, 2023

I think that's a great problem to have. It would indicate folks can't wait to get their hands on the component. It's also uncommon enough it doesn't feel like it warrants a fix. We could tie up some CI later to check that components that say they are in contrib are listed in manifest.yaml.

@codeboten codeboten merged commit efbd3a7 into open-telemetry:main Jul 11, 2023
81 checks passed
@github-actions github-actions bot added this to the next release milestone Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants