fix: recyclarr cronjob

Signed-off-by: Ludovic Ortega <[email protected]>
M0NsTeRRR · Jun 26, 2024 · db6e511 · db6e511
1 parent 6e65e6c
commit db6e511
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 12 deletions.
diff --git a/charts/recyclarr/Chart.yaml b/charts/recyclarr/Chart.yaml
@@ -3,7 +3,7 @@ kubeVersion: ">=1.23.0-0"
 name: recyclarr
 description: recyclarr helm chart for Kubernetes
 type: application
-version: 0.1.2
+version: 0.1.4
 # image: ghcr.io/recyclarr/recyclarr
 appVersion: "6.0.2"
 maintainers:

diff --git a/charts/recyclarr/README.md b/charts/recyclarr/README.md
@@ -1,6 +1,6 @@
 # recyclarr
 
-![Version: 0.1.2](https://img.shields.io/badge/Version-0.1.2-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 6.0.2](https://img.shields.io/badge/AppVersion-6.0.2-informational?style=flat-square)
+![Version: 0.1.4](https://img.shields.io/badge/Version-0.1.4-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 6.0.2](https://img.shields.io/badge/AppVersion-6.0.2-informational?style=flat-square)
 
 recyclarr helm chart for Kubernetes
 
@@ -35,14 +35,17 @@ helm repo add adminafk https://helm-charts.adminafk.fr
 | affinity | object | `{}` | |
 | extraEnv | list | `[]` | Environment variables to add to the bazarr pods |
 | extraEnvFrom | list | `[]` | Environment variables from secrets or configmaps to add to the bazarr pods |
-| failedJobsHistoryLimit | int | `1` | |
 | fullnameOverride | string | `""` | |
 | image.pullPolicy | string | `"IfNotPresent"` | |
 | image.registry | string | `"ghcr.io"` | |
 | image.repository | string | `"recyclarr/recyclarr"` | |
 | image.sha | string | `""` | |
 | image.tag | string | `""` | Overrides the image tag whose default is the chart appVersion. |
 | imagePullSecrets | list | `[]` | |
+| job.failedJobsHistoryLimit | int | `1` | |
+| job.restartPolicy | string | `"never"` | |
+| job.schedule | string | `"@daily"` | |
+| job.successfulJobsHistoryLimit | int | `1` | |
 | jobAnnotations | object | `{}` | |
 | jobLabels | object | `{}` | |
 | nameOverride | string | `""` | |
@@ -51,7 +54,6 @@ helm repo add adminafk https://helm-charts.adminafk.fr
 | podSecurityContext.fsGroupChangePolicy | string | `"OnRootMismatch"` | |
 | replicaCount | int | `1` | |
 | resources | object | `{}` | |
-| scfhedule | string | `"@daily"` | |
 | securityContext.allowPrivilegeEscalation | bool | `false` | |
 | securityContext.capabilities.drop[0] | string | `"ALL"` | |
 | securityContext.privileged | bool | `false` | |
@@ -64,7 +66,6 @@ helm repo add adminafk https://helm-charts.adminafk.fr
 | serviceAccount.automount | bool | `true` | Automatically mount a ServiceAccount's API credentials? |
 | serviceAccount.create | bool | `true` | Specifies whether a service account should be created |
 | serviceAccount.name | string | `""` | If not set and create is true, a name is generated using the fullname template |
-| successfulJobsHistoryLimit | int | `1` | |
 | tolerations | list | `[]` | |
 | volumeMounts | list | `[]` | Additional volumeMounts on the output Deployment definition. |
 | volumes | list | `[]` | Additional volumes on the output Deployment definition. |
diff --git a/charts/recyclarr/templates/cronjob.yaml b/charts/recyclarr/templates/cronjob.yaml
@@ -6,9 +6,9 @@ metadata:
  {{- include "recyclarr.labels" . | nindent 4 }}
 spec:
  concurrencyPolicy: "Replace"
- successfulJobsHistoryLimit: {{ .Values.successfulJobsHistoryLimit }}
- failedJobsHistoryLimit: {{ .Values.failedJobsHistoryLimit }}
- schedule: {{ .Values.schedule }}
+ successfulJobsHistoryLimit: {{ .Values.job.successfulJobsHistoryLimit }}
+ failedJobsHistoryLimit: {{ .Values.job.failedJobsHistoryLimit }}
+ schedule: {{ .Values.job.schedule }}
  jobTemplate:
  spec:
  template:
@@ -23,6 +23,7 @@ spec:
  {{- toYaml . | nindent 12 }}
  {{- end }}
  spec:
+ restartPolicy: {{- with .Values.job.restartPolicy }}
  {{- with .Values.imagePullSecrets }}
  imagePullSecrets:
  {{- toYaml . | nindent 12 }}
@@ -31,7 +32,7 @@ spec:
  securityContext:
  {{- toYaml .Values.podSecurityContext | nindent 12 }}
  containers:
- - name: "ffff"
+ - name: {{ .Chart.Name }}
  securityContext:
  {{- toYaml .Values.securityContext | nindent 16 }}
  {{- if .Values.image.sha }}

diff --git a/charts/recyclarr/values.yaml b/charts/recyclarr/values.yaml
@@ -12,9 +12,11 @@ imagePullSecrets: []
 nameOverride: ""
 fullnameOverride: ""
 
-scfhedule: "@daily"
-successfulJobsHistoryLimit: 1
-failedJobsHistoryLimit: 1
+job:
+ schedule: "@daily"
+ successfulJobsHistoryLimit: 1
+ failedJobsHistoryLimit: 1
+ restartPolicy: "never"
 
 # -- Environment variables to add to the bazarr pods
 extraEnv: []