Project

General

Profile

Actions

Feature #52387

closed

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

Added by Loïc Dachary over 2 years ago. Updated over 2 years 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.


Files


Related issues 1 (1 open0 closed)

Related to Ceph - Feature #51222: ceph-backport.sh: minimal automated testFix Under ReviewLoïc Dachary

Actions
Actions #1

Updated by Loïc Dachary over 2 years ago

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

Updated by Loïc Dachary over 2 years 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

Actions #3

Updated by Loïc Dachary over 2 years ago

  • Status changed from In Progress to Fix Under Review
Actions #4

Updated by Loïc Dachary over 2 years 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
Actions #5

Updated by Loïc Dachary over 2 years ago

  • Pull request ID set to 43032
Actions #6

Updated by Patrick Donnelly over 2 years ago

  • Target version set to v17.0.0
  • Source set to Development
Actions #7

Updated by Patrick Donnelly over 2 years ago

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

Updated by Patrick Donnelly over 2 years ago

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

Updated by Patrick Donnelly over 2 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF