Page MenuHomePhabricator

Error for invalid Celery config
ClosedPublic

Authored by nate on Mon, Feb 10, 1:01 AM.

Details

Summary

We want to catch the case when non-distributed storage is used with remote execution. See also https://github.com/dagster-io/dagster/issues/2132

Test Plan

unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nate created this revision.Mon, Feb 10, 1:01 AM
nate edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Mon, Feb 10, 7:39 PM
alangenfeld added inline comments.Mon, Feb 10, 7:40 PM
python_modules/dagster-celery/dagster_celery/engine.py
44

it will be nice to deal with this better re: #2132

This revision was automatically updated to reflect the committed changes.