fix pin and add test

Abandoning for https://dagster.phacility.com/D4791

Does it not matter that the other libraries depend on dagster core?

@bob could you post the custom resource here just so we have one more place to find it if we need it again? After that I agree this can be abandoned for now

Should be ready for re-review

rebase and up dependency pin

fix helm liveness check

add client to dagster cli

rebase

lgtm

Is there anything we can do to make it easier for users to include/remember to include the binary in their user deployment images? It could just be bundled in dagster and invoked via a dagster cli but I don't think that's great, especially given that the binary is only available for linux. We could build our own client for it?

up

add to helm

rebase

edit invariant error

up

up

@catherinewu I think this has addressed most concerns.

up

up

fix helm_values.py

rebase

up

helm

rebase

various renamings

rebase

up

up

up

nits

Ah you're right Cat, this is redundant. Thanks for catching.

up

rebase

reduce flakiness

typo

Lgtm, would just add a util function

rebase in black fix

@nate @catherinewu This is ready if you guys want to take a look at the Celery and Helm changes

rebase and cleanup

up

rename

clean up

separate instance

up

enable queue in bk