Page MenuHomeElementl

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

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

Details

Summary

Reach feature parity with the launch command.

Test Plan

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

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
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: https://dagster.phacility.com/D3670

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