search

dagster-io / dagster

An orchestration platform for the development, production, and observation of data assets.
https://dagster.io
Apache License 2.0
11.05k stars 1.38k forks source link

Helm chart doesn't pass run_k8s_config to CeleryK8sRunLauncher #21417

Open zzz4zzz opened 4 months ago

zzz4zzz commented 4 months ago

Dagster version

1.7.2

What's the issue?

Helm chart doesn't pass security_context, run_k8s_config to CeleryK8sRunLauncher. Config -> _run-launcher.tpl. As far I can see CeleryK8sRunLauncher accept these arguments.

This prevents job scheduling.

What did you expect to happen?

Pass missing values to CeleryK8sRunLauncher config.

How to reproduce?

No response

Deployment type

Dagster Helm chart

Deployment details

I need to set security context and tolerations to schedule a job.

Additional information

No response

Message from the maintainers

Impacted by this issue? Give it a 👍! We factor engagement into prioritization.

zzz4zzz commented 3 months ago

Probably related: https://github.com/dagster-io/dagster/pull/13111