Configure Gerrit for Patch Review

Registered by Zach Pfeffer on 2011-07-05

Configure Gerrit for patch review.

We just need to make sure a patch can be reviewed in Gerrit. Auto-merging and auto-validation will be delivered as another step.

Gerrit will be setup on a server and a Git (most likely the manifest gits) will be selected to be managed.

Blueprint information

Status:
Complete
Approver:
Zach Pfeffer
Priority:
Essential
Drafter:
Paul Sokolovsky
Direction:
Approved
Assignee:
Paul Sokolovsky
Definition:
Approved
Series goal:
Accepted for 2011q2
Implementation:
Implemented
Milestone target:
milestone icon 11.07
Started by
James Westby on 2011-07-21
Completed by
Zach Pfeffer on 2011-07-28

Related branches

Sprints

Whiteboard

[pfalcon 2011-07-28] This was implemented, just waiting for someone else to try to post review to Gerrit and approve. Changes which were posted to Gerrit, were reviewed, and then merged: http://android.git.linaro.org/#q,status:merged,n,z . More changes: http://android.git.linaro.org/#q,status:open,n,z . Postponed items were overflowed to https://blueprints.launchpad.net/linaro-android/+spec/linaro-android-switch-to-gerrit . (Waiting on RT ticket and more specs from Android team).

Headline:
Gerrit, a change review tool, which is essential part of Android development workflow and best practices, has been set up to track Linaro Android code changes.

Acceptance:
Gerrit is tested for change review for 11.07, ready to be used as code review and hosting system for 11.08 milestone. This BP will be accepted when one change is pushed, reviewed and merged.

Docs are at:
https://wiki.linaro.org/Platform/Android/Gerrit (official howto)
https://wiki.linaro.org/PaulSokolovsky/GerritNotes (detailed implementation notes)

(?)

Work Items

Work items:
Brainstorm Gerrit (how it works, how it can be made work for our setup): DONE
Get admin access to http://android.git.linaro.org/: DONE
Import few initial projects to http://android.git.linaro.org/ and create test change reviews: DONE
Import all android/* components from git.linaro.org: DONE
Set up android.git.linaro.org as default review server in manifests: DONE
Decide if Gerrit should be master git hosting service or just review service: DONE
Split linaro remotes in current manifests to get rid of android/ prefix and make names match upstream: DONE
Rename platform/manifests.git to platform/manifest.git to match upstream: DONE
Mirror rest of AOSP and 3rd-party components to android.git.linaro.org: DONE
Set up anon git access for android.git.linaro.org: POSTPONED
Set up gitweb for android.git.linaro.org: POSTPONED
Disable write access for android components on git.linaro.org: POSTPONED
Documentation: DONE
Elaborate Linaro Gerrit workflow and processes: POSTPONED
Work through docs with pfefferz to make sure thigns work: DONE

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.