Page MenuHomeElementl

Remove external_pipeline_from_run
ClosedPublic

Authored by dgibson on Mar 22 2021, 1:46 PM.

Details

Summary

This was only used in tests (the daemon fetches pipelines more efficiently now) - so change the tests that used it to be more like the daemon.

(The test that specifically tested the method was in utils_tests, which isn't covered by tox at all - so delete it and move the other test in that folder (which was failing) into core_tests instead)

Test Plan

Integration

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Mar 22 2021, 2:35 PM
Harbormaster failed remote builds in B27850: Diff 34108!

flaky integration test

This revision is now accepted and ready to land.Mar 22 2021, 2:46 PM
This revision was landed with ongoing or failed builds.Mar 22 2021, 3:09 PM
This revision was automatically updated to reflect the committed changes.