Date: prev next · Thread: first prev next last
2017 Archives by date, by thread · List index


On Fri, Dec 1, 2017 at 1:32 AM, David Ostrovsky <d.ostrovsky@gmx.de> wrote:
On Thu Nov 30 08:25:40 UTC 2017, Stephan Bergmann wrote:

Does "(with a help of maintainers)" imply that there are change
owners who could not override a VRFY-1 from Jenkins themselves?

Yes. I believe that "registered users" group members will not be
able to remove VRFY-1 vote.

I do not know of anyway, anyone can remove it using the ui, short of
impersonating jenkins' user.

the only available way to remove the -1 is to rebase the patch, since
hte -1 is bound to the particular patcheset version...
then hope that jenkins will not put another -1, or at least quickly
+2/+1 the rebased patch and submit... neither are desirable.

hence the choice of +1/0 rather than +1/-1 for jenkins.

For all practical purpose: 0 means jenkins has not yet try or was not
successful.
to submit one would have to manually add a verify +1. to do that you
need to go the the change view, and jenkins activity or lack thereof
is pretty apparent there.

I suspect the intent is to quicky avoid changeset that failed
jenkins... but then whether the change set has failed or has not yet
been built, what is the difference wrt to avoiding going in the
details of the change ?
from a review perspective no verification or failed verification means
the same thing: not ready

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.