Attempts to set the correct readonly values (#1607)

opensource-observer · Jun 9, 2024 · 4e94b80 · 4e94b80
1 parent 742d360
commit 4e94b80
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/ops/helm-charts/oso-dagster/Chart.yaml b/ops/helm-charts/oso-dagster/Chart.yaml
@@ -3,7 +3,7 @@ name: oso-dagster
 description: Extension of the dagster template
 
 type: application
-version: 0.1.10
+version: 0.1.11
 appVersion: "1.0.0"
 dependencies:
 - name: dagster

diff --git a/ops/helm-charts/oso-dagster/templates/_helpers.tpl b/ops/helm-charts/oso-dagster/templates/_helpers.tpl
@@ -31,5 +31,5 @@ postgres_db:
 {{- $name := default "webserver" .Values.dagsterWebserver.nameOverride -}}
 {{- $fullname := include "dagster.fullname" . -}}
 {{- printf "%s-%s" $fullname  $name | trunc 63 | trimSuffix "-" -}}
-{{- if .webserverReadOnly -}} --read-only {{- end -}}
+{{- if .webserverReadOnly -}} -read-only {{- end -}}
 {{- end -}}
diff --git a/ops/helm-charts/oso-dagster/values.yaml b/ops/helm-charts/oso-dagster/values.yaml
@@ -8,7 +8,6 @@ dagster:
   postgresql:
     enabled: false
     postgresqlPassword: "gcp:secretmanager:dagster-db-password/versions/1"
-  enableReadOnly: true
 
 configMap:
   name: "dagster-extra-env-config-map"

diff --git a/ops/k8s-apps/base/dagster/dagster.yaml b/ops/k8s-apps/base/dagster/dagster.yaml
@@ -35,6 +35,8 @@ spec:
       pythonLogs:
         pythonLogLevel: "INFO"
       nameOverride: "dagster"
+      dagsterWebserver:
+        enableReadOnly: true
       ingress:
         enabled: true
         ingressClassName: ingress-internal-cloudflare