<div dir="ltr"><div>Hi,</div><div><br></div>I might be posting a question to a wrong thread, but what would be the 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?<div>
I wouldn't like such patches to affect gerrit stream or trigger CIs, but gerrit could still be used for regular reviewing process.<br><div><br></div><div>Thanks,</div><div>Eugene.</div></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Sat, May 31, 2014 at 12:51 AM, Sergey Lukjanov <span dir="ltr"><<a href="mailto:slukjanov@mirantis.com" target="_blank">slukjanov@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Yay!<br>
<br>
No more weird CR chains.<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, May 30, 2014 at 9:32 PM, Clark Boylan <<a href="mailto:clark.boylan@gmail.com">clark.boylan@gmail.com</a>> wrote:<br>
> On Wed, May 21, 2014 at 4:24 PM, Clark Boylan <<a href="mailto:clark.boylan@gmail.com">clark.boylan@gmail.com</a>> wrote:<br>
>> Hello everyone,<br>
>><br>
>> Gerrit has long supported "Draft" patchsets, and the infra team has 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 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<br>
>> Draft patchsets will remain in a Draft state until it is published.<br>
>><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<br>
>>   for user expectations and may expose data that shouldn't be exposed.<br>
>> * Draft patchsets pushed after published patchsets confuse reviewers as<br>
>>   they cannot vote with a value because the latest patchset is hidden.<br>
>> * Draft patchsets confuse the Gerrit event stream output making it<br>
>>   difficult for automated tooling to do the correct thing with Drafts.<br>
>> * 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>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Sincerely yours,<br>
Sergey Lukjanov<br>
Sahara Technical Lead<br>
(OpenStack Data Processing)<br>
Principal Software Engineer<br>
Mirantis Inc.<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>