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
Unknown Object (File)
Thu, May 25, 5:14 AM
Unknown Object (File)
Fri, May 19, 6:17 PM
Unknown Object (File)
Wed, May 17, 7:14 AM
Unknown Object (File)
Tue, May 16, 6:19 AM
Unknown Object (File)
Mon, May 15, 3:59 AM
Unknown Object (File)
Mon, May 15, 12:54 AM
Unknown Object (File)
Sun, May 14, 9:55 PM
Unknown Object (File)
Fri, May 12, 5:56 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