Page MenuHomeElementl

feat!: add schema in Helm for run coordinator
ClosedPublic

Authored by rexledesma on Jul 7 2021, 4:50 PM.
Tags
None
Referenced Files
F2890907: D8737.diff
Fri, Mar 24, 3:13 AM
Unknown Object (File)
Thu, Mar 16, 12:25 PM
Unknown Object (File)
Mon, Mar 13, 3:48 AM
Unknown Object (File)
Fri, Mar 10, 5:31 PM
Unknown Object (File)
Sat, Feb 25, 11:14 AM
Unknown Object (File)
Feb 19 2023, 12:25 PM
Unknown Object (File)
Feb 15 2023, 11:04 PM
Unknown Object (File)
Feb 15 2023, 3:29 AM
Subscribers
None

Details

Summary

Previously, the run coordinator was allowed a custom class, but its configuration did not
fit the convention of "type:, config:" since that would be a breaking change. Here, we migrate
the schema to this convention.

Will change queuedRunCoordinator to runCoordinator in a followup.

Test Plan

pytest
integration

Diff Detail

Repository
R1 dagster
Branch
rl/add-types-to-dagster-daemon (branched from master)
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Jul 7 2021, 5:26 PM
rexledesma retitled this revision from feat: add schema in Helm for run coordinator to feat!: add schema in Helm for run coordinator.Jul 7 2021, 6:07 PM