Page MenuHomePhabricator

CI runs same tests twice
Closed, DuplicatePublicBug

Description

See screenshot:

For 511001 it runs tests twice. Test pipeline and Gate-and-submit. Gate-and-submit should have priority and when someone set +2 on patch while CI runs tests, it should abort it and start only Gate-and-submit.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 18 2019, 3:17 PM

Pausing the test pipeline whilst gate-and-submit runs, and re-using the results from gate-and-submit, would be very complicated and not save much.

hashar added a subscriber: hashar.May 21 2019, 9:12 PM

T105474 is to prevent patches that got a CR+2 from being added to test. So that is really similar.

I have identified a bunch of issues in doing so. But that needs a lot more though to define all the behavior, then write tests and finally figure out the implementation.

For now, lets consider this is just a dupe of T105474