Remove retries from integration test suite
ClosedPublic
Actions

Authored by rexledesma on Jan 10 2021, 9:14 PM.

Details

Reviewers

max
• nate

Commits

R1:4640ae8e2c83: Remove retries from integration test suite

Summary

Integration test suite shouldn't be flaky anyways?

Test Plan

integration

Diff Detail

Repository

R1 dagster

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rexledesma created this revision.Jan 10 2021, 9:14 PM

Harbormaster completed remote builds in B24067: Diff 29261.Jan 10 2021, 9:34 PM

rexledesma requested review of this revision.Jan 10 2021, 9:34 PM

rexledesma added reviewers: max, • nate.Jan 11 2021, 4:32 PM

ambitious, I like it

This revision is now accepted and ready to land.Jan 11 2021, 4:49 PM

Closed by commit R1:4640ae8e2c83: Remove retries from integration test suite (authored by rexledesma). · Explain WhyJan 11 2021, 5:04 PM

This revision was automatically updated to reflect the committed changes.

rexledesma added a commit: R1:4640ae8e2c83: Remove retries from integration test suite.

rexledesma added a reverting change: D5932: Revert "Remove retries from integration test suite".Jan 12 2021, 7:02 PM

rexledesma added a reverting change: R1:03869485962f: Revert "Remove retries from integration test suite".Jan 12 2021, 7:39 PM

Revision Contents
Changeset List

Path

Size

.buildkite/

dagster-buildkite/

dagster_buildkite/

steps/

integration.py

1 line

Diff 29274

View Options

.buildkite/dagster-buildkite/dagster_buildkite/steps/integration.py

Remove retries from integration test suiteClosedPublicActions