Skip to content

Remove 'operator.observability.prometheus' from Helm chart comments #1661

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

Merged
merged 1 commit into from
May 8, 2025

Conversation

Stogas
Copy link
Contributor

@Stogas Stogas commented May 6, 2025

@Stogas Stogas requested review from Allex1, jvoravong and a team as code owners May 6, 2025 15:38
Copy link

linux-foundation-easycla bot commented May 6, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Stogas / name: Ovidijus Balkauskas (5023367)

Copy link
Contributor

@jaronoff97 jaronoff97 left a comment

Choose a reason for hiding this comment

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

thank you for your contribution! although this is just a comment change, can you please bump the patch version for the chart and then re-run make generate-examples CHARTS=opentelemetry-operator

…nt example

Remove 'operator.observability.prometheus: true' from 'featureGatesMap' comment

As per https://github.com/open-telemetry/opentelemetry-helm-charts/blob/0bdda7e6b65507833bdd71555c975f7e4706fd34/charts/opentelemetry-operator/UPGRADING.md, the property is no longer supported here, removed in commit f9707d3
@Stogas
Copy link
Contributor Author

Stogas commented May 7, 2025

@jaronoff97 done

@jaronoff97 jaronoff97 merged commit 19eedd9 into open-telemetry:main May 8, 2025
5 checks passed
@jaronoff97
Copy link
Contributor

@Stogas thank you for your contribution! 🙇

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.

2 participants