Page MenuHomePhabricator

Add more info to pipeline run status checks
AcceptedPublic

Authored by nate on Sep 26 2020, 2:30 AM.

Details

Test Plan

tbd

Diff Detail

Repository
R1 dagster
Branch
add_pipeline_run_info
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

nate requested review of this revision.Sep 26 2020, 2:46 AM
This revision is now accepted and ready to land.Sep 27 2020, 9:26 PM

any thoughts on how to test this? Would love to get some test coverage before merging

python_modules/libraries/dagster-k8s/dagster_k8s/client.py
202

i think this error is actually dropped here https://github.com/dagster-io/dagster/blob/master/python_modules/libraries/dagster-celery-k8s/dagster_celery_k8s/executor.py#L420

we can check for this error message in the event logs in terminated pipelines