Page MenuHomeElementl

consolidate event type to run status lookup
ClosedPublic

Authored by yuhan on Jul 14 2021, 5:52 PM.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 14 2021, 6:34 PM
Harbormaster failed remote builds in B33709: Diff 41626!
yuhan requested review of this revision.Jul 14 2021, 7:51 PM
prha added inline comments.
python_modules/dagster/dagster/core/events/__init__.py
153–161

consider adding this as a static method on PipelineRunStatus? e.g. from_pipeline_event

163

we don't use this anywhere, right?

This revision is now accepted and ready to land.Jul 14 2021, 10:48 PM
python_modules/dagster/dagster/core/events/__init__.py
163

will use it in https://dagster.phacility.com/D8900
but it depends on the api decision over there

This revision was landed with ongoing or failed builds.Jul 22 2021, 5:29 PM
This revision was automatically updated to reflect the committed changes.