Mark important values as required

opendevstack · Nov 2, 2023 · fa6270c · fa6270c
1 parent f7b51c3
commit fa6270c
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 16 deletions.
diff --git a/deploy/chart/templates/configmap-bitbucket.yaml b/deploy/chart/templates/configmap-bitbucket.yaml
@@ -5,5 +5,5 @@ metadata:
   labels:
     {{- include "chart.labels" . | nindent 4}}
 data:
-  url: '{{.Values.bitbucketUrl | trimSuffix "/"}}'
-  repoBase: '{{.Values.bitbucketUrl | trimSuffix "/"}}/scm'
+  url: '{{required "bitbucketUrl is required" .Values.bitbucketUrl | trimSuffix "/"}}'
+  repoBase: '{{required "bitbucketUrl is required" .Values.bitbucketUrl | trimSuffix "/"}}/scm'
diff --git a/deploy/chart/templates/configmap-cluster.yaml b/deploy/chart/templates/configmap-cluster.yaml
@@ -5,4 +5,4 @@ metadata:
   labels:
     {{- include "chart.labels" . | nindent 4}}
 data:
-  consoleUrl: '{{.Values.consoleUrl | trimSuffix "/"}}'
+  consoleUrl: '{{required "consoleUrl is required" .Values.consoleUrl | trimSuffix "/"}}'
diff --git a/deploy/chart/templates/configmap-nexus.yaml b/deploy/chart/templates/configmap-nexus.yaml
@@ -5,4 +5,4 @@ metadata:
   labels:
     {{- include "chart.labels" . | nindent 4}}
 data:
-  url: '{{.Values.nexusUrl | trimSuffix "/"}}'
+  url: '{{required "nexusUrl is required" .Values.nexusUrl | trimSuffix "/"}}'
diff --git a/deploy/chart/templates/deployment.yaml b/deploy/chart/templates/deployment.yaml
@@ -44,11 +44,11 @@ spec:
                   key: debug
                   name: ods-pipeline
             - name: ODS_STORAGE_PROVISIONER
-              value: '{{.Values.pipelineManager.storageProvisioner}}'
+              value: '{{required "pipelineManager.storageProvisioner is required" .Values.pipelineManager.storageProvisioner}}'
             - name: ODS_STORAGE_CLASS_NAME
-              value: '{{.Values.pipelineManager.storageClassName}}'
+              value: '{{required "pipelineManager.storageClassName is required" .Values.pipelineManager.storageClassName}}'
             - name: ODS_STORAGE_SIZE
-              value: '{{.Values.pipelineManager.storageSize}}'
+              value: '{{required "pipelineManager.storageSize is required" .Values.pipelineManager.storageSize}}'
             - name: ODS_PRUNE_MIN_KEEP_HOURS
               value: '{{int .Values.pipelineRunMinKeepHours}}'
             - name: ODS_PRUNE_MAX_KEEP_RUNS

diff --git a/deploy/chart/values.yaml b/deploy/chart/values.yaml
@@ -29,7 +29,7 @@ nexusUsername: ''
 
 # Cluster
 # URL (including scheme, without trailing slash) of the OpenShift Web Console.
-consoleUrl: 'http://example.com'
+consoleUrl: ''
 
 # Notification Webhook
 notification:
@@ -92,10 +92,10 @@ pipelineRunMaxKeepRuns: '20'
 # Pipeline Manager
 pipelineManager:
   # PVC (used for the pipeline workspace)
-  # Storage provisioner. On AWS backed clusters, use 'kubernetes.io/aws-ebs'.
-  storageProvisioner: 'kubernetes.io/aws-ebs'
-  # Storage class. On AWS backed clusters, use 'gp2'.
-  storageClassName: 'gp2'
+  # Storage provisioner.
+  storageProvisioner: ''
+  # Storage class.
+  storageClassName: ''
   # Storage size. Defaults to 2Gi unless set explicitly here.
   storageSize: '5Gi'
   # Number of replicas to run for the pipeline manager.

diff --git a/deploy/values.yaml.tmpl b/deploy/values.yaml.tmpl
@@ -21,7 +21,7 @@ consoleUrl: ''
 # Pipeline Manager
 pipelineManager:
   # PVC (used for the pipeline workspace)
-  # Storage provisioner. On AWS backed clusters, use 'kubernetes.io/aws-ebs'.
-  storageProvisioner: 'kubernetes.io/aws-ebs'
-  # Storage class. On AWS backed clusters, use 'gp2'.
-  storageClassName: 'gp2'
+  # Storage provisioner. On AWS backed clusters, use e.g. 'ebs.csi.aws.com'.
+  storageProvisioner: ''
+  # Storage class. On AWS backed clusters, use e.g. 'gp3-csi'.
+  storageClassName: ''