Page MenuHomeElementl
Differential D5872

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

Authored by • rexledesma on Jan 8 2021, 1:11 AM.
Tags
None
Referenced Files
F2292441: D5872.diff
Fri, Jul 1, 10:32 AM
F2291535: D5872.diff
Fri, Jul 1, 3:28 AM
Unknown Object (File)
Thu, Jun 30, 6:40 AM
Unknown Object (File)
Tue, Jun 28, 1:47 AM
Unknown Object (File)
Mon, Jun 13, 4:14 AM
Unknown Object (File)
Sun, Jun 12, 3:15 PM
Unknown Object (File)
Thu, Jun 9, 12:52 PM
Subscribers
None

Details

Reviewers
alangenfeld
• nate
• catherinewu
Commits
R1:f5c4ec5f20df: [1/2] Add helm configuration for dagster run launcher
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
Branch
rl-helm-enumify-run-launcher
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

• rexledesma created this revision.Jan 8 2021, 1:11 AM
• rexledesma added a child revision: D5873: [2/2] Add helm schema for dagster run launcher.Jan 8 2021, 1:22 AM
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!
• rexledesma updated this revision to Diff 29048.Jan 8 2021, 1:29 AM

isort

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!
• rexledesma updated this revision to Diff 29057.Jan 8 2021, 9:45 AM

fix integration tests

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!
• rexledesma updated this revision to Diff 29060.Jan 8 2021, 10:41 AM

up

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 updated this revision to Diff 29069.Jan 8 2021, 3:39 PM

up

Harbormaster completed remote builds in B23912: Diff 29069.Jan 8 2021, 3:58 PM
• rexledesma requested review of this revision.Jan 8 2021, 3:58 PM
• rexledesma added reviewers: alangenfeld, • nate, • catherinewu.Jan 8 2021, 4:04 PM
• rexledesma added a child revision: Restricted Differential Revision.Jan 8 2021, 5:01 PM
• rexledesma edited the summary of this revision. (Show Details)Jan 8 2021, 6:08 PM
• rexledesma added a parent revision: D4920: Update Scheduler helm schema to use enum types.
alangenfeld accepted this revision.Jan 8 2021, 10:49 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
Closed by commit R1:f5c4ec5f20df: [1/2] Add helm configuration for dagster run launcher (authored by • rexledesma). · Explain WhyJan 8 2021, 11:24 PM
This revision was automatically updated to reflect the committed changes.
• rexledesma added a commit: R1:f5c4ec5f20df: [1/2] Add helm configuration for dagster run launcher.

Revision Contents
Changeset List

PathSize
2 lines
1 line
9 lines
helm/
dagster/
templates/
15 lines
12 lines
54 lines
59 lines
helpers/
31 lines
instance/
63 lines
2 lines
204 lines

Diff 29045

View Options

.yamllint.yaml

Loading...
View Options

CHANGES.md

Loading...
View Options

MIGRATION.md

Loading...
View Options

helm/dagster/templates/configmap-celery.yaml

Loading...
View Options

helm/dagster/templates/configmap-env-celery.yaml

Loading...
View Options

helm/dagster/templates/configmap-instance.yaml

Loading...
View Options

helm/dagster/templates/deployment-celery-queues.yaml

Loading...
View Options

helm/dagster/templates/helpers/_helpers.tpl

Loading...
View Options

helm/dagster/templates/helpers/instance/_run-launcher.tpl

Loading...
View Options

helm/dagster/templates/helpers/instance/_scheduler.tpl

Loading...
View Options

helm/dagster/values.yaml

Loading...