Page MenuHomePhabricator

rm py2+windows tests
ClosedPublic

Authored by dgibson on Tue, Nov 10, 3:54 PM.

Details

Summary

core-tests on py27+windows have been off for a while now due to the tests just generally being slow overall and timing out fairly regularly. Since we're planning on removing py2 altogether as per nick's post today, I think it's reasonable to remove the few py2 tests that are left from the windows builds and clear up some azure capacity?

Test Plan

Azure run

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Nov 10, 4:10 PM
Harbormaster failed remote builds in B20941: Diff 25403!

Well i want folks a chance to weigh in. I don't consider the decision made yet. Maybe tmrw?

Sent via Superhuman ( https://sprh.mn/?vip=schrockn@elementl.com )

seems like we are locking in- land at your discretion

This revision is now accepted and ready to land.Thu, Nov 12, 6:02 PM
This revision was landed with ongoing or failed builds.Thu, Nov 12, 7:11 PM
Closed by commit R1:b08f5534a0b0: rm py2+windows tests (authored by dgibson). · Explain Why
This revision was automatically updated to reflect the committed changes.