<div dir="ltr"><div>+1<br></div>Nice work Vikram.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 31, 2016 at 4:04 PM, Jeff Peeler <span dir="ltr"><<a href="mailto:jpeeler@redhat.com" target="_blank">jpeeler@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Mar 29, 2016 at 12:07 PM, Steven Dake (stdake) <<a href="mailto:stdake@cisco.com">stdake@cisco.com</a>> wrote:<br>
> Hey folks,<br>
><br>
> Consider this proposal a +1 in favor of Vikram joining the core reviewer<br>
> team.  His reviews are outstanding.  If he doesn’t have anything useful to<br>
> add to a review, he doesn't pile on the review with more –1s which are<br>
> slightly disheartening to people.  Vikram has started a trend amongst the<br>
> core reviewers of actually diagnosing gate failures in peoples patches as<br>
> opposed to saying gate failed please fix.  He does this diagnosis in nearly<br>
> every review I see, and if he is stumped  he says so.  His 30 days review<br>
> stats place him in pole position and his 90 day review stats place him in<br>
> second position.  Of critical notice is that Vikram is ever-present on IRC<br>
> which in my professional experience is the #1 indicator of how well a core<br>
> reviewer will perform long term.   Besides IRC and review requirements, we<br>
> also have code requirements for core reviewers.  Vikram has implemented only<br>
> 10 patches so far, butI feel he could amp this up if he had feature work to<br>
> do.  At the moment we are in a holding pattern on master development because<br>
> we need to fix Mitaka bugs.  That said Vikram is actively working on<br>
> diagnosing root causes of people's bugs in the IRC channel pretty much 12-18<br>
> hours a day so we can ship Mitaka in a working bug-free state.<br>
><br>
> Our core team consists of 11 people.  Vikram requires at minimum 6 +1 votes,<br>
> with no veto –2 votes within a 7 day voting window to end on April 7th.  If<br>
> there is a veto vote prior to April 7th I will close voting.  If there is a<br>
> unanimous vote prior to April 7th, I will make appropriate changes in<br>
> gerrit.<br>
><br>
> Regards<br>
> -steve<br>
><br>
> [1] <a href="http://stackalytics.com/report/contribution/kolla-group/30" rel="noreferrer" target="_blank">http://stackalytics.com/report/contribution/kolla-group/30</a><br>
> [2] <a href="http://stackalytics.com/report/contribution/kolla-group/90" rel="noreferrer" target="_blank">http://stackalytics.com/report/contribution/kolla-group/90</a><br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> __________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>