Page MenuHomeElementl

refactor(helm): remove enabled flag in run coordinator
AcceptedPublic

Authored by rexledesma on Jul 9 2021, 4:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 12, 4:39 PM
Unknown Object (File)
Dec 30 2022, 6:44 AM
Unknown Object (File)
Nov 18 2022, 12:58 AM
Unknown Object (File)
Nov 18 2022, 12:58 AM
Unknown Object (File)
Nov 18 2022, 12:58 AM
Unknown Object (File)
Nov 14 2022, 5:29 PM
Unknown Object (File)
Nov 6 2022, 3:38 AM
Unknown Object (File)
Nov 3 2022, 4:15 AM
Subscribers
None

Details

Summary

I forgot the DefaultRunCoordinator is a thing. Use that as an enum to remove the odd enabled toggle.

Test Plan

pytest
integration

Diff Detail

Repository
R1 dagster
Branch
rl/remove-enabled-run-coordinator (branched from master)
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

lgtm besides the comment

helm/dagster/values.yaml
757–758

presumably there are other reasons to want to use the queuedRunCoordinator?

This revision is now accepted and ready to land.Jul 9 2021, 3:00 PM
helm/dagster/values.yaml
757–758

oops