<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 16, 2014 at 6:12 PM, Russell Bryant <span dir="ltr"><<a href="mailto:russell@russellbryant.net" target="_blank">russell@russellbryant.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Tue, Sep 16, 2014 at 6:01 PM, Russell Bryant <span dir="ltr"><<a href="mailto:russell@russellbryant.net" target="_blank">russell@russellbryant.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>From a high level, all patches go to a code review system.  *Every* patch must be peer reviewed (usually by 2 people, but that's a policy decision).  *Every* patch must also pass tests.  Once a patch passes both tests and peer review, it is automatically merged into the repository.</div></div></div></div></blockquote><div><br></div></span><div>I just thought of another important bit of the workflow ... the CLA handling.</div><div><br></div><div>With Asterisk today, all patches go through the issue tracker.  The issue tracker handles the CLA.  Uploading code to the issue tracker bypasses that, so we had to hack reviewboard to also know about CLAs.  OpenStack uses a CLA, as well, and gerrit has built-in CLA handling.</div></div></div></div></blockquote><div><br></div><div>Some more workflow comments, sorry... and then maybe I'll shut up.  :-)</div><div><br></div><div>One thing I really like about gerrit vs review board is that gerrit is focused on git and as a result, has more native git integration.  Posting code reviews is just "git review" from your git tree.  "git review" is really just a helper around a normal "git push".  You can push a patch series to gerrit and gerrit understands what that is and tracks the patch dependencies.  Last I checked, review board still lacked any sort of support for a series of patches related to each other. </div><div><br></div><div>Also, if you're really attached to doing code reviews in a console and maybe even offline, someone in the OpenStack community made gertty [1], which is a replacement for using the web UI.  It's gerrit, but entirely synced locally and in a terminal.  I've used it for several hours while offline on an airplane and it's pretty darn amazing.  It syncs all the reviews you did back to gerrit once you're back online.</div><div><br></div><div>[1] <a href="http://lists.openstack.org/pipermail/openstack-dev/2014-September/045013.html">http://lists.openstack.org/pipermail/openstack-dev/2014-September/045013.html</a></div></div></div></div>