<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=us-ascii"><meta name=Generator content="Microsoft Word 12 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.hoenzb
        {mso-style-name:hoenzb;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body lang=EN-US link=blue vlink=purple><div class=WordSection1><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D'>I took Mondays!<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D'>Edgar<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D'><o:p> </o:p></span></p><div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal><b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'>From:</span></b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'> netstack-bounces+eperdomo=cisco.com@lists.launchpad.net [mailto:netstack-bounces+eperdomo=cisco.com@lists.launchpad.net] <b>On Behalf Of </b>Salvatore Orlando<br><b>Sent:</b> Monday, June 11, 2012 10:06 AM<br><b>To:</b> Dan Wendlandt<br><b>Cc:</b> netstack@lists.launchpad.net; openstack@lists.launchpad.net<br><b>Subject:</b> Re: [Netstack] [Quantum] Review days for Quantum core devs<o:p></o:p></span></p></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>As the link to the wiki page is now buried deep in this thread, here it is again:  <a href="http://wiki.openstack.org/Quantum/ReviewDays">http://wiki.openstack.org/Quantum/ReviewDays</a> <o:p></o:p></p><div><p class=MsoNormal>Weekend days have been added on GaryK's request!<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Salvatore<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p><div><p class=MsoNormal>On 11 June 2012 16:51, Dan Wendlandt <<a href="mailto:dan@nicira.com" target="_blank">dan@nicira.com</a>> wrote:<o:p></o:p></p><p class=MsoNormal style='margin-bottom:12.0pt'><o:p> </o:p></p><div><div><p class=MsoNormal>On Mon, Jun 11, 2012 at 2:25 AM, Salvatore Orlando <<a href="mailto:salv.orlando@gmail.com" target="_blank">salv.orlando@gmail.com</a>> wrote:<o:p></o:p></p><p class=MsoNormal>Hi,<o:p></o:p></p><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>During our last project meeting there was some interest in introducing "review days" for Quantum core devs.<o:p></o:p></p></div><div><p class=MsoNormal>I see multiple benefits in this approach:<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>1) Contributors would know which core member the need to contact on a given days for soliciting a review for their patches<o:p></o:p></p></div><div><p class=MsoNormal>2) Predictable code review effort for core devs<o:p></o:p></p></div><div><p class=MsoNormal>3) The PTL knows who should be whipped!<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div></div><div><p class=MsoNormal>indeed :)  <o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Thanks for taking the lead on this Salvatore.  Please send out the wiki page, no need for more +1s :)<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>dan<o:p></o:p></p></div><div><p class=MsoNormal> <o:p></o:p></p></div><blockquote style='border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in'><div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>I think we can leverage the fact that our core devs span several time zones. So for each day we can have, for instance, an EMEA core reviewer on duty between 1AM and 9AM PST, an US core reviewer on duty between 9AM and 5PM PST, and an APAC core reviewer on duty between 5PM and 1AM PST. <o:p></o:p></p></div><div><p class=MsoNormal>Of course the plan needs to be flexible especially around milestone release, where all the devs which are not busy coding should probably dedicate more time to code review.<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>I am going to start a wiki page with a 5x3 grid, with days on the rows and zones on the columns. If you like this idea, +1 this email and add your name to the grid :)<o:p></o:p></p></div><div><p class=MsoNormal>Otherwise, please let me have your feedback and concerns.<o:p></o:p></p></div><div><p class=MsoNormal><span style='color:#888888'><o:p> </o:p></span></p></div><div><p class=MsoNormal><span style='color:#888888'>Salvatore<o:p></o:p></span></p></div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>--<br>Mailing list: <a href="https://launchpad.net/~netstack" target="_blank">https://launchpad.net/~netstack</a><br>Post to     : <a href="mailto:netstack@lists.launchpad.net" target="_blank">netstack@lists.launchpad.net</a><br>Unsubscribe : <a href="https://launchpad.net/~netstack" target="_blank">https://launchpad.net/~netstack</a><br>More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><o:p></o:p></p></div></blockquote></div><p class=MsoNormal><span style='color:#888888'><br><br clear=all><span class=hoenzb><o:p></o:p></span></span></p><div><p class=MsoNormal><o:p> </o:p></p></div><p class=MsoNormal><span class=hoenzb><span style='color:#888888'>-- </span></span><span style='color:#888888'><br><span class=hoenzb>~~~~~~~~~~~~~~~~~~~~~~~~~~~</span><br><span class=hoenzb>Dan Wendlandt </span></span><span class=hoenzb><o:p></o:p></span></p><div><p class=MsoNormal><span style='color:#888888'>Nicira, Inc: <a href="http://www.nicira.com" target="_blank">www.nicira.com</a></span><o:p></o:p></p><div><p class=MsoNormal><span style='color:#888888'>twitter: danwendlandt<br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<o:p></o:p></span></p></div></div><p class=MsoNormal><o:p> </o:p></p></div><p class=MsoNormal><o:p> </o:p></p></div></div></body></html>