<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><font class="Apple-style-span" face="'Courier New'">Hey Nova Core.</font><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">It is that time again. We've fallen behind on reviews. I know everyone is extremely busy, but we can't make progress on merging patches if we don't have people doing reviews. Here are the stats on the number of different change sets each person in nova core has reviewed in the past two weeks:</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">Brian Waldon 57<br>Sandy Walsh 38<br>Brian Lamar 37<br>Paul Voccio 22<br>Chris Behrens 21<br>Matt Dietz 18<br>Mark Washenberger 17<br>Dan Prince 15<br>Vish Ishaya 14<br>Soren Hansen 9<br>Christopher MacGown 6<br>Devin Carlen 6<br>Rick Harris 5<br>Trey Morris 5<br>Josh Kearney 2<br>Joshua McKenty 1<br>Jesse Andrews 0<br>Todd Willey 0 <br>Andy Smith 0 <br>Monty Taylor 0</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">Waldon, thanks for making the rest of us look bad. I would like to encourage everyone to ramp up the number of reviews. I think we should do a stat check every two weeks, and I'd like to propose that anyone who has single digit reviews on two consecutive lists is removed from nova-core. Nova-core has to take the responsibility for trunk code, and we can't do it if core isn't reviewing. Remember it is a responsibility, not a privilege.</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">We have also had some non-core members helping out with reviews. I would love to see more of this, especially from the subteams. Numbers from the past two weeks:</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">Jenkins 72<br>Kevin L. Mitchell 20<br>Johannes Erdfelt 11<br>Lorin Hochstein 13<br>Aaron Lee 6<br>Stanislaw Pitucha 6<br>Brad Hall 5<br>Jason Kölker 4<br>Naveed Massjouni 4<br>Anne Gentle 4<br>Asbjørn Sannes 3<br>joe-gordon0 3</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">Based on this, I'd like to propose we add Jenkins to nova-core :p. There are a few people on this list that could definitely be added, so if anyone feels like proposing some of these people for core, please do.</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'">Finally, we have a couple patches that need some review before tomorrow, so dive in:</font></div><div><font class="Apple-style-span" face="'Courier New'"><br></font></div><div><font class="Apple-style-span" face="'Courier New'"><a href="https://review.openstack.org/#change,976">https://review.openstack.org/#change,976</a></font></div><div><a href="https://review.openstack.org/#change,916"><font class="Apple-style-span" face="'Courier New'">https://review.openstack.org/#change,916</font></a></div></body></html>