<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">I’ve had this question asked numerous times (by previous coworkers, people interested in contributing to OpenStack, etc). The general feeling has always been that the individual is concerned about 3 things when considering drafts in gerrit.</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">1. Patch is very much WIP and doesn’t need to be reviewed yet or is meant to be a collaboration between a couple individuals first</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">2. Since a patch is very much WIP, there is concern about consuming CI resources with needless testing.</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">3. The code is “example”, “toy”, or “exploratory” (not planning to submit to the project, but not private/proprietary)</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">The general advice I give to people is to post the patches (especially at checkpoints, e.g. taking a break for the night) and ensure that they are running the tests that they can locally. I also explain the WIP process for a patch. Usually the combination is good enough to convince them that a “Draft” isn’t really needed. If there is still concern about posting the patch to gerrit prematurely (option 3 above), I recommend using another system to collaborate on the initial patch such as what I use my GitHub account for (out-of-tree development / examples / playing with code that won’t ever be submitted to the main repositories).</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">I, for one am very pleased that Drafts are now disabled. I never liked the feature (it felt like it was missing a chunk of functionality to be really useful).</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Cheers,</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Morgan</div> <div id="bloop_sign_1401567678743205120" class="bloop_sign"><p><strong>—</strong><br><strong>Morgan Fainberg</strong></p></div> <div style="color:black"><br>From: <span style="color:black">Clark Boylan</span> <a href="mailto:clark.boylan@gmail.com">clark.boylan@gmail.com</a><br>Reply: <span style="color:black">OpenStack Development Mailing List (not for usage questions)</span> <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>Date: <span style="color:black">May 31, 2014 at 12:33:33</span><br>To: <span style="color:black">OpenStack Development Mailing List (not for usage questions)</span> <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>Subject: <span style="color:black"> Re: [openstack-dev] [All] Disabling Pushes of new Gerrit Draft Patchsets <br></span></div><br> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>There isn't an option to push non public patches (and there really

<br>wasn't before either, drafts are not properly private and this false

<br>expectation is one of the reasons we have disabled them). Currently

<br>the recommended alternative is work in progress. The code cannot merge

<br>with a work in progress vote but it can otherwise be reviewed and

<br>tested. I think we would like to have changes tested even if they are

marked as work in progress because it is useful feedback.

<br>

Is there a specific use case where not running tests is desirable?

<br>

<br>Clark

<br>

<br>On Sat, May 31, 2014 at 6:45 AM, Eugene Nikanorov

<br><enikanorov@mirantis.com> wrote:

<br>> Hi,

<br>>

<br>> I might be posting a question to a wrong thread, but what would be the

<br>> option to push a patch that I would like to share only with certain group of

> people. In other words, is there still an option to push non-public patches?

<br>> I wouldn't like such patches to affect gerrit stream or trigger CIs, but

> gerrit could still be used for regular reviewing process.

<br>>

<br>> Thanks,

<br>> Eugene.

<br>>

<br>>

<br>> On Sat, May 31, 2014 at 12:51 AM, Sergey Lukjanov <slukjanov@mirantis.com>

<br>> wrote:

<br>>>

<br>>> Yay!

<br>>>

>> No more weird CR chains.

<br>>>

<br>>> On Fri, May 30, 2014 at 9:32 PM, Clark Boylan <clark.boylan@gmail.com>

<br>>> wrote:

<br>>> > On Wed, May 21, 2014 at 4:24 PM, Clark Boylan <clark.boylan@gmail.com>

<br>>> > wrote:

<br>>> >> Hello everyone,

<br>>> >>

<br>>> >> Gerrit has long supported "Draft" patchsets, and the infra team has

<br>>> >> long

<br>>> >> recommended against using them as they are a source of bugs and

<br>>> >> confusion (see below for specific details if you are curious). The

<br>>> >> newer

<br>>> >> version of Gerrit that we recently upgraded to allows us to prevent

<br>>> >> people from pushing new Draft patchsets. We will take advantage of this

<br>>> >> and disable pushes of new Drafts on Friday May 30, 2014.

<br>>> >>

<br>>> >> The impact of this change should be small. You can use the Work in

<br>>> >> Progress state instead of Drafts for new patchsets. Any existing

>> >> Draft patchsets will remain in a Draft state until it is published.

<br>>> >>

>> >> Now for the fun details on why drafts are broken.

<br>>> >>

<br>>> >> * 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.

<br>>> >> * Draft patchsets pushed after published patchsets confuse reviewers as

>> >>   they cannot vote with a value because the latest patchset is hidden.

<br>>> >> * 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.

<br>>> >>

<br>>> >> Let us know if you have any questions,

<br>>> >>

<br>>> >> Clark (on behalf of the infra team)

<br>>> >

<br>>> > Heads up everyone, this is now in effect and pushes of new draft

<br>>> > patchsets have been disabled.

<br>>> >

<br>>> > Thanks,

<br>>> > Clark

<br>>> >

<br>>> > _______________________________________________

<br>>> > OpenStack-dev mailing list

<br>>> > OpenStack-dev@lists.openstack.org

<br>>> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

<br>>>

<br>>>

<br>>>

<br>>> --

<br>>> Sincerely yours,

<br>>> Sergey Lukjanov

<br>>> Sahara Technical Lead

<br>>> (OpenStack Data Processing)

<br>>> Principal Software Engineer

<br>>> Mirantis Inc.

<br>>>

<br>>> _______________________________________________

<br>>> OpenStack-dev mailing list

<br>>> OpenStack-dev@lists.openstack.org

<br>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

<br>>

<br>>

<br>>

<br>> _______________________________________________

<br>> OpenStack-dev mailing list

<br>> OpenStack-dev@lists.openstack.org

<br>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

<br>>

<br>

<br>_______________________________________________

<br>OpenStack-dev mailing list

<br>OpenStack-dev@lists.openstack.org

<br>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

<br></div></div></span></blockquote></body></html>