Uploaded image for project: 'Qt Quality Assurance Infrastructure'
  1. Qt Quality Assurance Infrastructure
  2. QTQAINFRA-3382

Automated cherry-picking into stable branches

    XMLWordPrintable

Details

    • User Story
    • Resolution: Done
    • P2: Important
    • None
    • None
    • Gerrit
    • None

    Description

      As per the discussion of Branch Policy at the Qt Contributors Summit 2019, we want to move away from the current flow (new features in dev; certain bug fixes in stable branches; merge-master forward-merges changes from stable to dev; some cherry-picking for LTS branches), towards a flow where all changes go into dev, and are then cherry-picked into relevant stable branches.

      The flow we would like to end up with should be supported by automation:

      • contributors puts all "fresh" changes up for code-review, normally against dev
        • Exceptions are release-specific changes: changes-files or rare hot-fixes, which go directly into the respective release branch
      • commit footer may include a tag indicating which branch(es) the change should be picked into after successful integration
        • Tag name suggestions: Pick-to:, Pick-into:, Backport-to:
        • Tag value: comma-separated list of branch names; most stable branch (in-between branches are inferred by tool)
        • exceptional changes made in release branches MUST include such a commit footer
      • Reviewers/approvers confirm that change is suitable for specified target branches
        • once approved, staging and integration into dev as usual
      • once integrated, gerrit triggers a tool that reads the tag in the commit footer, and generates a cherry-pick commit (with -x) into each branch listed
        • the tool ensures that commits are cherry-picked in the same order in which they appear in the original target branch
        • for each cherry-pick, generate a code review with the same owners, reviewers, and approvals as original commit
        • if there is no conflict, stage the cherry-picked change automatically (integrate into target branch if successful)
        • in case of conflict, or in case of CI failure: alert owners and reviewers

      At this point, the change is either successfully cherry-picked into all branches, or requires manual follow-up (conflict resolution or back-porting of the change) by the owner of the change for some or all branches. The usual process applies, ie amending the change, getting new reviews and approvals, and staging; or abandoning those changes where the backporting effort turns out to be too significant. The owner of the change can freely decide how they resolve issues (each problematic branch separately, or fix once and cherry-pick further from there).

      Attachments

        Issue Links

          There are no Sub-Tasks for this issue.
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              daniel.smith Daniel Smith
              vhilshei Volker Hilsheimer
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes