https://dagster.io/ logo
#dagster-kubernetes
Title
# dagster-kubernetes
m

Michel Rouly

06/15/2021, 5:13 PM
Heya. I tried installing Dagster via the Helm chart on latest
master
, it's resulting in a CrashLoopBackoff in
dagster-dagit
and
dagster-daemon
on this log:
Copy code
Error 1: Received unexpected config entry "params" at path root:postgres_db. Expected: "{ db_name: (String | { env: String }) hostname: (String | { env: String }) password: (String | { env: String }) port?: (Int | { env: String }) username: (String | { env: String }) }
Looks like it isn't expecting
params
in the `ConfigMap`:
Copy code
config:
    postgres_db:
      username: test
      password:
        env: DAGSTER_PG_PASSWORD
      hostname: dagster-postgresql
      db_name:  test
      port: 5432
      params: 
        {}
(occurs in multiple sections in the default
dagster.yaml
)
r

rex

06/15/2021, 5:15 PM
What version of dagster are you using?
m

Michel Rouly

06/15/2021, 5:16 PM
Copy code
helm -n dagster install dagster dagster-0.11.3.tgz -f dagster-values.yaml
where
dagster-0.11.3.tgz
is from:
Copy code
helm package helm/dagster
in latest
master
i.e. local chart in the
master
branch
I'm circling back to that PR I opened up last week, had just merged in
master
to my branch and noticed this happening.
r

rex

06/15/2021, 5:19 PM
Ah let me clarify - what version of dagster are you using in the images in the chart?
m

Michel Rouly

06/15/2021, 5:21 PM
I'm not overriding the image tag, so it's getting the default from the Helm chart, which looks like 0.11.3.
Copy code
helm -n dagster list
NAME   	NAMESPACE	REVISION	UPDATED                             	STATUS  	CHART         	APP VERSION
dagster	dagster  	1       	2021-06-15 13:07:55.068108 -0400 EDT	deployed	dagster-0.11.3	0.11.3
and the container is:
Copy code
<http://docker.io/dagster/dagster-celery-k8s:0.11.3|docker.io/dagster/dagster-celery-k8s:0.11.3>
Was
params
added to a newer release of Dagster? Guess the default app version would just need updating in that case.
I'll try with the tag
0.11.13
👍🏼 1
Looks like there's no global key to set the same version between
daemon
and
dagit
(and possibly other containers) 😞 unless I'm missing it
r

rex

06/15/2021, 5:26 PM
Yeah- these app version/chart version are correct in the packaged versions of the chart (in the remote helm repository), but not on master. We can change the version to “dev” on master to make this explicit
👍 1
m

Michel Rouly

06/15/2021, 5:28 PM
Setting
dagit.image.tag
and
dagsterDaemon.image.tag
to
0.11.13
works with the chart on master, no more CrashLoopBackoff 👍
It would be nice I think to have a global
dagsterVersion
key, since there are so many components. But certainly not a critical thing.
2 Views