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)
Tue, Aug 9, 12:18 AM
Unknown Object (File)
Thu, Jul 28, 11:58 AM
Unknown Object (File)
Wed, Jul 27, 7:18 AM
Unknown Object (File)
Sun, Jul 24, 11:23 PM
Unknown Object (File)
Sat, Jul 23, 2:31 AM
Unknown Object (File)
Wed, Jul 20, 6:41 PM
Unknown Object (File)
Tue, Jul 12, 6:01 AM
Unknown Object (File)
Tue, Jul 12, 6:01 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