default to k8s scheduler, update docs
AcceptedPublic
Actions

Authored by catherinewu on Wed, Oct 14, 4:11 PM.

Details

Reviewers

nate
johann
alangenfeld

Test Plan

Diff Detail

Repository

R1 dagster

Branch

default-k8s-scheduler

Lint

No Linters Available

Unit

No Unit Test Coverage

Event Timeline

catherinewu created this revision.Wed, Oct 14, 4:11 PM

Harbormaster completed remote builds in B19500: Diff 23689.Wed, Oct 14, 4:29 PM

catherinewu requested review of this revision.Wed, Oct 14, 4:29 PM

catherinewu added reviewers: nate, johann.Wed, Oct 14, 4:53 PM

Harbormaster completed remote builds in B19504: Diff 23694.Wed, Oct 14, 5:15 PM

catherinewu added a reviewer: alangenfeld.Fri, Oct 16, 6:44 PM

I think this will be fine - unless someone deploys this change in a helm upgrade that somehow doesnt take down dagit running cron.

Should https://dagster.phacility.com/source/dagster/browse/master/helm/dagster/templates/deployment-dagit.yaml$54-59 get updated at all? Not a huge deal to be running cron if not needed - could prevent ^ if deployment spec changes

helm/dagster/values.yaml
157	hm, may need to come up with better names around this

alangenfeld accepted this revision.Mon, Oct 19, 4:34 PM

This revision is now accepted and ready to land.Mon, Oct 19, 4:34 PM

Revision Contents
Changeset List

Path

Size

docs/

next/

src/

pages/

deploying/

k8s_part2.mdx

9 lines

helm/

dagster/

values.yaml

4 lines

Diff 23694

View Options

docs/next/src/pages/deploying/k8s_part2.mdx