1.10.1 - fixes for validation check and migration job

Author: farioas

heartex/label-studio/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.10.1
+- Fix migraiton job to avoid accidental concurrency and reruns on failure.
+- Fix validation for deprecated feature flags in a format fflag-.
+
 ## 1.10.0
 - Added an optional migration job feature (supported only in GitOps and Helm) to support deployment workflows.
 

heartex/label-studio/Chart.yaml

@@ -5,7 +5,7 @@ home: https://labelstud.io/
 type: application
 icon: https://raw.githubusercontent.com/heartexlabs/label-studio/master/images/logo.png
 # Chart version
-version: 1.10.0
+version: 1.10.1
 # Label Studio release version
 appVersion: "1.20.0"
 kubeVersion: ">= 1.14.0-0"

heartex/label-studio/templates/_checkConfig.tpl

@@ -172,13 +172,13 @@ Label Studio:
 {{- end -}}
 {{/* END ls.checkConfig.gcsConfig */}}
 
-{{/* Ensure that feature flags has ff_ or fflag_ prefix */}}
+{{/* Ensure that feature flags has ff_, fflag_, or fflag- (deprecated) prefix */}}
 {{- define "ls.checkConfig.featureFlagsFFprefix" -}}
 {{- if .Values.global.featureFlags -}}
 {{- range $key, $value := .Values.global.featureFlags -}}
-{{- if and (not (hasPrefix "ff_" (printf "%s" $key))) (not (hasPrefix "fflag_" (printf "%s" $key))) }}
+{{- if and (not (hasPrefix "ff_" (printf "%s" $key))) (not (hasPrefix "fflag_" (printf "%s" $key))) (not (hasPrefix "fflag-" (printf "%s" $key))) }}
 Label Studio:
-  Feature Flags: flags should starts from `ff_` or `fflag_` in lower case. Please check spelling in `.Values.global.featureFlags`
+  Feature Flags: flags should start with `ff_`, `fflag_`, or `fflag-`(deprecated) in lower case. Please check spelling in `.Values.global.featureFlags`
 {{- end -}}
 {{- end -}}
 {{- end -}}

heartex/label-studio/templates/job-migrate.yaml

@@ -13,9 +13,9 @@ metadata:
     {{- toYaml .Values.migrationJob.annotations | nindent 4 }}
   {{- end }}
 spec:
-  {{- if .Values.migrationJob.backoffLimit }}
-  backoffLimit: {{ .Values.migrationJob.backoffLimit }}
-  {{- end }}
+  backoffLimit: 0
+  completions: 1
+  parallelism: 1
   template:
     spec:
       {{- if .Values.app.serviceAccount.create }}

heartex/label-studio/values.yaml

@@ -414,7 +414,6 @@ migrationJob:
   #  "argocd.argoproj.io/hook": PreSync
   #  "argocd.argoproj.io/hook-delete-policy": BeforeHookCreation
   annotations: {}
-  backoffLimit: 0
 
 rqworker:
   enabled: true