Project

General

Profile

Actions

Bug #42474

closed

build-integration-branch merges newer PRs first

Added by Nathan Cutler over 4 years ago. Updated over 3 years ago.

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

0%

Source:
Tags:
Backport:
nautilus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

build-integration-branch takes PRs in the default sort order, which is "sort=created&direction=desc" - see https://developer.github.com/v3/pulls/#list-pull-requests

As a result, newer PRs get merged before older ones, which is unfair and leads to situations like https://github.com/ceph/ceph/pull/28452 where a contributor had to rebase their PR 3 times and were then requested to rebase a fourth time...


Related issues 1 (0 open1 closed)

Copied to Ceph - Backport #47878: nautilus: build-integration-branch merges newer PRs firstResolvedNathan CutlerActions
Actions #1

Updated by Nathan Cutler over 4 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 31132
Actions #2

Updated by Nathan Cutler over 3 years ago

  • Subject changed from build-integration-branch mergest newer PRs first to build-integration-branch merges newer PRs first
Actions #3

Updated by Nathan Cutler over 3 years ago

  • Backport changed from luminous, mimic, nautilus to nautilus
Actions #4

Updated by Nathan Cutler over 3 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #5

Updated by Nathan Cutler over 3 years ago

  • Copied to Backport #47878: nautilus: build-integration-branch merges newer PRs first added
Actions #6

Updated by Nathan Cutler over 3 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions

Also available in: Atom PDF