-
Notifications
You must be signed in to change notification settings - Fork 26
Open
Labels
type/enhancementEn enhancement/improvement to an already existing featureEn enhancement/improvement to an already existing feature
Description
Issue submitter TODO list
- I've looked up my issue in FAQ
- I've searched for an already existing issues here (legacy) and here
- I've tried installing latest charts and the issue still persists there
- I'm running a supported version of the application & chart which is listed here
Feature request
Hi Team,
I’d like to contribute to the Kafka-UI Helm chart but currently don’t have access to create a branch for review.
Proposed Implementation:
I’ve identified the need to make the dnsPolicy value customizable in the Deployment for Kafka-UI.
The changes would look like this:
templates/deployment.yaml:
dnsPolicy: {{ .Values.dnsPolicy }}
values.yaml:
## @param dnsPolicy for Kafka-UI pods
## ref: https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/
##
dnsPolicy: "ClusterFirst"
Additionally, I noticed some trailing spaces during local testing. While not critical, for better code quality.
Expected behavior
dnsPolicy: {{ .Values.dnsPolicy }}
will allow users to configure the DNS policy directly through values, adding flexibility to the chart.
Your installation details
- commit sha:
a5d34a702017539ce10300628a508120aa372f58 - chart version:
1.4.11 - config: as default, nothing special
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
type/enhancementEn enhancement/improvement to an already existing featureEn enhancement/improvement to an already existing feature