Page MenuHomePhabricator

Fix Celery configuration issue

Authored by nate on Feb 12 2020, 2:37 AM.



Previously if we had multiple configs for Celery, we were printing them erroneously entirely on one line:

broker_url = 'pyampqp://foo@bar:1234//'foo = bar

instead of:

broker_url = 'pyampqp://foo@bar:1234//'
foo = bar

I removed the indenting line printer since it is a little overkill for this use case, and updated the test to cover this scenario

Test Plan


Diff Detail

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

Event Timeline

nate created this revision.Feb 12 2020, 2:37 AM
nate edited the summary of this revision. (Show Details)Feb 12 2020, 2:43 AM
nate added reviewers: max, prha, schrockn.
alangenfeld accepted this revision.Feb 12 2020, 3:57 PM
This revision is now accepted and ready to land.Feb 12 2020, 3:57 PM
This revision was automatically updated to reflect the committed changes.