Page MenuHomeElementl

[1/2] Add helm configuration for dagster run launcher
ClosedPublic

Authored by rexledesma on Jan 8 2021, 1:11 AM.

Details

Summary

Depends on D4920

We consolidate celery and k8sRunLauncher under one field:
runLauncher. Similar to D5745 and D4920, we use enum types
to switch between the two supported run launchers.

Test Plan

integration
diff the output of helm template before and after change

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 8 2021, 1:27 AM
Harbormaster failed remote builds in B23891: Diff 29045!
Harbormaster returned this revision to the author for changes because remote builds failed.Jan 8 2021, 2:05 AM
Harbormaster failed remote builds in B23894: Diff 29048!
Harbormaster returned this revision to the author for changes because remote builds failed.Jan 8 2021, 10:24 AM
Harbormaster failed remote builds in B23903: Diff 29057!
Harbormaster returned this revision to the author for changes because remote builds failed.Jan 8 2021, 11:21 AM
Harbormaster failed remote builds in B23906: Diff 29060!
rexledesma added a child revision: Restricted Differential Revision.Jan 8 2021, 5:01 PM

i take it the new defaults will be in a followup diff?

helm/dagster/values.yaml
268

don't we want to default to k8srunlauncher?

406

and this false?

This revision is now accepted and ready to land.Jan 8 2021, 10:49 PM