Merge pull request #153 from sergey-aerokube/master

fix if-else conditions
aerokube · Jun 16, 2022 · 8755b62 · 8755b62
2 parents 59b8c42 + fdbe5c6
commit 8755b62
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/browser-ops/templates/cron-job.yaml b/browser-ops/templates/cron-job.yaml
@@ -31,13 +31,10 @@ spec:
             - name: LIST
             {{- if eq "\"all\"" (.Values.browserImageVersions | quote) }}
               value: "0"
-            {{- else }}
-            {{- if eq "\"recent\"" (.Values.browserImageVersions | quote) }}
+            {{- else if eq "\"recent\"" (.Values.browserImageVersions | quote) }}
               value: "-1"
             {{- else }}
               value: {{ .Values.browserImageVersions | quote }}
             {{- end }}
             {{- end }}
-            {{- end }}
           restartPolicy: Never
-
diff --git a/browser-ops/values.yaml b/browser-ops/values.yaml
@@ -1,7 +1,6 @@
 ##
 ## List of browsersets names to update
 ##
-
 browsersets:
 - default
 
@@ -10,17 +9,14 @@ browsersets:
 ##
 ##   (102.0 vs 102.0.1245.30-1)
 ##
-
 browserImageTagFormat: short
 
 ##
-## Browser versions appeared in web interface:  all | recent | positive number
+## Browser versions appeared in web interface:  all | recent | positive number(last N versions)
 ##
-
 browserImageVersions: recent
 
 ##
 ## Schedule
 ##
-
 schedule: "0 1 * * *"