use safe_tempfile_path

Adding Apache license for code derived from Airflow

py2

up

yep, manually verified. thanks!

ok, I don't think we have any explicit reference to PresetDefinition.from_files in the docs

let me check the docs also

up

up

up

py2

up

Cool, thank you for taking a close look! Updated to address comments.

up

up

up

up

up

up

up

experiment with graphql query instead of execute_solid_within_pipeline()

up

rebase

up

Thanks for doing this! Looks reasonable enough to me. Do you want to update Buildkite also to run bin/clean.py on every merge to master?

looks reasonable to me!

up

up

up

up

up

up

The Wheel of Time turns, and Ages come and pass, leaving memories that become legend. Legend fades to myth, and even myth is long forgotten when the Age that gave it birth comes again

This wasn't working before, because we were servicing the "default" queue by default; but dagster-celery uses "dagster" by default https://github.com/dagster-io/dagster/blob/master/python_modules/libraries/dagster-celery/dagster_celery/defaults.py#L13

I think the repository.yaml constraint is reasonable, but maybe we can relax the requirement for the entrypoint being set? I think in our other images we haven't set this, that way the end user can choose to run dagster, dagster-graphql, dagit, etc. on the same image depending on use case

this is super nice and clean

marking planned changes again—going to continue chipping away at this, but wanted to get WIP off my laptop

up