Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(opentelemetry-operator): ensure valid version label #1483

Merged

Conversation

sathieu
Copy link
Contributor

@sathieu sathieu commented Jan 7, 2025

To test:

helm template my-release charts/opentelemetry-operator/ \
  --set manager.collectorImage.repository=otel/opentelemetry-collector-contrib \
  --set manager.image.tag=v0.116.0@sha256:11edfad24ead5f48140eb12aea642a2716ca0cd9e63eae3851f39c4477f3c2c2 \
  | grep app.kubernetes.io/version

Bug introduced by #1461.
See grafana/loki#8260 for a similar fix.

Signed-off-by: Mathieu Parent <math.parent@gmail.com>
@sathieu sathieu force-pushed the otel-operator-valid-version-label branch from 8c5bd5a to b997aee Compare January 8, 2025 13:28
@TylerHelmuth TylerHelmuth merged commit 2d8972d into open-telemetry:main Jan 8, 2025
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants