Thanks Sumit. 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.
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? Here is an excerpt From [1] under Testing Requirements: 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. 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). Mohammad [1] https://wiki.openstack.org/wiki/Neutron_Plugins_and_Drivers From: Sumit Naiksatam <sumitnaiksa...@gmail.com> To: Anita Kuno <ante...@anteaya.info>, Cc: Kevin Benton <kevin.ben...@bigswitch.com>, Sumit Naiksatam <sumit.naiksa...@bigswitch.com>, openstack-infra@lists.openstack.org, openstack...@bigswitch.com Date: 01/17/2014 05:07 PM Subject: Re: [OpenStack-Infra] Please turn off verification voting for Big Switch CI Just wanted to confirm that we have turned off -1 on failed builds. We have also triggered rechecks on any failed builds. We have also fixed the issue with permissions to the tempest log. To confirm, the failures were not random. Thanks, ~Sumit. On Fri, Jan 17, 2014 at 1:15 PM, Anita Kuno <ante...@anteaya.info> wrote: > Thank you for your prompt response. > > Please confirm once -1 voting has been turned off for your system. > > Thank you, > Anita. > > On 01/17/2014 04:11 PM, Sumit Naiksatam wrote: >> Thanks for the heads up. We will turn off the negative vote. Just a >> clarification, the current negative votes are not incorrect, we are >> hitting existing race conditions for which bugs exist. This should have >> been clear from the tempest test run log, however this file was not getting >> copied over correctly and hence was not visible in the logs that we have >> made publicly available. >> On Jan 17, 2014 1:03 PM, "Anita Kuno" <ante...@anteaya.info> wrote: >> >>> Please turn off verification voting for Big Switch CI. >>> >>> It is voting -1 on patches which is raising questions from developers. >>> [0][1] >>> >>> We are moving in the direction of requiring 3rd party testing systems to >>> only vote +1 when tests pass and removing -1 voting entirely. >>> >>> You do yourself a big favour if you make this change voluntarily. >>> >>> We need a response from you that -1 voting is turned off for your system >>> within 4 hours from the time this email is sent. If we don't get your >>> response that you have turned off -1 voting for your system, we reserve >>> the right to turn off verification voting for your system within gerrit. >>> >>> Please respond promptly. >>> >>> Thank you, >>> Anita. >>> >>> [0] https://review.openstack.org/#/c/63558/ >>> [1] https://review.openstack.org/#/c/66453/ >>> >> > > > _______________________________________________ > OpenStack-Infra mailing list > OpenStack-Infra@lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra _______________________________________________ OpenStack-Infra mailing list OpenStack-Infra@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra
<<inline: graycol.gif>>
_______________________________________________ OpenStack-Infra mailing list OpenStack-Infra@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra
