Open Stack

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 <sumitnaiksatam at gmail.com>
To:	Anita Kuno <anteaya at anteaya.info>,
Cc:	Kevin Benton <kevin.benton at bigswitch.com>, Sumit Naiksatam
            <sumit.naiksatam at bigswitch.com>,
            openstack-infra at lists.openstack.org, openstack-ci at 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 <anteaya at 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" <anteaya at 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 at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra

_______________________________________________
OpenStack-Infra mailing list
OpenStack-Infra at lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-infra/attachments/20140117/6a038ae9/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: graycol.gif
Type: image/gif
Size: 105 bytes
Desc: not available
URL: <http://lists.openstack.org/pipermail/openstack-infra/attachments/20140117/6a038ae9/attachment-0001.gif>