<div dir="ltr">For what is worth, I'm trying below to provide my perspective on Luke's question both as a reviewer and as developer.<div><br></div><div>Salvatore<br><div class="gmail_extra"><br><br><div class="gmail_quote">
On 26 July 2014 20:02, Luke Gorrie <span dir="ltr"><<a href="mailto:luke@snabb.co" target="_blank">luke@snabb.co</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="">On 25 July 2014 20:05, Stefano Maffulli <span dir="ltr"><<a href="mailto:stefano@openstack.org" target="_blank">stefano@openstack.org</a>></span> wrote:<br></div><div class="gmail_extra">
<div class="gmail_quote"><div class="">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>Indeed, communication is key. I'm not sure how you envision to<br>

</div>
implement this though. We do send a message to first time<br>
contributors[1] to explain them how the review process works and give<br>
them very basic suggestions on how to react to comments (including what<br>
to do if things seem stuck). The main issue here though is that few<br>
people read emails, it's a basic fact of life.<br></blockquote><div><br></div></div><div>That welcome message does seem to do a really good job of setting expectations.</div><div class=""><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Can you explain more what you have in mind?<br></blockquote><div><br></div></div><div>Here are some other topics that seem to take some time to develop a mental model of:</div><div><br></div><div>How quickly and how often should you revise your patchset after a -1? (Is it better to give the community a week or so to collectively comment? Or should you revise ASAP after every negative review?)</div>
</div></div></div></blockquote><div><br></div><div>This is a very hard question to answer. From a developer perspective often you have tenths of outstanding patches, or the required changes translates into a need of redoing your patch from scratch, so quickly addressing a -1 is not possible. On the other hand, from a reviewer perspective, if there is too much time between two reviews then "memory" of previous reviews will be lost and the review process will basically begin again from scratch.</div>
<div><br></div><div>Usually in order to help reviewers I try to be as pedant as possible in the commit message. I also add comments on the patch to help reviewers understand how I address their comments. This might help reviewers working on your patch, and avoid questions which, despite being legitimate, will just add more time to the review process for your patch.</div>
<div>I try to ping core reviewers on IRC as less as possible, unless there is some sort of urgency from a community perspective. This is because I think all core reviewers have their own review queue, and are typically already working on sorting out reviews according to that queue.</div>
<div>However, if you can identify a core reviewer which has reviewed several times your patch, then I guess it's ok to ping him directly, especially when you are at the last stages of the review cycle and your patch just needs a little push.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div><br></div><div>How do you know if your change is likely to merge? (If you have had 15 rounds of -1 votes and the last milestone deadline is a few days away, should you relax because your code is so thoroughly reviewed or should you despair because it should have been merged by now?)</div>
</div></div></div></blockquote><div><br></div><div>If you have 15 -1 votes in just 1 round maybe you should consider abandoning your patch!</div><div>Jokes apart, "-1" has a whole set of meanings. As a core reviewer I can -1 because:</div>
<div>- I think your approach is completely wrong and you should do your patch again from scratch</div><div>- Your patch is ok but you should consider a more efficient implementation</div><div>- Somebody else has done a patch like yours</div>
<div>- The patch is ok but there are a few style/readability/maintainability nits to address</div><div>- I have a possibly dumb question on the approach you've chosen and I would like to discuss that with you before approving</div>
<div>Therefore the likeness of your patch merging depends on the specific nature of the -1 you received.</div><div>I try to avoid -2s as much as possible. I put a -2 only when I reckon your patch should never be merged because it'll make the software unstable or tries to solve a problem that does not exist. -2s stick across patches and tend to put off other reviewers.</div>
<div><br></div><div>Regarding your specific question, if after 15 rounds you still have a -1 that means you have at least a core reviewer constantly following it, so you should probably be relaxed.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div><br></div><div>In the final days before a merge deadline, would it be rude to "poke" the person responsible for merging, or would it be negligent not to?</div></div></div></div></blockquote><div><br></div>
<div>Poking is never rude. However, poking a reviewer is not a guarantee that your patch will be approved. The reviewer may add your patch to his/her queue, but it might still slip through the cracks.</div><div>Most core reviewers two weeks before release time enter a "review crunch" period where it's not unlikely for them to look at 10 to 20 patches per day.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>How do you decide which IRC meetings to attend? (For meetings that occur at difficult times outside of working hours in your timezone, when are you expected to attend them? Is it okay to focus on email/informal communication if that suits you better and gets the job done?)</div>
</div></div></div></blockquote><div><br></div><div>Unfortunately there are too many meetings. I usually look at the agendas for the meetings where I can be remotely interested in. If there's a meeting where I think I have something to say, I will participate. Otherwise I will read the meeting minutes and logs.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div><br></div><div>If you're new to the project and you don't know anybody, who can you ask about this stuff?</div></div></div></div></blockquote><div><br></div><div>I guess we'd just need to document that, but how this should be formalised is probably more a question for the TC and the community manager. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div><br></div></div></div></div>
<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>
<br></blockquote></div><br></div></div></div>