[devops] optimize fuel-ci queue handling

Bug #1308010 reported by Aleksandra Fedorova
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Aleksandra Fedorova

Bug Description

When patchset comes to the queue at Fuel-CI, all the previous patchsets with the same change_id should be removed from the queue.

Tags: devops
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

Consider the manual trigger as well.
E.g. "CI no test" comment should abort CI jobs for given patch set, if any

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

2nd use case is to ensure the trivial rebases would never trigger a CI jobs, if the patchset already has a +2 or more. But it should, if there are at least one -1 exist...

Mike Scherbakov (mihgen)
summary: - optimize fuel-ci queue handling
+ [devops] optimize fuel-ci queue handling
Changed in fuel:
status: New → Confirmed
milestone: none → 5.0
Revision history for this message
Andrew Woodward (xarses) wrote :

to the original issue, yes we need to remove previous patch sets in the ci queue. However we reuse change-id when we backport, so we should also check branch.

To Bogdan, all aborted CI should result in CI voting -1

Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

Openstack Infra has no workaround for the queue updates. They use zuul for those.

But we can create 'cleanup' job for every Gerrit event. It will go through the current queue and abort all the jobs with the same branch and change_id.

Changed in fuel:
milestone: 5.0 → 5.1
no longer affects: fuel/4.x
no longer affects: fuel/5.x
Igor Shishkin (teran)
Changed in fuel:
milestone: 5.1 → 6.0
Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

Gerrit Trigger plugin now has the "Build Current Patches Only" option which cancels jobs fro previous patchsets of the same change. Let's try it.

Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

Seems to work

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.