Page MenuHomeElementl

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

Authored by rexledesma on Jul 9 2021, 4:56 AM.

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