<html><body><div style="font-family: times new roman, new york, times, serif; font-size: 12pt; color: #000000"><div><span data-mce-bogus="true" id="_mce_caret"><span style="font-family: arial,helvetica,sans-serif;">Hi Juan, David,</span></span><br></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Thank you for including us on this thread. From my side, I think what Juan has suggested makes sense, and should work for our needs.<br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Please also add the following names to the group of users that can set this flag:<br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Tahlia Richardson <trichard at redhat.com><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Megan Lewis <melewis at redhat.com><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Byron Gravenorst <bgraveno at redhat.com><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Julie Wu <juwu at redhat.com><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Just let me know if you need any further information, or if there are any issues.<br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Kind Regards,<br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;"><br></span></span></div><div><span data-mce-bogus="true"><span style="font-family: arial,helvetica,sans-serif;">Lucy<br></span></span></div><div><br></div><div><span name="x"></span><div><span style="font-family: arial,helvetica,sans-serif;" data-mce-style="font-family: arial,helvetica,sans-serif;"><br></span></div><div><span style="font-family: arial,helvetica,sans-serif;" data-mce-style="font-family: arial,helvetica,sans-serif;">Lucy Bopf</span></div><div><span style="font-family: arial,helvetica,sans-serif;">Associate Documentation Program Manager<br></span></div><div><span style="font-family: arial,helvetica,sans-serif;" data-mce-style="font-family: arial,helvetica,sans-serif;"><strong>Phone:</strong> +<span><span class="Object" id="OBJ_PREFIX_DWT131_com_zimbra_phone"><a href="callto:61%2007-3514-7124" data-mce-href="callto:61%2007-3514-7124">61 07-3514-7181</a></span></span><br></span></div><div><span style="font-family: arial,helvetica,sans-serif;" data-mce-style="font-family: arial,helvetica,sans-serif;"><span><strong>Email:</strong> <span class="Object" id="OBJ_PREFIX_DWT132_com_zimbra_email"><a href="mailto:lbopf@redhat.com" data-mce-href="mailto:lbopf@redhat.com"></a><a>lbopf@redhat.com</a></span></span></span><br data-mce-bogus="1"></div><div><span style="font-family: arial,helvetica,sans-serif;" data-mce-style="font-family: arial,helvetica,sans-serif;"><span><span class="Object"><br></span></span></span></div><div><span style="font-family: arial,helvetica,sans-serif;" data-mce-style="font-family: arial,helvetica,sans-serif;"><span><span class="Object"><img doc="Briefcase/mail.corp.redhat.com.png" src="cid:a214dd695fc5521c9e514134b7924bf17227ca35@zimbra" data-mce-src="http://mail.corp.redhat.com/home/lbopf@redhat.com/Briefcase/mail.corp.redhat.com.png"><br></span></span></span></div><span name="x"></span><br></div><hr id="zwchr"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><b>From: </b>"Juan Hernández" <jhernand@redhat.com><br><b>To: </b>"David Caro" <dcaro@redhat.com>, "Andrew Dahms" <adahms@redhat.com>, "Lucy Bopf" <lbopf@redhat.com>, "Derek Cadzow" <dcadzow@redhat.com><br><b>Cc: </b>"infra" <infra@ovirt.org>, "Yaniv Dary" <ydary@redhat.com><br><b>Sent: </b>Thursday, June 2, 2016 7:11:35 PM<br><b>Subject: </b>Re: Adding a new flag to a specific gerrit project?<br><div><br></div>Thanks David. I am answering your questions below. Derek, Lucy,<br>Andrew, please review and confirm if you agree.<br><div><br></div>On 06/02/2016 09:35 AM, David Caro wrote:<br>> On 06/01 16:54, Juan Hernández wrote:<br>>> Hello,<br>>> <br>>> As part of the effort to improve the documentation of the REST<br>>> API the documentation team will help us review the patches for<br>>> the specification. I was wondering if it is possible to add a new<br>>> flag, like the CR, CI, and V that we currently have. Would be<br>>> nice to have a "Documentation" flag (or similar) so that the<br>>> documentation team can +1 that flag. Is this possible? Can this<br>>> flag be added to the ovirt-engine-api-model project?<br>> <br>> Yes, it's possible, I'll need more details about it though like: *<br>> max and min values?<br><div><br></div>The min should be -1, and the max +1. The associated descriptions<br>could be the following:<br><div><br></div>-1: Documentation doesn't look good<br> 0: Documentation hasn't been reviewed<br>+1: Documentation looks good<br><div><br></div>> * Should a +1/+2 be required to submit?<br><div><br></div>Yes, +1 should be required to submit.<br><div><br></div>> * Should -1/-2 block submits?<br><div><br></div>Yes, -1 should block submits.<br><div><br></div>> <br>>> Can we define a group of users that have permissions to set this<br>>> flag?<br>> <br>> Yep, for that a list of the gerrit usernames will be required. Also<br>> think if you want to give any other group (like maintainers) the<br>> ability to set it too, as depending on the policy you select in the<br>> previous question it might be required to merge.<br><div><br></div>Yes, the maintainers group should have permission to set this flag.<br><div><br></div>To start with, the names of the users of that group should be the<br>following:<br><div><br></div> Lucy Bopf <lbopf at redhat.com><br> Andrew Dahms <adahms at redhat.com><br><div><br></div>Derek, Lucy, Andrew, any other names to add to this group?<br><div><br></div>David, please wait for comments from Derek, Lucy and Andrew before<br>making this changes.<br><div><br></div>-- <br>Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta<br>3ºD, 28016 Madrid, Spain<br>Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.<br></blockquote><div><br></div></div></body></html>