Page MenuHomeElementl

better error message for invalid combos of parameters in pipeline execute command

Authored by dgibson on Oct 2 2020, 6:47 PM.



Reach feature parity with the launch command.

Test Plan

BK, run pipeline execute with both -m and -d set

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

This is definitely an improvement on the current state. That said, is there some fundamental reason why these combinations shouldn't work together? If I were a user coming to the system, I'd expect that I should be able to dagster pipeline execute -m anything I'm able to python -m.

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 2 2020, 7:05 PM
Harbormaster failed remote builds in B19085: Diff 23179!

I’m not sure, we should check with @prha when he is back on Monday.

test failure was flaky

@sandyryza this diff may have more context on why this behavior exists:

This revision is now accepted and ready to land.Oct 6 2020, 6:40 PM

up, make help message specify that -d can only be used with -f