Truncate pre-delete-controller Job name to 63 characters

CHANGELOG.md

@@ -1,3 +1,8 @@
+## Unreleased
+
+Fix:
+* Helm: rename and truncate the pre-delete cleanup job to 63 characters: [GH-506](https://github.com/hashicorp/vault-secrets-operator/pull/501)
+
 ## 0.4.2 (December 7th, 2023)
 
 Fix:

chart/templates/deployment.yaml

@@ -147,7 +147,10 @@ spec:
 apiVersion: batch/v1
 kind: Job
 metadata:
-  name: {{ include "vso.chart.fullname" . }}-pre-delete-controller-cleanup
+  # This name is truncated because kubernetes applies labels to the job which contain the job and pod
+  # name, and labels are limited to 63 characters. If we do not truncate the user will not have any
+  # feedback until after runtime.
+  name: {{ printf "%s-%s" "pdcc" (include "vso.chart.fullname" .) | trunc 63 | trimSuffix "-" }}
   namespace: {{ .Release.Namespace }}
   labels:
   {{- include "vso.chart.labels" . | nindent 4 }}
@@ -159,7 +162,10 @@ metadata:
 spec:
   template:
     metadata:
-      name: {{ include "vso.chart.fullname" . }}-pre-delete-controller-cleanup
+      # This name is truncated because kubernetes applies labels to the job which contain the job and pod
+      # name, and labels are limited to 63 characters. If we do not truncate the user will not have any
+      # feedback until after runtime.
+      name: {{ printf "%s-%s" "pdcc" (include "vso.chart.fullname" .) | trunc 63 | trimSuffix "-" }}
     spec:
       serviceAccountName: {{ include "vso.chart.fullname" . }}-controller-manager
       securityContext:

test/unit/deployment.bats

@@ -691,3 +691,31 @@ load _helpers
    local actual=$(echo "$object" | yq '.[4]' | tee /dev/stderr)
    [ "${actual}" = "--bar=qux" ]
 }
+
+
+#--------------------------------------------------------------------
+# pre-delete-controller
+
+@test "controller/Deployment: pre-delete-controller Job name is not truncated by default" {
+  cd `chart_dir`
+  local object=$(helm template \
+      -s templates/deployment.yaml  \
+      . | tee /dev/stderr |
+      yq 'select(.kind == "Job") | .metadata' | tee /dev/stderr)
+
+  local actual=$(echo "$object" | yq '.name' | tee /dev/stderr)
+  [ "${actual}" = "pdcc-release-name-vault-secrets-operator" ]
+}
+
+@test "controller/Deployment: pre-delete-controller Job name is truncated to 63 characters" {
+  cd `chart_dir`
+  local object=$(helm template \
+      -s templates/deployment.yaml  \
+      --set fullnameOverride=abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz \
+      . | tee /dev/stderr |
+      yq 'select(.kind == "Job") | .metadata' | tee /dev/stderr)
+
+  local actual=$(echo "$object" | yq '.name' | tee /dev/stderr)
+  [ "${actual}" = "pdcc-abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdef" ]
+  [ "${#actual}" -eq 63 ]
+}