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.

Sign up for GitHub

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

Jump to bottom

[processor/metricsgeneration] Remove processor's experimental_ prefix in name #35426

Closed
crobert-1 opened this issue Sep 25, 2024 · 1 comment · Fixed by #35821
Closed

[processor/metricsgeneration] Remove processor's experimental_ prefix in name #35426

crobert-1 opened this issue Sep 25, 2024 · 1 comment · Fixed by #35821
Assignees
@dmitryax
Labels
processor/metricsgeneration Metrics Generation processor

Comments

@crobert-1
Copy link
Member

Component(s)

processor/metricsgeneration

Describe the issue you're reporting

This processor was first introduced to the contrib distribution in #3266, more than 3 years ago. It is considered alpha, and has been a part of contrib releases since at least v0.41.0. I think it's safe to remove the experimental_ prefix, and simply call it the metricsgeneration processor.

Please let me know if there's any reason we should leave it as-is. I realize renaming components is a breaking change, but as it's a simple rename it should be relatively simple to handle for users.
@crobert-1 crobert-1 added the needs triage New item requiring triage label Sep 25, 2024
@github-actions github-actions bot added the processor/metricsgeneration Metrics Generation processor label Sep 25, 2024
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

This was referenced Oct 1, 2024
Closed
Closed
@atoulme atoulme removed the needs triage New item requiring triage label Oct 11, 2024
@dmitryax dmitryax mentioned this issue Oct 15, 2024
Merged
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this issue Dec 17, 2024
@dmitryax @sbylica-splunk
[processor/metricsgeneration] Remove "experimental_" prefix (open-tel…
5f2bc3c 
…emetry#35821)

The processor now has to be defined as `metricsgeneration` instead of
`experimental_metricsgeneration` in OTel collector configuration files.

Fixes
open-telemetry#35426
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmitryax dmitryax

Labels
processor/metricsgeneration Metrics Generation processor
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[processor/metricsgeneration] Remove "experimental_" prefix dmitryax/opentelemetry-collector-contrib
3 participants
@atoulme @dmitryax @crobert-1