chore: add imagePullSecrets and slightly refac the statefulset

README.md

@@ -94,6 +94,7 @@ You should be good to go!
 ### Global
 
 | Name                      | Description                    | Value |
-| ---------------------- | ------------------------------ | ----- |
+| ------------------------- | ------------------------------ | ----- |
 | `global.imageRegistry`    | global image registry override | `""`  |
+| `global.imagePullSecrets` | global image pull secrets      | `""`  |
 | `global.storageClass`     | global storage class override  | `""`  |

templates/statefulset.yaml

@@ -52,7 +52,6 @@ spec:
           env:
             {{- toYaml .Values.statefulset.dind.extraEnvs | nindent 12 }}
           {{- end }}
-          restartPolicy: Always
           securityContext:
             privileged: true
           startupProbe:
@@ -75,6 +74,7 @@ spec:
             {{- with .Values.statefulset.dind.extraVolumeMounts }}
             {{- toYaml . | nindent 12 }}
             {{- end }}
+      restartPolicy: Always
       containers:
         - name: act-runner
           image: "{{ include "gitea.actions.actRunner.image" . }}"
@@ -109,6 +109,12 @@ spec:
             {{- with .Values.statefulset.actRunner.extraVolumeMounts }}
             {{- toYaml . | nindent 12 }}
             {{- end }}
+      {{- if .Values.global.imagePullSecrets }}
+      imagePullSecrets:
+      {{- range .Values.global.imagePullSecrets }}
+        - name: {{ . }}
+      {{- end }}
+      {{- end }}
       {{- range $key, $value := .Values.statefulset.nodeSelector }}
       nodeSelector:
         {{ $key }}: {{ $value | quote }}
@@ -135,7 +141,9 @@ spec:
         name: data-act-runner
       spec:
         accessModes: [ "ReadWriteOnce" ]
-        {{- include "gitea.actions.persistence.storageClass" . | nindent 8 }}
+        {{- if .Values.global.storageClass }}
+        {{- include "gitea.actions.persistence.storageClass" . | indent 8 }}
+        {{- end }}
         resources:
           requests:
             storage: {{ .Values.statefulset.persistence.size }}

unittests/helm/config-act-runner.yaml

@@ -48,7 +48,7 @@ tests:
       enabled: true
       statefulset:
         actRunner:
-          config:
+          config: |
             container:
               valid_volumes:
                 - /var/run/docker.sock

values.yaml

@@ -48,6 +48,22 @@ statefulset:
   extraVolumes: []
   securityContext: {}
 
+  dind:
+    registry: ""
+    repository: docker
+    tag: 28.3.3-dind
+    digest: ""
+    pullPolicy: IfNotPresent
+    fullOverride: ""
+    extraVolumeMounts: []
+
+    # If the container keeps crashing in your environment, you might have to add the `DOCKER_IPTABLES_LEGACY` environment variable.
+    # See https://github.com/docker-library/docker/issues/463#issuecomment-1881909456
+    extraEnvs:
+      []
+      #  - name: "DOCKER_IPTABLES_LEGACY"
+      #    value: "1"
+
   actRunner:
     registry: "docker.gitea.com"
     repository: act_runner
@@ -73,22 +89,6 @@ statefulset:
         require_docker: true
         docker_timeout: 300s
 
-  dind:
-    registry: ""
-    repository: docker
-    tag: 28.3.3-dind
-    digest: ""
-    pullPolicy: IfNotPresent
-    fullOverride: ""
-    extraVolumeMounts: []
-
-    # If the container keeps crashing in your environment, you might have to add the `DOCKER_IPTABLES_LEGACY` environment variable.
-    # See https://github.com/docker-library/docker/issues/463#issuecomment-1881909456
-    extraEnvs:
-      []
-      #  - name: "DOCKER_IPTABLES_LEGACY"
-      #    value: "1"
-
   persistence:
     size: 1Gi
 
@@ -121,7 +121,9 @@ giteaRootURL: ""
 ## @section Global
 #
 ## @param global.imageRegistry global image registry override
+## @param global.imagePullSecrets global image registry pull secrets
 ## @param global.storageClass global storage class override
 global:
   imageRegistry: ""
+  imagePullSecrets: []
   storageClass: ""