Page MenuHomePhabricator

[2/n] Add helm schema validation for Scheduler configuration
Needs ReviewPublic

Authored by rexledesma on Oct 27 2020, 5:52 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Depends on D4895.

Test Plan

bk/helm lint

Diff Detail

Repository
R1 dagster
Branch
rl-json-schema-scheduler
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

rexledesma retitled this revision from [3/n] Add schema for scheduler to [2/n] Add helm schema for Scheduler configuration.Oct 28 2020, 12:20 AM
rexledesma edited the summary of this revision. (Show Details)
rexledesma retitled this revision from [2/n] Add helm schema for Scheduler configuration to [2/n] Add helm schema validation for Scheduler configuration.Oct 28 2020, 1:28 AM
rexledesma added a reviewer: Restricted Project.Oct 28 2020, 6:52 PM
rexledesma added inline comments.
python_modules/automation/automation/helm/schema/scheduler.py
38

support for conditional schemas isn't first class in pydantic so we'll have to override the schema here (https://github.com/samuelcolvin/pydantic/issues/529)

alangenfeld added inline comments.
helm/dagster/templates/configmap-instance.yaml
13

this is a breaking change right? do we want to do that ahead of 0.10.0?

python_modules/automation/automation/helm/schema/scheduler.py
38

maybe leave this is a comment in the code

separate out breaking changes from things we can schema-tize without breaking

This revision now requires changes to proceed.Oct 29 2020, 6:29 PM

Will wait until cherry pick mode to land this - removing reviewers for now

This revision now requires review to proceed.Mon, Nov 2, 7:35 PM