Page MenuHomeElementl
Differential D5646

Log error/interrupt events in the execute_run CLI call
ClosedPublic
Actions

Authored by • dgibson on Dec 16 2020, 8:15 PM.

Details

Reviewers
• johann
• nate
alangenfeld
Commits
R1:c24edab9f6ab: Log error/interrupt events in the execute_run CLI call
Summary

Getting errors surfaced here would be extremely useful fro debugging.

Test Plan

Integration

Diff Detail

Repository
R1 dagster
Branch
retryondupes
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

• dgibson created this revision.Dec 16 2020, 8:15 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Dec 16 2020, 8:44 PM
Harbormaster failed remote builds in B23021: Diff 27997!
• dgibson updated this revision to Diff 28003.Dec 16 2020, 8:45 PM

let's try this instead - log the exception event just like we log the other events

• dgibson updated this revision to Diff 28005.Dec 16 2020, 8:50 PM

up

• dgibson retitled this revision from RFC: Raise errors in the execute_run function to Log error/interrupt events in the execute_run CLI call.Dec 16 2020, 9:01 PM
• dgibson edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B23026: Diff 28003.Dec 16 2020, 9:03 PM
Harbormaster completed remote builds in B23028: Diff 28005.Dec 16 2020, 9:08 PM
• dgibson requested review of this revision.Dec 16 2020, 9:08 PM
• dgibson updated this revision to Diff 28040.Dec 16 2020, 10:56 PM

up

alangenfeld accepted this revision.Dec 16 2020, 11:00 PM

slamdunk

This revision is now accepted and ready to land.Dec 16 2020, 11:00 PM
Harbormaster completed remote builds in B23060: Diff 28040.Dec 16 2020, 11:16 PM
Closed by commit R1:c24edab9f6ab: Log error/interrupt events in the execute_run CLI call (authored by • dgibson). · Explain WhyDec 16 2020, 11:29 PM
This revision was automatically updated to reflect the committed changes.
• dgibson added a commit: R1:c24edab9f6ab: Log error/interrupt events in the execute_run CLI call.

Revision Contents
Changeset List

PathSize
python_modules/
dagster/
dagster/
cli/
14 lines

Diff 28005

View Options

python_modules/dagster/dagster/cli/api.py

Loading...