Open Stack

Super awesome!

I also noticed the new version of git-review is available now which
will not rebase patches automatically if they don't need a rebase.

Vish

On Sep 6, 2012, at 12:01 PM, Jeremy Stanley <fungi at yuggoth.org> wrote:

> The CI Team will be merging a long-requested[1]_ feature to Gerrit
> on review.openstack.org at 16:00 UTC this coming Monday, 2012-09-10.
> 
> Automatic Detection of Trivially-Rebased Commits
> ================================================
> 
> Any committed patchset with no substantive changes (where the git
> patch-id[2]_ and commit message remain the same) will have code
> review flags automatically[3]_ reapplied from its predecessor
> patchset on that Change-Id. Patchsets where the commit message
> differs but the patch-id still remains the same will *not* have code
> reviews reapplied, but will have a comment automatically added from
> the Trivial Rebase user noting that the code itself is unchanged.
> 
> It's hoped that these changes will improve workflow for code
> reviews, allowing reviewers to avoid re-reading identical patchsets
> looking for nonexistent alterations. The script itself is provided
> in upstream Gerrit's contrib[4]_ directory and has been put through
> its paces with our OpenStack-specific modifications[5]_ on a couple
> of non-production Gerrit servers already (including review-dev.o.o
> where it is currently installed and active[6]_), so the CI Team
> doesn't anticipate any disruption. Follow up to the list with any
> questions and I'll be happy to answer!
> 
> References...
> 
> .. [1] https://bugs.launchpad.net/openstack-ci/+bug/881184
> 
> .. [2] http://www.kernel.org/pub/software/scm/git/docs/git-patch-id.html
> 
> .. [3] Automatically but not instantly--there is a delay of a few
>   seconds before the review flags reappear in the WebUI.
> 
> .. [4] https://gerrit.googlesource.com/gerrit/+/master/contrib
> 
> .. [5] https://review.openstack.org/12373
> 
> .. [6] https://review-dev.openstack.org/488
> 
> -- 
> Jeremy Stanley on behalf of the OpenStack CI Team
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev