Page MenuHomePhabricator

Fix Celery configuration issue
ClosedPublic

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

Details

Summary

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

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.Wed, Feb 12, 2:37 AM
nate edited the summary of this revision. (Show Details)Wed, Feb 12, 2:43 AM
nate added reviewers: max, prha, schrockn.
alangenfeld accepted this revision.Wed, Feb 12, 3:57 PM
This revision is now accepted and ready to land.Wed, Feb 12, 3:57 PM
This revision was automatically updated to reflect the committed changes.