Chromium is a main-first development team; generally, all code should land on main then roll out to stable users only after the milestone containing the code is branched, stabilized and shipped to the stable channel (to learn more about the release cycle, click here). This is because merging (also known as cherry-picking) code to an older release branch introduces risk and costs time across the team. However, there are times when the benefits outweigh the costs and a merge might be appropriate, e.g. to fix a web platform regression, address a crash or patch a security vulnerability.
To ensure we make the right decisions, release managers leverage a merge review process to evaluate each request. They‘ll ask questions about the reason you would like to merge a change and the risk of the merge itself, and you’ll work together to make a judgement call on whether or not the merge should be approved or rejected.
Generally, merges follow these high-level steps:
For details on each step, see below.
NOTE: Because security issues (identified with Type=Bug-Security) follow a more complex flow, you may simply mark security issues as Fixed in Monorail and automation will handle the remainder of the merge request process flow for you; simply process the merge if it is requested and approved.
Before requesting a merge, first ensure your change is a good merge candidate:
Ensure it meets the merge criteria (via Chromium Dash) of the branch(es) you'd like to merge to; merge criteria become more strict the older the branch is, more details on criteria below
Verify merging the change to an older branch would be safe, e.g. unlikely to introduce new regressions, no major merge conflicts, automated test coverage present, etc; chat with your TL for input if you're not sure
Confirm your change fixes the issue at hand, preferably by testing on and monitoring the canary channel for 24 hours post-release (see Chromium Dash to determine if your change has shipped)
Next, ensure you have a crbug/ (generally the bug being fixed by the merge) with the following information present and accurate:
Title and description clearly describing the bug being fixed
Priority (Pri-#), OS (OS-OS) and target milestone(s) (Target-##)
Owner, generally the person requesting / performing the merge
Release block label if applicable (ReleaseBlock=Channel)
Issue status:
Once you‘ve verified all the above, you’re ready to request a merge! Simply add the label Merge-Request-## to the issue (where ## indicates the milestone you'd like to merge to), and use multiple labels for multiple milestones, e.g. Merge-Request-91 Merge-Request-92 for M91 and M92. Please also copy the following questions and answer them in a comment on the issue:
After you've applied the Merge-Request-## label, automation will evaluate your request and may either approve it, reject it, or pass it along to a release manager for manual evaluation; see here to learn more about this automation. If manual review is required, release managers strive to answer all merge requests within two business days, but extenuating circumstances may cause delays.
At this point, following along via bug comments sent by email will always keep you in the loop, but you can also use the following saved project queries in Monorail (dropdown to the left of the search bar) to track your merges:
For a description of each label used to track the merge process, see the appendix below.
Once your merge has been approved for a given milestone (via the release manager or automation applying the Merge-Approved-## label), you have two options to land the merge:
Regardless of which method you choose, please ensure you land your cherry-pick ASAP so that it can be included in the next release built from the branch; if you don't merge your cherry-pick soon after approval, it will eventually be rejected for merge.
Select the “More” button in the Gerrit UI, then choose “Cherry Pick”. When prompted for a branch, enter refs/branch-heads/####, where #### corresponds to the release branch you are merging to (available on Chromium Dash).
Once the cherry-pick CL is prepared, you can have it approved and landed by adding Rubber Stamper (rubber-stamper@appspot.gserviceaccount.com) as a reviewer and setting Auto-Submit+1;the Rubber Stamper bot will approve and submit the CL to CQ on your behalf.
Note: the Rubber Stamper does not provide OWNERS approval, and only works within 7 days of the original change; Googlers can learn more here.
The commands below should set up your environment to be able to successfully upload a cherry-pick to a release branch, where #### corresponds to the release branch you are merging to (available on Chromium Dash):
$ gclient sync --with_branch_heads $ git fetch $ git checkout -b BRANCH_NAME refs/remotes/branch-heads/#### $ git cl upstream branch-heads/#### $ git cherry-pick -x COMMIT_HASH_MAIN
From here, your environment should be ready to adjust the change as required; use ninja to build and test your changes, and when ready upload for review:
$ git cl upload
Adjust the change description to omit the “Change-Id: ...” line from original patch, otherwise you may experience issues when uploading the change to Gerrit. Once complete, use Gerrit to initiate review and approval of the merge as TBR has been discontinued.
Other tips & tricks when merging with git via release branches:
The release team has built automation via Sheriffbot to assist in several merge flows: security merge triage, general merge request triage, and preventing missed merges.
Given the additional complexity inherent in security merges, the security team has built custom automation to handle this flow end to end; simply mark any security issue as Fixed and Sheriffbot will evaluate applicable milestones, determine if merges are required and automatically request them if need be.
To reduce release manager toil, Sheriffbot performs the first pass review of all merge requests; it may auto-approve the issue if it can detect the issue meets the right criteria for the current merge phase (e.g. a ReleaseBlock-Dev issue requesting a merge before beta promotion), and it may auto-reject the issue similarly (e.g. a Pri-3 issue requesting a merge post-stable). If it cannot decide, it will pass the issue to a release manager for manual review.
Generally, Sheriffbot takes action on merge requests only after one of the two conditions below are met:
These conditions help ensure any relevant changelists have had sufficient runtime in our canary channel and thus are low risk for introducing a new regression onto our release branch.
To avoid the situation where a critical issue is present on a release branch but the fix isn‘t merged, Sheriffbot evaluates all release-blocking issues targeting a milestone that has already branched and adds a Merge-TBD-## label if the issue was marked as fixed after branch day but hasn’t been merged. When this occurs, developers should evaluate the issue and either request a merge if required (e.g. the fix did miss the release branch point) by adding the Merge-Request-## label, or add the Merge-NA-## label if not (e.g. the fix is present in the release branch already or the merge is unnecessary for other reasons).
The table below describes the different phases that each milestone progresses through during its release cycle; this data is available via the Chromium Dash front-end and API.
Branch Phase | Period Begins | Period Ends | Acceptable Merges Include Fixes For: |
---|---|---|---|
branch | M(X) Branch | M(X) Beta | Polish issues for Finch-gated features (no workflow changes), any new regressions, any release blockers, any security issues, any string issues (.GRD changes) |
beta | M(X) Beta | M(X) Stable Cut | Non-functional issues for Finch-gated features (e.g. add metrics, fix crash), noticeable new regressions, any release blockers, any security issues, urgent string issues (.GRD changes) |
stable_cut | M(X) Stable Cut | M(X) Stable | Urgent new regressions, all release blockers, important security issues (medium severity or higher), emergency string issues (.GRD changes) |
stable | M(X) Stable | M(X+1) Stable | Urgent new regressions (especially user reports), urgent release blockers, important security issues (medium severity or higher) |
extended (if applicable) | M(X+1) Stable | M(X+2) Stable | Important security issues (medium severity or higher) applicable to Windows, Mac or Chrome OS |
The table below describes the different merge states and labels used to track them. All labels follow the form Merge-[State]-##, where ## corresponds to the applicable milestone. If multiple merges are required, these labels may appear multiple times on the same bug in different states (e.g. a merge request could have both Merge-Approved-92 and Merge-Rejected-91 at the same time).
Label / State | Step Owner | Next Steps |
---|---|---|
Request | Release manager | Automation will review and either approve / reject directly, or pass the review to a release manager for manual evaluation |
Review | Release manager | Release manager will evaluate and either approve, reject, or request additional information within two business days |
Approved | Issue owner | Issue owner should cherry-pick the fix to the appropriate release branch ASAP |
Merged | None | N/A; merge has already been landed, no further work required for given milestone |
Rejected | Issue owner | Issue owner should re-add Merge-Request-## to escalate if they feel the merge was erroneously rejected and should be re-evaluated |
TBD | Issue owner | Issue owner should evaluate if a merge is required, then remove Merge-TBD-## and replace it with Merge-NA-## (if no merge needed) or Merge-Request-## (if merge needed) |
NA | None | N/A; merge is not required to the relevant milestone, no further work required for given milestone |