<br><br><div class="gmail_quote">On Tue, Jul 3, 2012 at 9:54 AM, Monty Taylor <span dir="ltr"><<a href="mailto:mordred@inaugust.com" target="_blank">mordred@inaugust.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
On 07/03/2012 08:47 AM, Doug Hellmann wrote:<br>
><br>
> On Jul 3, 2012, at 5:57 AM, Thierry Carrez <<a href="mailto:thierry@openstack.org">thierry@openstack.org</a>><br>
> wrote:<br>
><br>
>> Monty Taylor wrote:<br>
>>> At the moment, the only people with permission to upload tags is<br>
>>> the openstack-release team. However, since we're letting client<br>
>>> libs manage their own versions, I kinda think we should give PTLs<br>
>>> the right on their own project - so Vish would get tag access to<br>
>>> python-novaclient, Brian to python-glanceclient, etc.<br>
>><br>
>> Ideally it would be a two-side approval process (PTL +<br>
>> openstack-release), because openstack-release shouldn't be able to<br>
>> tag without PTL approval, and openstack-release should still be<br>
>> kept in the loop before a tag is pushed by PTLs (there are multiple<br>
>> reasons why a few hours delay before tagging would be a good idea,<br>
>> and the openstack-release people actually keep track of those).<br>
>><br>
>> That said, we don't have that approval mechanism available yet<br>
>> (same issue with the core projects release tagging) so in the mean<br>
>> time we should probably let the small set of individuals with an<br>
>> understanding of the issues (PTLs + openstack-release) have the<br>
>> power to do it. Within that group, we can have a soft two-side<br>
>> approval process (based on IRC pings) to check "everything is OK"<br>
>> before triggering a release.<br>
><br>
> Could we use gerrit's 2-step approval like we do for other projects<br>
> and combine it with a fancy tag detector like was just added for<br>
> DocImpact?<br>
<br>
</div></div>I have an outstanding bug/feature request for gerrit to allow for the<br>
review of and approval or disapproval of tags.<br></blockquote><div><br></div><div>Ah, I actually meant the "magical text" feature you mention below, but approving tags sounds like a good thing to have.</div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
That being said - if we wanted to go the route you're talking about in<br>
the mean time, instead of actually using the git tag route we could have<br>
an additional commit with a magical text in the commit message which, on<br>
merging, would cause a tag to be created... We've got guys on the team<br>
who hack gerrit now though - lemme get some feedback on how much it<br>
would take to actually get proper tag reviewing.<br></blockquote><div><br></div><div>Sounds good.</div><div><br></div><div>Doug</div><div><br></div></div>