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

[processor/deltatocumulative]: alpha stability #34747

Merged
merged 6 commits into from
Aug 21, 2024

Conversation

sh0rez
Copy link
Member

@sh0rez sh0rez commented Aug 20, 2024

Description: most major functionality is implemented and ready for wider testing. lets release as alpha and gather feedback

Link to tracking Issue: #30705

@sh0rez sh0rez requested a review from jpkrohling as a code owner August 20, 2024 09:19
@sh0rez sh0rez requested a review from a team August 20, 2024 09:19
@github-actions github-actions bot requested a review from RichieSams August 20, 2024 09:19
@sh0rez
Copy link
Member Author

sh0rez commented Aug 20, 2024

@jpkrohling does this need a changelog?

@jpkrohling
Copy link
Member

Yes, we need to communicate to users that this is now available as alpha.

Copy link
Member

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

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

some small nitpicks 😬

.chloggen/deltatocumulative-alpha.yaml Outdated Show resolved Hide resolved
.chloggen/deltatocumulative-alpha.yaml Outdated Show resolved Hide resolved
@crobert-1
Copy link
Member

From failing CI action:

Generated code is out of date, please run "make gendistributions" and commit the changes in this PR.

Copy link
Member

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

LGTM.

Just in case it isn't clear, to be able to have this component included in releases of the contrib distribution you'll also need to open a PR against the releases repository as described here. 👍

@sh0rez
Copy link
Member Author

sh0rez commented Aug 23, 2024

PR against the releases repo opened: open-telemetry/opentelemetry-collector-releases#647

f7o pushed a commit to f7o/opentelemetry-collector-contrib that referenced this pull request Sep 12, 2024
**Description:** most major functionality is implemented and ready for
wider testing. lets release as alpha and gather feedback

**Link to tracking Issue:**
open-telemetry#30705
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.

6 participants