<div dir="ltr">Hi Nachi, <div><br></div><div><br></div><div>Sorry for question, but did you try stackalytics?</div><div><br></div><div>E.g. this page <a href="http://stackalytics.com/report/contribution/neutron/30">http://stackalytics.com/report/contribution/neutron/30</a></div>
<div><br></div><div>Best regards,</div><div>Boris Pavlovic </div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 20, 2014 at 5:20 AM, Nachi Ueno <span dir="ltr"><<a href="mailto:nachi@ntti3.com" target="_blank">nachi@ntti3.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks<br>
<br>
As per neutron discussion,  we agreed there is really important to distribute<br>
 core-reviewer loads.<br>
so we got an idea to specify primary/secondary reviewers for each reviews.<br>
<br>
IMO, this is almost impossible without some helper tool.<br>
<br>
so, I wrote it.<br>
<br>
<a href="https://www.youtube.com/watch?v=ouS5h5Z-W50&feature=youtu.be" target="_blank">https://www.youtube.com/watch?v=ouS5h5Z-W50&feature=youtu.be</a><br>
<br>
This is a review for review stats.<br>
<a href="https://review.openstack.org/#/c/94288/" target="_blank">https://review.openstack.org/#/c/94288/</a><br>
# Note, this code is my side work,, so hopefully we can get lower bar<br>
for review :)<br>
<br>
sudo python setup.py develop<br>
./run_server.sh<br>
open <a href="http://127.0.0.1:8080/" target="_blank">http://127.0.0.1:8080/</a><br>
<br>
This webui identifies primary/secondary reviewers from comment.<br>
so if you put the comment like this,<br>
<br>
primary: mestery, secondary:maru<br>
<br>
Kyle, and Maru will be assigned as primary/secondary reviewer.<br>
This is regexp "primary:\s*(.+),\s*secondary:\s*(.+)"<br>
<br>
Otherwise, this webui selects primary/secondary from existing<br>
core-reviewer who are reviewing the change set.<br>
<br>
Some my thoughts for next steps.<br>
<br>
(1) core-reviewer-scheduler<br>
Currently, 208 patch needs core in Neutron related projects.<br>
I feel manual assignment by discussion won't work in this scale...<br>
<br>
I think we need core-reviewer-scheduler such as nova-scheduler too.<br>
# I'm not sure cores likes this idea, though<br>
<br>
(2) launchpad integration<br>
Yep, it is great if we can also show priorities by launchpad data in here.<br>
<br>
(3) Hosting this code in somewhere, (hopefully, in infra)<br>
(4) Performance impact for <a href="http://review.openstack.org" target="_blank">review.openstack.org</a><br>
This code is using a heavy rest api call (get all reviews,comment,<br>
reviews for all patches for a project). I hope this code won't kill<br>
the <a href="http://review.openstack.org" target="_blank">review.openstack.org</a><br>
<br>
Best<br>
Nachi<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>