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

Helm: sanitize resource name lengths #3250

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions charts/kueue/templates/certmanager/certificate.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: cert-manager.io/v1
kind: Issuer
metadata:
name: {{ include "kueue.fullname" . }}-selfsigned-issuer
name: {{ printf "%s-selfsigned-issuer" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
Expand All @@ -12,16 +12,16 @@ spec:
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
name: {{ include "kueue.fullname" . }}-serving-cert
name: {{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
spec:
dnsNames:
- '{{ include "kueue.fullname" . }}-webhook-service.{{ .Release.Namespace }}.svc'
- '{{ include "kueue.fullname" . }}-webhook-service.{{ .Release.Namespace }}.svc.{{ .Values.kubernetesClusterDomain }}'
- '{{ printf "%s-webhook-service.%s.svc" (include "kueue.fullname" .) .Release.Namespace | trunc 63 | trimSuffix "-" }}'
- '{{ printf "%s-webhook-service.%s.svc.%s" (include "kueue.fullname" .) .Release.Namespace .Values.kubernetesClusterDomain | trunc 63 | trimSuffix "-" }}'
Comment on lines +21 to +22
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would this actually work?

Doesn't it always have to finish in .svc or .svc.{ClusterDomain}?

Do we need to worry about length for dns names?

issuerRef:
kind: Issuer
name: {{ include "kueue.fullname" . }}-selfsigned-issuer
secretName: {{ include "kueue.fullname" . }}-webhook-server-cert
name: {{ printf "%s-selfsigned-issuer" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
secretName: {{ printf "%s-webhook-server-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: admissionchecks.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
4 changes: 2 additions & 2 deletions charts/kueue/templates/crd/kueue.x-k8s.io_clusterqueues.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: clusterqueues.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
4 changes: 2 additions & 2 deletions charts/kueue/templates/crd/kueue.x-k8s.io_cohorts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: cohorts.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
4 changes: 2 additions & 2 deletions charts/kueue/templates/crd/kueue.x-k8s.io_localqueues.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: localqueues.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: multikueueclusters.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: multikueueconfigs.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: provisioningrequestconfigs.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: resourceflavors.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
4 changes: 2 additions & 2 deletions charts/kueue/templates/crd/kueue.x-k8s.io_topologies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: topologies.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: workloadpriorityclasses.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
4 changes: 2 additions & 2 deletions charts/kueue/templates/crd/kueue.x-k8s.io_workloads.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
{{- include "kueue.labels" . | nindent 4 }}
annotations:
{{- if .Values.enableCertManager }}
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ include "kueue.fullname" . }}-serving-cert
cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ printf "%s-serving-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- end }}
controller-gen.kubebuilder.io/version: v0.16.4
name: workloads.kueue.x-k8s.io
Expand All @@ -16,7 +16,7 @@ spec:
webhook:
clientConfig:
service:
name: {{ include "kueue.fullname" . }}-webhook-service
name: {{ printf "%s-webhook-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
path: /convert
conversionReviewVersions:
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/internalcert/secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: v1
kind: Secret
metadata:
name: {{ include "kueue.fullname" . }}-webhook-server-cert
name: {{ printf "%s-webhook-server-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/manager/auth_proxy_service.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
kind: Service
metadata:
name: {{ include "kueue.fullname" . }}-controller-manager-metrics-service
name: {{ printf "%s-controller-manager-metrics-service" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/manager/manager-config.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: {{ include "kueue.fullname" . }}-manager-config
name: {{ printf "%s-manager-config" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
Expand Down
8 changes: 4 additions & 4 deletions charts/kueue/templates/manager/manager.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ include "kueue.fullname" . }}-controller-manager
name: {{ printf "%s-controller-manager" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
Expand Down Expand Up @@ -82,7 +82,7 @@ spec:
resources: {}
securityContext:
{{- toYaml .Values.controllerManager.manager.podSecurityContext | nindent 8 }}
serviceAccountName: {{ include "kueue.fullname" . }}-controller-manager
serviceAccountName: {{ printf "%s-controller-manager" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
{{- with .Values.controllerManager.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand All @@ -92,7 +92,7 @@ spec:
- name: cert
secret:
defaultMode: 420
secretName: {{ include "kueue.fullname" . }}-webhook-server-cert
secretName: {{ printf "%s-webhook-server-cert" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
- configMap:
name: {{ include "kueue.fullname" . }}-manager-config
name: {{ printf "%s-manager-config" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
name: manager-config
2 changes: 1 addition & 1 deletion charts/kueue/templates/prometheus/monitor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
name: {{ include "kueue.fullname" . }}-controller-manager-metrics-monitor
name: {{ printf "%s-controller-manager-metrics-monitor" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
labels:
{{- include "kueue.labels" . | nindent 4 }}
namespace: '{{ .Release.Namespace }}'
Expand Down
6 changes: 3 additions & 3 deletions charts/kueue/templates/prometheus/role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
name: {{ include "kueue.fullname" . }}-prometheus-k8s
name: {{ printf "%s-prometheus-k8s" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
Expand Down Expand Up @@ -37,14 +37,14 @@ rules:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: {{ include "kueue.fullname" . }}-prometheus-k8s
name: {{ printf "%s-prometheus-k8s" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}
namespace: '{{ .Release.Namespace }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: '{{ include "kueue.fullname" . }}-prometheus-k8s'
name: '{{ printf "%s-prometheus-k8s" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
subjects:
- kind: ServiceAccount
name: prometheus-k8s
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: ClusterRole
metadata:
labels:
{{- include "kueue.labels" . | nindent 4 }}
name: '{{ include "kueue.fullname" . }}-metrics-reader'
name: '{{ printf "%s-metrics-reader" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
rules:
- nonResourceURLs:
- "/metrics"
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/rbac/auth_proxy_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: ClusterRole
metadata:
labels:
{{- include "kueue.labels" . | nindent 4 }}
name: '{{ include "kueue.fullname" . }}-proxy-role'
name: '{{ printf "%s-proxy-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
rules:
- apiGroups:
- authentication.k8s.io
Expand Down
6 changes: 3 additions & 3 deletions charts/kueue/templates/rbac/auth_proxy_role_binding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@ kind: ClusterRoleBinding
metadata:
labels:
{{- include "kueue.labels" . | nindent 4 }}
name: '{{ include "kueue.fullname" . }}-proxy-rolebinding'
name: '{{ printf "%s-proxy-rolebinding" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
name: '{{ include "kueue.fullname" . }}-proxy-role'
name: '{{ printf "%s-proxy-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
subjects:
- kind: ServiceAccount
name: '{{ include "kueue.fullname" . }}-controller-manager'
name: '{{ printf "%s-controller-manager" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
namespace: '{{ .Release.Namespace }}'
2 changes: 1 addition & 1 deletion charts/kueue/templates/rbac/batch_admin_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ kind: ClusterRole
metadata:
labels:
{{- include "kueue.labels" . | nindent 4 }}
name: '{{ include "kueue.fullname" . }}-batch-admin-role'
name: '{{ printf "%s-batch-admin-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
aggregationRule:
clusterRoleSelectors:
- matchLabels:
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/rbac/batch_user_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ kind: ClusterRole
metadata:
labels:
{{- include "kueue.labels" . | nindent 4 }}
name: '{{ include "kueue.fullname" . }}-batch-user-role'
name: '{{ printf "%s-batch-user-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
aggregationRule:
clusterRoleSelectors:
- matchLabels:
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/rbac/clusterqueue_editor_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: '{{ include "kueue.fullname" . }}-clusterqueue-editor-role'
name: '{{ printf "%s-clusterqueue-editor-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
rbac.kueue.x-k8s.io/batch-admin: "true"
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/rbac/clusterqueue_viewer_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: '{{ include "kueue.fullname" . }}-clusterqueue-viewer-role'
name: '{{ printf "%s-clusterqueue-viewer-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
rbac.kueue.x-k8s.io/batch-admin: "true"
Expand Down
2 changes: 1 addition & 1 deletion charts/kueue/templates/rbac/job_editor_role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: '{{ include "kueue.fullname" . }}-job-editor-role'
name: '{{ printf "%s-job-editor-role" (include "kueue.fullname" .) | trunc 63 | trimSuffix "-" }}'
labels:
{{- include "kueue.labels" . | nindent 4 }}
rbac.kueue.x-k8s.io/batch-admin: "true"
Expand Down
Loading