<html><body>
<p><font size="2" face="sans-serif">Thanks Sumit. </font><br>
<br>
<font size="2" face="sans-serif">I did not particularly care about the vote on my patch as the patch has a -2 vote already (until we have our own 3rd party testing set up). I got worried that other patches will receive the -1 and people all over will get blocked. So that's why I got alarmed.</font><br>
<br>
<font size="2" face="sans-serif">There is one other point I would like to make: A 3rd party test shouldn't be triggered on patches to a different plugin. Does that make sense? </font><br>
<br>
<font size="2" face="sans-serif">Here is an excerpt From [1] under Testing Requirements:</font><br>
<br>
<font size="4" color="#2F2F2F" face="ArialUnicodeMS">The Neutron team expects that the third party testing will provide a +/-1 verify vote for all changes to a plugin or driver’s code. In addition, the Neutron team expects that the third party test will also vote on all code submissions by the jenkins user.</font>
<p><font size="2" face="sans-serif">The above should be perhaps modified to indicate the no -1 vote policy but beyond that it says that essentially the 3rd party test needs to be done on the corresponding plugin (and the Jenkins submissions). </font>
<p><font size="2" face="sans-serif">Mohammad</font>
<p><font size="2" face="sans-serif">[1] </font><font size="2" face="sans-serif"><a href="https://wiki.openstack.org/wiki/Neutron_Plugins_and_Drivers">https://wiki.openstack.org/wiki/Neutron_Plugins_and_Drivers</a></font><br>
<br>
<img width="16" height="16" src="cid:1__=0ABBF6F0DFE9D6238f9e8a93df938@us.ibm.com" border="0" alt="Inactive hide details for Sumit Naiksatam ---01/17/2014 05:07:53 PM---Just wanted to confirm that we have turned off -1 on fail"><font size="2" color="#424282" face="sans-serif">Sumit Naiksatam ---01/17/2014 05:07:53 PM---Just wanted to confirm that we have turned off -1 on failed builds. We have also triggered rechecks</font><br>
<br>
<font size="1" color="#5F5F5F" face="sans-serif">From: </font><font size="1" face="sans-serif">Sumit Naiksatam <sumitnaiksatam@gmail.com></font><br>
<font size="1" color="#5F5F5F" face="sans-serif">To: </font><font size="1" face="sans-serif">Anita Kuno <anteaya@anteaya.info>, </font><br>
<font size="1" color="#5F5F5F" face="sans-serif">Cc: </font><font size="1" face="sans-serif">Kevin Benton <kevin.benton@bigswitch.com>, Sumit Naiksatam <sumit.naiksatam@bigswitch.com>, openstack-infra@lists.openstack.org, openstack-ci@bigswitch.com</font><br>
<font size="1" color="#5F5F5F" face="sans-serif">Date: </font><font size="1" face="sans-serif">01/17/2014 05:07 PM</font><br>
<font size="1" color="#5F5F5F" face="sans-serif">Subject: </font><font size="1" face="sans-serif">Re: [OpenStack-Infra] Please turn off verification voting for Big Switch CI</font><br>
<hr width="100%" size="2" align="left" noshade style="color:#8091A5; "><br>
<br>
<br>
<tt><font size="2">Just wanted to confirm that we have turned off -1 on failed builds. We<br>
have also triggered rechecks on any failed builds. We have also fixed<br>
the issue with permissions to the tempest log.<br>
<br>
To confirm, the failures were not random.<br>
<br>
Thanks,<br>
~Sumit.<br>
<br>
On Fri, Jan 17, 2014 at 1:15 PM, Anita Kuno <anteaya@anteaya.info> wrote:<br>
> Thank you for your prompt response.<br>
><br>
> Please confirm once -1 voting has been turned off for your system.<br>
><br>
> Thank you,<br>
> Anita.<br>
><br>
> On 01/17/2014 04:11 PM, Sumit Naiksatam wrote:<br>
>> Thanks for the heads up. We will turn off the negative vote. Just a<br>
>> clarification, the current negative votes are not incorrect, we are<br>
>> hitting existing race conditions for which bugs exist. This should have<br>
>> been clear from the tempest test run log, however this file was not getting<br>
>> copied over correctly and hence was not visible in the logs that we have<br>
>> made publicly available.<br>
>> On Jan 17, 2014 1:03 PM, "Anita Kuno" <anteaya@anteaya.info> wrote:<br>
>><br>
>>> Please turn off verification voting for Big Switch CI.<br>
>>><br>
>>> It is voting -1 on patches which is raising questions from developers.<br>
>>> [0][1]<br>
>>><br>
>>> We are moving in the direction of requiring 3rd party testing systems to<br>
>>> only vote +1 when tests pass and removing -1 voting entirely.<br>
>>><br>
>>> You do yourself a big favour if you make this change voluntarily.<br>
>>><br>
>>> We need a response from you that -1 voting is turned off for your system<br>
>>> within 4 hours from the time this email is sent. If we don't get your<br>
>>> response that you have turned off -1 voting for your system, we reserve<br>
>>> the right to turn off verification voting for your system within gerrit.<br>
>>><br>
>>> Please respond promptly.<br>
>>><br>
>>> Thank you,<br>
>>> Anita.<br>
>>><br>
>>> [0] </font></tt><tt><font size="2"><a href="https://review.openstack.org/#/c/63558/">https://review.openstack.org/#/c/63558/</a></font></tt><tt><font size="2"><br>
>>> [1] </font></tt><tt><font size="2"><a href="https://review.openstack.org/#/c/66453/">https://review.openstack.org/#/c/66453/</a></font></tt><tt><font size="2"><br>
>>><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> OpenStack-Infra mailing list<br>
> OpenStack-Infra@lists.openstack.org<br>
> </font></tt><tt><font size="2"><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra</a></font></tt><tt><font size="2"><br>
<br>
_______________________________________________<br>
OpenStack-Infra mailing list<br>
OpenStack-Infra@lists.openstack.org<br>
</font></tt><tt><font size="2"><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra</a></font></tt><tt><font size="2"><br>
<br>
</font></tt><br>
</body></html>