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

Custom annotation for service name #128

Closed
DanTulovsky opened this issue Aug 21, 2023 · 3 comments
Closed

Custom annotation for service name #128

DanTulovsky opened this issue Aug 21, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@DanTulovsky
Copy link

According to: https://docs.otterize.com/reference/configuration/network-mapper#pod-annotations setting the intents.otterize.com/service-name annotation on a pod will set the name of the service.

Please consider allowing this annotation to be set by the users. We already have such an annotation on our pods and it would be best not to duplicate it.

(As an aside, maybe a label would be better? Perhaps default to the app.kubernetest.io/name label? As virtually everything should have that set already.)

Thank you

@orishoshan orishoshan added the enhancement New feature or request label Aug 21, 2023
@orishoshan
Copy link
Contributor

Hey @DanTulovsky!

We're thinking of adding a helm-chart configuration value named global.serviceNameOverrideAnnotationName. It would go in the global section because it applies to multiple subcharts: to the intents-operator, network-mapper and credentials-operator. If you don't specify it, you'll get the default value intents.otterize.com/service-name.

Regarding use of labels: we preferred use of an annotation because it is not size-limited, unlike labels.
Since we want to keep it backwards-compatible, we prefer not to use app.kubernetes.io/name at the moment, as it would alter default behavior for existing users.

@usarid
Copy link
Contributor

usarid commented Aug 22, 2023

LGTM - let’s do it. global.serviceNameOverrideAnnotationName

@omris94
Copy link
Contributor

omris94 commented Aug 24, 2023

Hey @DanTulovsky, I've just merged and released everything. You can set the annotation now using global.serviceNameOverrideAnnotationName. You can find more information here.

@omris94 omris94 closed this as completed Aug 24, 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

4 participants