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

Unable to specifying custom labels for the deployable components in production strategy #1531

Closed
primeroz opened this issue Aug 11, 2021 · 0 comments · Fixed by #2277
Closed
Labels
bug Something isn't working

Comments

@primeroz
Copy link

Describe the bug
this is essentialy a copy of #629 . in that issue the problem was fixed only for the all-in-one strategy and not for the production one

To Reproduce
Steps to reproduce the behavior:

  1. Install jaeger operator
  2. create a Jaeger instance in production mode and specify labels for components like query and collector
  3. jaeger operator will fail to create the pods with an error like
level=error msg="failed to apply the changes" error="Deployment.apps \"jaeger-collector\" is invalid: spec.template.metadata.labels: Invalid value: map[string]string{\"app\":\"jaeger\", \"app.kubernetes.io/component\":\"collector\", \"app.kubernetes.io/instance\":\"jaeger\", \"app.kubernetes.io/managed-by\":\"jaeger-operator\", \"app.kubernetes.io/name\":\"jaeger-collector\", \"app.kubernetes.io/part-of\":\"jaeger-server\"}: `selector` does not match template `labels`" execution="2021-08-11 08:50:22.135410659 +0000 UTC" instance=jaeger namespace=jaeger-server

Expected behavior
custom labels are added to the deployments and jaeger pods get created

Version (please complete the following information):

  • OS: [e.g. Linux] Linux
  • Jaeger version: [e.g. 1.8] 1.24
  • Deployment: [e.g. bare metal, Docker, Kubernetes, ...] EKS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant