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

Add podLabels to Gateway Helm Chart #41057

Closed
blhagadorn opened this issue Sep 19, 2022 · 5 comments
Closed

Add podLabels to Gateway Helm Chart #41057

blhagadorn opened this issue Sep 19, 2022 · 5 comments

Comments

@blhagadorn
Copy link
Contributor

Describe the feature request

Add podLabels to Gateway Helm chart, for just adding labels to the pods (not the Deployment)

Describe alternatives you've considered

Kustomize, forking the chart

Affected product area (please put an X in all that apply)

[ ] Ambient
[ ] Docs
[ X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ X] User Experience
[ ] Developer Infrastructure

Affected features (please put an X in all that apply)

[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane

Additional context

The gateway helm chart doesn't expose podLabels like istiod does:

Gateway:
https://github.com/istio/istio/blob/master/manifests/charts/gateway/values.yaml

IstioD:


{{- range $key, $val := .Values.pilot.podLabels }}
{{ $key }}: "{{ $val }}"
{{- end }}

@zirain
Copy link
Member

zirain commented Sep 20, 2022

@blhagadorn
Copy link
Contributor Author

That's definitely one way to do it, would it be okay to add to the helm chart as well?

@blhagadorn
Copy link
Contributor Author

Since it's already in the other helm chart, I think it would bring parity by adding it to the helm chart. What do you think?

@afk42
Copy link

afk42 commented Feb 13, 2023

Hi @blhagadorn ,
May I ask why this wasn't accepted and merged into the project?
This is some feature that I could really use since I need to add these podLabels due to some internal implementations which I have no control over and we are not currently using Kustomize.
As refereed, this type of configuration is already possible on istio-discovery, so is it also possible to have the same here?

Thanks.

@magdalenakuhn
Copy link

Hi @blhagadorn , May I ask why this wasn't accepted and merged into the project? This is some feature that I could really use since I need to add these podLabels due to some internal implementations which I have no control over and we are not currently using Kustomize. As refereed, this type of configuration is already possible on istio-discovery, so is it also possible to have the same here?

Thanks.

Having the same issue! Is there any chance we can expose podLabels as value?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants