<div dir="ltr">On Fri, Feb 17, 2017 at 8:44 PM, Matt Riedemann <span dir="ltr"><<a href="mailto:mriedemos@gmail.com" target="_blank">mriedemos@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5">On 2/15/2017 12:40 PM, Zane Bitter wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Traditionally Heat has given current and former PTLs of the project +2<br>
rights on stable branches for as long as they remain core reviewers.<br>
Usually I've done that by adding them to the heat-release group.<br>
<br>
At some point the system changed so that the review rights for these<br>
branches are no longer under the team's control (instead, the<br>
stable-maint core team is in charge), and as a result at least the<br>
current PTL (Rico Lin) and the previous PTL (Rabi Mishra), and possibly<br>
others (Thomas Herve, Sergey Kraynev), haven't been added to the group.<br>
That's slowing down getting backports merged, amongst other things.<br>
<br>
I'd like to request that we update the membership to be the same as<br>
<a href="https://review.openstack.org/#/admin/groups/152,members" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/admin/groups/152,members</a><br>
<br>
Rabi Mishra<br>
Rico Lin<br>
Sergey Kraynev<br>
Steve Baker<br>
Steven Hardy<br>
Thomas Herve<br>
Zane Bitter<br>
<br>
I also wonder if the stable-maint team would consider allowing the Heat<br>
team to manage the group membership again if we commit to the criteria<br>
above (all current/former PTLs who are also core reviewers) by just<br>
adding that group as a member of heat-stable-maint?<br>
<br>
thanks,<br>
Zane.<br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
</blockquote>
<br></div></div>
Reviewing patches on stable branches have different guidelines, expressed here [1]. In the past when this comes up I've asked if the people being asked to be added to the stable team for a project have actually been doing reviews on the stable branches to show they are following the guidelines, and at times when this has come up the people proposed (usually PTLs) haven't, so I've declined at that time until they start actually doing reviews and can show they are following the guidelines.<br>
<br>
There are reviewstats tools for seeing the stable review numbers for Heat, I haven't run that though to check against those proposed above, but it's probably something I'd do first before just adding a bunch of people.<br></blockquote><div><br>Would it not be appropriate to trust the stable cross-project liaison 
for heat when he nominates stable cores? Having been the PTL for Ocata 
and one who struggled to get the backports on time for a stable release 
as planned, I don't recall seeing many reviews from stable maintenance 
core team for them to be able to judge the quality of reviews. So I 
don't think it's fair to decide eligibility only based on the review 
numbers and stats. <br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
[1] <a href="https://docs.openstack.org/project-team-guide/stable-branches.html" rel="noreferrer" target="_blank">https://docs.openstack.org/pro<wbr>ject-team-guide/stable-branche<wbr>s.html</a><span class="gmail-HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt Riedemann</font></span><div class="gmail-HOEnZb"><div class="gmail-h5"><br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
</div></div></blockquote></div><br>-- <br><div class="gmail_signature"><div dir="ltr"><div>Regards,</div>Rabi Mishra<div><br></div></div></div>
</div></div>