On Wed, Jan 2, 2019, at 3:18 AM, Dmitry Tantsur wrote:
Hi all and happy new year :)
As you know, tempest plugins are branchless, so the CI of ironic- tempest-plugin has to run tests on all supported branches. Currently it amounts to 16 (!) voting devstack jobs. With each of them have some small probability of a random failure, it is impossible to land anything without at least one recheck, usually more.
The bad news is, we only run master API tests job, and these tests are changed more often that the other. We already had a minor stable branch breakage because of it [1]. We need to run 3 more jobs: for Pike, Queens and Rocky. And I've just spotted a missing master multinode job, which is defined but does not run for some reason :(
Here is my proposal to deal with gate bloat on ironic-tempest-plugin:
1. Do not run CI jobs at all for unsupported branches and branches in extended maintenance. For Ocata this has already been done in [2].
2. Make jobs running with N-3 (currently Pike) and older non-voting (and thus remove them from the gate queue). I have a gut feeling that a change that breaks N-3 is very likely to break N-2 (currently Queens) as well, so it's enough to have N-2 voting.
3. Make the discovery and the multinode jobs from all stable branches non-voting. These jobs cover the tests that get changed very infrequently (if ever). These are also the jobs with the highest random failure rate.
Has any work been done to investigate why these jobs fail? And if not maybe we should stop running the jobs entirely. Non voting jobs that aren't reliable will just get ignored.
4. Add the API tests, voting for Queens to master, non-voting for Pike (as proposed above).
This should leave us with 20 jobs, but with only 11 of them voting. Which is still a lot, but probably manageable.
The corresponding change is [3], please comment here or there.
Dmitry
[1] https://review.openstack.org/622177 [2] https://review.openstack.org/621537 [3] https://review.openstack.org/627955