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

[Feature]: #469

Open
willemm opened this issue May 11, 2023 · 0 comments
Open

[Feature]: #469

willemm opened this issue May 11, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@willemm
Copy link

willemm commented May 11, 2023

Requirement

As a jaeger operator, I want to be able to override the name of the jaeger customresource in the jaeger-operator helm chart, so that I can upograde from using the CR I wrote myself to using the helm-chart generated CR, while keeping the name the same.

Problem

Currently, the name of the jaeger CR is defined as: {{ include "jaeger-operator.fullname" . }}-jaeger which makes it impossible to override

Proposal

Change the definiution at

name: {{ include "jaeger-operator.fullname" . }}-jaeger
to something like {{.Values.jaeger.name | default (printf "%s-jaeger" (include "jaeger-operator.fullname" .)) }}

Open questions

No response

@willemm willemm added the enhancement New feature or request label May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant