[openstack-dev] [All] Disabling Pushes of new Gerrit Draft Patchsets
ZZelle
zzelle at gmail.com
Fri May 23 20:56:42 UTC 2014
Hello,
One small remark:
"Work in Progress" can be set only by change owners (defined in
All-Projects ACLs) which could be a limitation when multiple people
contribute to the same change.
Cedric
ZZelle at IRC
On Thu, May 22, 2014 at 1:43 PM, Sergey Lukjanov <slukjanov at mirantis.com>wrote:
> Great, I think it'll make CRs more consistent, especially from the
> reviewers PoV.
>
> On Thu, May 22, 2014 at 3:24 AM, Clark Boylan <clark.boylan at gmail.com>
> wrote:
> > Hello everyone,
> >
> > Gerrit has long supported "Draft" patchsets, and the infra team has long
> > recommended against using them as they are a source of bugs and
> > confusion (see below for specific details if you are curious). The newer
> > version of Gerrit that we recently upgraded to allows us to prevent
> > people from pushing new Draft patchsets. We will take advantage of this
> > and disable pushes of new Drafts on Friday May 30, 2014.
> >
> > The impact of this change should be small. You can use the Work in
> > Progress state instead of Drafts for new patchsets. Any existing
> > Draft patchsets will remain in a Draft state until it is published.
> >
> > Now for the fun details on why drafts are broken.
> >
> > * Drafts appear to be "secure" but they offer no security. This is bad
> > for user expectations and may expose data that shouldn't be exposed.
> > * Draft patchsets pushed after published patchsets confuse reviewers as
> > they cannot vote with a value because the latest patchset is hidden.
> > * Draft patchsets confuse the Gerrit event stream output making it
> > difficult for automated tooling to do the correct thing with Drafts.
> > * Child changes of Drafts will fail to merge without explanation.
> >
> > Let us know if you have any questions,
> >
> > Clark (on behalf of the infra team)
> >
> > _______________________________________________
> > OpenStack-dev mailing list
> > OpenStack-dev at lists.openstack.org
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
> --
> Sincerely yours,
> Sergey Lukjanov
> Sahara Technical Lead
> (OpenStack Data Processing)
> Mirantis Inc.
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140523/7b10d932/attachment.html>
More information about the OpenStack-dev
mailing list