HOWTO backport commits » History » Version 46
Nathan Cutler, 05/10/2019 08:09 AM
1 | 24 | Loïc Dachary | h3. Backport commits |
---|---|---|---|
2 | 1 | Loïc Dachary | |
3 | 36 | Nathan Cutler | General guidelines: |
4 | 37 | Nathan Cutler | |
5 | * backports should be done by cherry-picking commits that are in master, and in the most minimal way possible - see [[HOWTO_review_backport_PRs]] for more information |
||
6 | * it should be clear which bug the backport is fixing |
||
7 | * features can be backported only with permission from the relevant lead |
||
8 | * the tracker issue should state why the backport is necessary |
||
9 | 36 | Nathan Cutler | |
10 | Detailed instructions: |
||
11 | |||
12 | 35 | Loïc Dachary | * Pick one entry with the highest *Priority* field in the list of backports linked from [[HOWTO|the top level page]] (for instance "hammer backports":http://tracker.ceph.com/projects/ceph/issues?query_id=78) |
13 | * Assign the issue to yourself (or add a comment if you do not have permission) so that people know you start working on it. |
||
14 | 23 | Nathan Cutler | * <code>git clone</code> a fork of http://github.com/ceph/ceph |
15 | * <code>git checkout -b wip-$issue-$release origin/$release</code> to create a branch with a name that reflects the issue being fixed and the target release: this naming is convenient to select all branches containing backports for firefly or select all branches containing backports for a specific issue |
||
16 | 32 | Loïc Dachary | * <code>git cherry-pick -x</code> the commits from the pull request that was targeting <code>master</code> (not from the pull requests targeting stable branches -- with the exception of the upcoming stable branch, which can be used because all commits to the upcoming stable branch are merged into <code>master</code> on a regular basis) |
17 | 19 | Loïc Dachary | ** the original issue is linked in the *Related issues* section of the issue, as a *Copied from* link |
18 | ** the pull request or commit targeting master can be found by following the *Copied from* link and searching the original issue |
||
19 | 34 | Nathan Cutler | ** TIP: if the cherry-pick fails due to whitespace differences, use <code>-Xignore-all-space</code> |
20 | 39 | Nathan Cutler | * <code>git push -u origin wip-$issue-$release</code> to publish the branch on github and be able to create a pull request from it |
21 | 44 | Nathan Cutler | * Create the pull request, targeting the branch you are backporting to, set milestone as appropriate, prefix the PR title with <code>$release: </code> and put the backport tracker URL in the PR description |
22 | 39 | Nathan Cutler | * Edit the backport tracker issue, putting the GitHub PR URL in the description - now the backport tracker and PR are interlinked |
23 | * Set the backport tracker status to "In Progress", and assign it to yourself |
||
24 | |||
25 | Do not merge the pull request yet. Merging only takes place after the backports pass [[HOWTO run integration and upgrade tests|integration and upgrade tests]] and with proper approval as described in [[HOWTO merge commits from the integration branch]]. |
||
26 | |||
27 | All commits being backported must be cherry-picked from master. There are exceptions to this rule (see https://github.com/ceph/ceph/pull/4175 for instance: it fixes a regression introduced by an incorrect resolution of a backport) but they are rare. When a backport is done to fix a bug that shows in the integration tests or upgrade tests run by cron, testing the backport in an integration branch is redundant because the existing tests will keep failing. |
||
28 | |||
29 | How to automate: |
||
30 | |||
31 | 16 | Loïc Dachary | * generate a *$github_token* by "following the GitHub instructions":https://help.github.com/articles/creating-an-access-token-for-command-line-use/ |
32 | 14 | Loïc Dachary | * retrieve the *$redmine_key*: it is a token that will allow you to script issue updates |
33 | ** login http://tracker.ceph.com/ |
||
34 | ** visit http://tracker.ceph.com/my/account |
||
35 | ** look for *API access key* in the page (Control-f) |
||
36 | ** click on the *Show* link just under *API access key* |
||
37 | ** copy the hexadecimal number it shows |
||
38 | 1 | Loïc Dachary | ** redmine_key=b586ce6a7a936e71f351c93ac0b65a588d4333 |
39 | 40 | Nathan Cutler | * retrieve the *$redmine_user_id* - this is an integer, not your username. It can be found by logging in at http://tracker.ceph.com and then hovering the mouse over the Redmine username at the top right of the page - where it says "Logged in as *myusername*" and examining the URL. The URL will be something like http://tracker.ceph.com/users/2544 where 2544 is the *$redmine_user_id* value. |
40 | 41 | Nathan Cutler | * grab the "ceph-backport.sh" script from src/script in "master" branch of https://github.com/ceph/ceph.git |
41 | 39 | Nathan Cutler | * make a "backport_common.sh" script that sets the redmine_key, redmine_user_id, and github_token variables to the right values and make sure the "source backport_common.sh" command in the script will not fail |
42 | 42 | Nathan Cutler | * follow the instructions in the comment block at the top of "ceph-backport.sh" |
43 | 39 | Nathan Cutler | |
44 | 45 | Nathan Cutler | Cherry-picking notes: |
45 | |||
46 | The "traditional" method is to cherry-pick each commit from the master PR in turn, and resolve conflicts as you go. It is also possible to cherry-pick all the commits at once. Since github generates branches for every PR, one can use the following trick: |
||
47 | |||
48 | <pre> |
||
49 | git fetch <upstream> pull/<prr-num>/head:pr-<prnum> |
||
50 | git cherry-pick -x pr-<pr-num>~<num-commits>..pr-<pr-num> |
||
51 | </pre> |
||
52 | |||
53 | Adding following to somewhere bash can find can make things easier |
||
54 | |||
55 | <pre> |
||
56 | prfetch() { |
||
57 | git fetch upstream pull/${1}/head:pr-${1} |
||
58 | } |
||
59 | </pre> |
||
60 | |||
61 | 46 | Nathan Cutler | Notes on what the ceph-backport.sh script is doing: |
62 | 39 | Nathan Cutler | |
63 | 23 | Nathan Cutler | * create a pull request from the <code>wip-$issue-$release</code> branch with the same title as the issue |
64 | 7 | Loïc Dachary | <pre> |
65 | 10 | Loïc Dachary | account=myaccount |
66 | 7 | Loïc Dachary | eval title=$(curl --silent 'http://tracker.ceph.com/issues/'$issue.json?key=$redmine_key | jq .issue.subject) ; echo $title |
67 | 10 | Loïc Dachary | number=$(curl --silent --data-binary '{"title":"'"$title"'","head":"'$account':wip-'$issue-$release'","base":"'$release'","body":"http://tracker.ceph.com/issues/'$issue'"}' 'https://api.github.com/repos/ceph/ceph/pulls?access_token='$github_token | jq .number) |
68 | </pre> |
||
69 | 13 | Loïc Dachary | * assigned the pull request to yourself (because it needs to got through integration tests before it can be approved by the original author), add the relevant labels (rgw, core, bug fix, feature...) and set the milestone to $release (requires write permission to the ceph repository) |
70 | 10 | Loïc Dachary | <pre> |
71 | component=core ; curl --silent --data-binary '{"milestone":"'$release_number'","assignee":"'$account'","labels":["bug fix","'$component'"]}' 'https://api.github.com/repos/ceph/ceph/issues/'$number'?access_token='$github_token |
||
72 | </pre> |
||
73 | * wait for the make check bot to run the *run-make-check.sh* script on the branch that was pushed and verify it works |
||
74 | 29 | Loïc Dachary | * Set the issue description to only contain the URL to the pull request and change the status to *In Progress* |
75 | 1 | Loïc Dachary | <pre> |
76 | 29 | Loïc Dachary | redmine_status=2 # In Progress |
77 | curl --verbose -X PUT --header 'Content-type: application/json' --data-binary '{"issue":{"description":"https://github.com/ceph/ceph/pull/'$number'","status_id":'$redmine_status'}}' 'http://tracker.ceph.com/issues/'$issue.json?key=$redmine_key |
||
78 | 2 | Loïc Dachary | </pre> |
79 | * if there is a conflict explain how it was resolved in the commit message, below the Conflicts line. For instance: |
||
80 | <pre> |
||
81 | commit c60da2f3c34e7325c748d2d6e55140a0a30013fd |
||
82 | Author: Samuel Just <sjust@redhat.com> |
||
83 | Date: Thu Nov 20 15:15:08 2014 -0800 |
||
84 | 1 | Loïc Dachary | |
85 | PGLog: include rollback_info_trimmed_to in (read|write)_log |
||
86 | |||
87 | Fixes: #10157 |
||
88 | Backport: firefly, giant |
||
89 | Signed-off-by: Samuel Just <sjust@redhat.com> |
||
90 | (cherry picked from commit 1fe8b846641486cc294fe7e1d2450132c38d2dba) |
||
91 | 11 | Loïc Dachary | |
92 | Conflicts: |
||
93 | src/osd/PGLog.cc |
||
94 | 18 | Loïc Dachary | in the context coll_t::META_COLL was replaced with META_COLL |
95 | 25 | Nathan Cutler | </pre> |
96 | * The difference between the original commit and the one including a conflict resolution can be displayed with |
||
97 | 1 | Loïc Dachary | <pre> |
98 | 28 | Nathan Cutler | commit=c7d0d51cb574594de6f09457c960347b11fc2474 ; picked_from=$(git show --no-patch --pretty=%b $commit | perl -ne 'print if(s/.*cherry picked from commit (\w+).*/$1/)') ; diff -u --ignore-matching-lines '^[^+-]' <(git show $picked_from) <(git show $commit) |
99 | </pre> |
||
100 | 10 | Loïc Dachary | * if the conflict is difficult to resolve, seek help from the author of the original commit (see "librbd: deadlock in image refresh":https://github.com/ceph/ceph/pull/4176 for instance) and assign the issue to her/him, or to a [[HOWTO#Leads|lead]] |
101 | |||
102 | 17 | Nathan Cutler | Here are the snippets above grouped together for easier copy/paste: |
103 | 10 | Loïc Dachary | <pre> |
104 | 33 | Nathan Cutler | set -x |
105 | 10 | Loïc Dachary | redmine_key=b586c588d4333 |
106 | 30 | Loïc Dachary | redmine_user_id=789 # as found in the URL near "Logged in as " top right of each page |
107 | 10 | Loïc Dachary | github_token=bc275830c635 |
108 | 12 | Loïc Dachary | github_user=dachary |
109 | 10 | Loïc Dachary | issue=$1 |
110 | # can't seem to extract the release number with the api |
||
111 | release=giant ; release_number=2 |
||
112 | 1 | Loïc Dachary | release=firefly ; release_number=3 |
113 | release=hammer ; release_number=5 |
||
114 | release=infernalis ; release_number=7 |
||
115 | 38 | Loïc Dachary | release=jewel ; release_number=8 |
116 | #release_branch=$release-next |
||
117 | release_branch=$release |
||
118 | 33 | Nathan Cutler | if [ $(curl --silent http://tracker.ceph.com/issues/$issue.json | jq -r .issue.tracker.name) != "Backport" ] |
119 | then |
||
120 | echo "not a backport issue" |
||
121 | exit 1 |
||
122 | 1 | Loïc Dachary | fi |
123 | 38 | Loïc Dachary | git checkout -f -b wip-$issue-$release ceph/$release_branch |
124 | 10 | Loïc Dachary | git cherry-pick -x ..... |
125 | 1 | Loïc Dachary | git push loic wip-$issue-$release ; sleep 2 # let github catch up |
126 | title=$(curl --silent 'http://tracker.ceph.com/issues/'$issue.json?key=$redmine_key | jq .issue.subject | tr -d '\\"') |
||
127 | 17 | Nathan Cutler | echo "Issue title: $title" |
128 | 38 | Loïc Dachary | number=$(curl --silent --data-binary '{"title":"'"$title"'","head":"'$github_user':wip-'$issue-$release'","base":"'$release_branch'","body":"http://tracker.ceph.com/issues/'$issue'"}' 'https://api.github.com/repos/ceph/ceph/pulls?access_token='$github_token | jq .number) |
129 | 1 | Loïc Dachary | echo "Opened pull request $number" |
130 | component=core ; curl --silent --data-binary '{"milestone":"'$release_number'","assignee":"'$github_user'","labels":["bug fix","'$component'"]}' 'https://api.github.com/repos/ceph/ceph/issues/'$number'?access_token='$github_token |
||
131 | 12 | Loïc Dachary | firefox https://github.com/ceph/ceph/pull/$number |
132 | 1 | Loïc Dachary | redmine_status=2 # In Progress |
133 | 30 | Loïc Dachary | curl --verbose -X PUT --header 'Content-type: application/json' --data-binary '{"issue":{"description":"https://github.com/ceph/ceph/pull/'$number'","status_id":'$redmine_status',"assigned_to_id":'$redmine_user_id'}}' 'http://tracker.ceph.com/issues/'$issue.json?key=$redmine_key |
134 | 10 | Loïc Dachary | firefox http://tracker.ceph.com/issues/$issue |
135 | </pre> |
||
136 | 26 | Nathan Cutler | |
137 | 29 | Loïc Dachary | If you would like to automate update of the tracker target version, try something like this instead of the last <code>curl</code> command: |
138 | 26 | Nathan Cutler | <pre> |
139 | curl --verbose -X PUT --header 'Content-type: application/json' --data-binary '{"issue":{"description":"https://github.com/ceph/ceph/pull/'$number'","status_id":'$redmine_status',"fixed_version_id":'$redmine_fixed_version'}}' 'http://tracker.ceph.com/issues/'$issue.json?key=$redmine_key |
||
140 | </pre> |
||
141 | 27 | Nathan Cutler | The values of <code>$redmine_status</code> and <code>$redmine_fixed_version</code> can be determined by setting the desired values in an issue and then peeking at the JSON using a command like this: |
142 | <pre> |
||
143 | curl --silent 'http://tracker.ceph.com/issues/12395.json' |
||
144 | </pre> |