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

chore: Helm chart option to set POD container name #7384

Merged
merged 10 commits into from
Dec 5, 2024
1 change: 1 addition & 0 deletions charts/karpenter/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ cosign verify public.ecr.aws/karpenter/karpenter:1.0.0 \
| additionalClusterRoleRules | list | `[]` | Specifies additional rules for the core ClusterRole. |
| additionalLabels | object | `{}` | Additional labels to add into metadata. |
| affinity | object | `{"nodeAffinity":{"requiredDuringSchedulingIgnoredDuringExecution":{"nodeSelectorTerms":[{"matchExpressions":[{"key":"karpenter.sh/nodepool","operator":"DoesNotExist"}]}]}},"podAntiAffinity":{"requiredDuringSchedulingIgnoredDuringExecution":[{"topologyKey":"kubernetes.io/hostname"}]}}` | Affinity rules for scheduling the pod. If an explicit label selector is not provided for pod affinity or pod anti-affinity one will be created from the pod selector labels. |
| controller.containerName | string | `"controller"` | Distinguishing container name (containerName: karpenter-controller). |
| controller.env | list | `[]` | Additional environment variables for the controller pod. |
| controller.envFrom | list | `[]` | |
| controller.extraVolumeMounts | list | `[]` | Additional volumeMounts for the controller pod. |
Expand Down
4 changes: 4 additions & 0 deletions charts/karpenter/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,11 @@ spec:
hostNetwork: true
{{- end }}
containers:
{{- if .Values.controller.containerName }}
- name: {{ .Values.controller.containerName }}
{{- else }}
udhos marked this conversation as resolved.
Show resolved Hide resolved
- name: controller
{{- end }}
securityContext:
runAsUser: 65532
runAsGroup: 65532
Expand Down
2 changes: 2 additions & 0 deletions charts/karpenter/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,8 @@ extraVolumes: []
# expirationSeconds: 86400
# path: token
controller:
# -- Distinguishing container name (containerName: karpenter-controller).
containerName: controller
image:
# -- Repository path to the controller image.
repository: public.ecr.aws/karpenter/controller
Expand Down