← Back to team overview

openstack team mailing list archive

Re: New Gerrit version (and server)

 

Thanks James and team for the work here, this will be very useful for
documentation drafts.

Anne

On Fri, Apr 13, 2012 at 8:55 AM, Sandy Walsh <sandy.walsh@xxxxxxxxxxxxx>wrote:

> Sounds awesome! Looking forward to Draft Changes, much needed.
>
> -S
>
>
> ________________________________________
> From: openstack-bounces+sandy.walsh=rackspace.com@xxxxxxxxxxxxxxxxxxx[openstack-bounces+sandy.walsh=
> rackspace.com@xxxxxxxxxxxxxxxxxxx] on behalf of James E. Blair [
> corvus@xxxxxxxxxxxx]
> Sent: Thursday, April 12, 2012 8:23 PM
> To: OpenStack Mailing List
> Subject: [Openstack] New Gerrit version (and server)
>
> Hi,
>
> We've just upgraded Gerrit to version 2.3.  There are a lot of changes
> behind the scenes that we've been looking forward to (like being able to
> store data in innodb rather than myisam tables for extra data
> longevity).  And there are a few visible changes that may be of interest
> to OpenStack developers.
>
> One new addition in 2.3 is "draft changes".  The idea behind a draft
> change in Gerrit is that it is a change that is not ready for merging,
> or even general code review, but you would like to share it with some
> people to get early comments.  If you upload a change as a draft, by
> default, no one else can see it.  You must explicitly add each person
> you would like to share it with as a reviewer.  Reviewers you add can
> leave comments, but can not vote at this stage.  You can continue to
> upload new patchsets to the change as it evolves, and once it is ready
> for general review, you can click the "Publish" button.  It will then
> become a normal change in Gerrit that everyone can see, including the
> earlier reviews from the draft stage.  This is a one way transition;
> once a draft is published, it can't be made a draft again.
>
> If you're using git-review from source or the latest version from PyPI
> (version 1.16, released today), you can easily upload a draft change by
> adding the "-D" option (eg, "git review -D").  Earlier versions of
> git-review also have the "-D" option, but the git ref that Gerrit uses
> to indicate a change should be a draft was changed between the 2.3
> release candidate and the final release; so if using "-D" results in an
> error, you may need to upgrade.
>
> You may notice some changes to the diff view.  Notably, the header which
> contained all of the possible viewing options has been split up into
> several parts; you can switch between them by selecting options that
> show up under the menu at the top.  I recommend setting "Retain Header
> On File Switch" under the Preferences section, as it is a nicer
> experience when changing files.
>
> Another notable new feature is the ability to add a group to the list of
> reviewers for a change.  Just type in the name of the group and click
> "Add Reviewer" and all of the individuals in the group will be added to
> the list of reviewers (and will see the change on their "review
> requests" list.
>
> Finally, we've modified some of our local OpenStack style changes so
> that it is easier for us to track upstream changes in layout.  It should
> mean a little more consistency throughout the interface, though we
> weren't able to keep the alternating row colors on the main table
> without a disproportionate amount of effort.  Do note that you can click
> on a line in a table, and it will be highlighted to improve legibility.
>
> We've tried to give this as much testing as possible before moving it
> into production.  If you encounter any issues, please let us know on IRC
> (mtaylor, jeblair, LinuxJedi), via email at
> <openstack-ci-admins@xxxxxxxxxxxxxxxxxxx>, or you can file a bug at:
>
>  https://bugs.launchpad.net/openstack-ci/
>
> Thanks,
>
> Jim
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openstack
> Post to     : openstack@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openstack
> More help   : https://help.launchpad.net/ListHelp
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openstack
> Post to     : openstack@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openstack
> More help   : https://help.launchpad.net/ListHelp
>

References