Project

General

Profile

Feature #52387

backport-create-issue: set the priority of the backport issue

Added by Loïc Dachary over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

Source:
Development
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

The backport issue should have the same priority as the issue it is derived from. This is a hint that helps backporters figure out which issues should be worked on first.

If the backport issues had their priority set manually, they would likely be different from the priority of the issue they are derived from. But noone is doing this as of 2021 and as a result all backport issues have the same priority. When someone has to pick a backport issue to work on, they chose at random.

Setting the priority of a backport is better because there is a good chance that an issue that had to be fixed urgently in master also has a higher backport priority.

0001-backport-create-issue-set-backport-priority.patch View (1.01 KB) Loïc Dachary, 08/24/2021 11:29 AM


Related issues

Related to Ceph - Feature #51222: ceph-backport.sh: minimal automated test Fix Under Review

History

#1 Updated by Loïc Dachary over 1 year ago

  • Blocked by Feature #51222: ceph-backport.sh: minimal automated test added

#2 Updated by Loïc Dachary over 1 year ago

@cory here is the patch that sets the priority of the backports to be the same as the priority of the main issue. There are no tests in the main tree for this script just yet (will be not too much work once tests run for ceph-backport.sh). There are tests for this change that you can review . Running them on your laptop is also possible and they also run in the CI of this forge

If this looks ok to you, I'd be grateful if you could create a pull request with the patch. I cannot do it myself because I actively boycott GitHub

#3 Updated by Loïc Dachary over 1 year ago

  • Status changed from In Progress to Fix Under Review

#4 Updated by Loïc Dachary over 1 year ago

  • Subject changed from ceph-backport.sh: set the priority of the backport issue to backport-create-issue: set the priority of the backport issue

#5 Updated by Loïc Dachary about 1 year ago

  • Pull request ID set to 43032

#6 Updated by Patrick Donnelly about 1 year ago

  • Target version set to v17.0.0
  • Source set to Development

#7 Updated by Patrick Donnelly about 1 year ago

  • Related to Feature #51222: ceph-backport.sh: minimal automated test added

#8 Updated by Patrick Donnelly about 1 year ago

  • Blocked by deleted (Feature #51222: ceph-backport.sh: minimal automated test)

#9 Updated by Patrick Donnelly about 1 year ago

  • Status changed from Fix Under Review to Resolved

Also available in: Atom PDF