]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
doc/SubmittingPatches-backports.rst: clarify backport PR labels
authorNathan Cutler <ncutler@suse.com>
Fri, 19 Jun 2020 13:48:24 +0000 (15:48 +0200)
committerNathan Cutler <ncutler@suse.com>
Fri, 19 Jun 2020 14:30:40 +0000 (16:30 +0200)
Signed-off-by: Nathan Cutler <ncutler@suse.com>
SubmittingPatches-backports.rst

index 514d0f784dc619bfc867d44a16e74a625a6fcca1..b88d13ebdfb81f847cb77c51e616525c69a287f4 100644 (file)
@@ -334,14 +334,19 @@ Once the backport PR is open, the first order of business is to set the
 Milestone tag to the stable release the backport PR is targeting. For example,
 if the PR is targeting "nautilus", set the Milestone tag to "nautilus".
 
-If you don't have sufficient GitHub permissions to set the Milestone, add
-a comment to the PR with the following content::
-
-    @ceph/backport-admins
-
-This will alert the `Stable Releases and Backports team`_ to your PR, and
-someone will ensure your PR gets the proper labels.
-
+If you don't have sufficient GitHub permissions to set the Milestone, don't
+worry. Members of the `Stable Releases and Backports team`_ periodically run
+a script (``ceph-backport.sh --milestones``) which scans all PRs targetting stable
+branches and automatically adds the correct Milestone tag if it is missing.
+
+Next, check which component label was applied to the master PR corresponding to
+this backport, and double-check that that label is applied to the backport PR as
+well. For example, if the master PR carries the component label "core", the
+backport PR should also get that label.
+
+In general, it is the responsibility of the `Stable Releases and Backports
+team`_ to ensure that backport PRs are properly labelled. If in doubt, just
+leave the labelling to them.
 
 .. _`backport PR reviewing`:
 .. _`backport PR testing`: