Page MenuHomePhabricator

dagster-aws test fixes
ClosedPublic

Authored by nate on Aug 7 2019, 11:17 PM.

Details

Test Plan

unit

Diff Detail

Repository
R1 dagster
Branch
enable_emr_test
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

nate created this revision.Aug 7 2019, 11:17 PM
nate retitled this revision from Re-enable EMR moto test to dagster-aws test fixes.Aug 8 2019, 2:45 PM
nate updated this revision to Diff 3514.Aug 8 2019, 2:45 PM
nate retitled this revision from dagster-aws test fixes to Re-enable EMR moto test.

up

Harbormaster completed remote builds in B2809: Diff 3515.
nate retitled this revision from Re-enable EMR moto test to dagster-aws test fixes.Aug 8 2019, 3:26 PM
max accepted this revision.Aug 8 2019, 4:48 PM
max added a subscriber: max.
max added inline comments.
python_modules/libraries/dagster-aws/dagster_aws_tests/s3_tests/test_intermediate_store.py
108

I think we should try to move to calling these pipeline_def everywhere

This revision is now accepted and ready to land.Aug 8 2019, 4:48 PM
nate updated this revision to Diff 3577.Aug 10 2019, 1:25 AM

rebase

nate updated this revision to Diff 3607.Aug 12 2019, 8:33 PM

rebase

This revision was automatically updated to reflect the committed changes.