<div dir="ltr">Erno, <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"><br><span style="color:rgb(31,73,125);font-family:Calibri,sans-serif;font-size:14.6666669845581px">My personal take is that if some piece of work has not been touched for a month, it’s probably not that important after all and the community should use the resources to do some work that has actual momentum. </span></blockquote><div><br></div><div>Based on my experience, one of the most common situation in OpenStack is next: </div><div>1) Somebody makes fast (but with right idea) changes, because he (and usually others) needs it</div><div>2) It doesn't pass review process fast</div><div>3) Author of this patch has billions others tasks (not related to upstream) and </div><div>can't work on this change anymore</div><div>4) Patch get's abounded and forgotten </div><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"><span style="color:rgb(31,73,125);font-family:Calibri,sans-serif;font-size:14.6666669845581px">The changes itself will not disappear the owner is still able to revive it if felt that there is right time to continue it.</span></blockquote><div><br></div><div>Nobody never reviews abounded changes..</div><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"><span style="color:rgb(31,73,125);font-family:Calibri,sans-serif;font-size:14.6666669845581px"> The cleanup will just make it easier for people to focus on things that are actually moving. </span></blockquote><div><br></div><div>Making decision based on activity around patches is not the best way to do things. </div><div><br></div><div>If we take a look at the structure of OpenStack projects we will see next things: </div><div><br></div><div>1) Things that re moving fast/good are usually related to things that core team (or active members) are working on.</div><div>This team is resolving limited set of use cases (this is mostly related because not every member is running it's own production cloud)</div><div><br></div><div>2) Operators/Admins/DevOps that are running their own cloud have a lot of experience knows a lot of missing use cases and </div><div>source of issues. But usually they are not involved in community process so they don't know whole road map of project, so they are not able to fully align their patches with road map, or eventually just don't have enough time to work on features. </div><div><br></div><div>So abounding patches from 2 group just because of inactivity can make big harm to project. </div><div><br></div><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"><span style="color:rgb(31,73,125);font-family:Calibri,sans-serif;font-size:14.6666669845581px"> Do you have resources in mind to dedicate for this?</span></blockquote><div><br></div><div>Sometimes I am doing it by my self, sometimes newbies in community (that want some work to get involved), sometimes core team is working on old patches. </div><div><br></div><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"><span style="font-size:12.8000001907349px">Important chagesets are supposed to have bugs (or blueprints) assigned<br></span><span style="font-size:12.8000001907349px">to them, so, even if the CS is abandoned, its description still<br></span><span style="font-size:12.8000001907349px">remains on Launchpad in one form or another, so we will not loose it<br></span><span style="font-size:12.8000001907349px">from general project's backlog</span></blockquote><div><br></div><div>This is not true in a lot of cases. =)</div><div> In many cases DevOps/Operators don't know or don't want to spend time for launchpad/specs/ and so on.</div><div><br></div><div><br></div><div><br></div><div>Best regards,</div><div>Boris Pavlovic </div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 13, 2015 at 5:17 PM, Kuvaja, Erno <span dir="ltr"><<a href="mailto:kuvaja@hp.com" target="_blank">kuvaja@hp.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Hi Boris,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Thanks for your input. I do like the idea of picking up the changes that have not been active. Do you have resources in mind to dedicate for this?<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">My personal take is that if some piece of work has not been touched for a month, it’s probably not that important after all and the community should use the
resources to do some work that has actual momentum. The changes itself will not disappear the owner is still able to revive it if felt that there is right time to continue it. The cleanup will just make it easier for people to focus on things that are actually
moving. It also will make bug tracking bit easier when one will see on the bug report that the patch got abandoned due to inactivity and indicates that the owner of that bug might not be working on it after all.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p><u></u><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><span>-<span style="font:7.0pt "Times New Roman"">
</span></span></span><u></u><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Erno<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt">
<div>
<div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri","sans-serif"">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri","sans-serif""> Boris Pavlovic [mailto:<a href="mailto:bpavlovic@mirantis.com" target="_blank">bpavlovic@mirantis.com</a>]
<br>
<b>Sent:</b> Friday, February 13, 2015 1:25 PM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [glance] Cleanout of inactive change proposals from review<u></u><u></u></span></p>
</div>
</div><div><div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">Hi,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I believe that keeping review queue clean is the great idea. <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">But I am not sure that set of these rules is enough to abandon patches.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Recently I wrote blogpost related to making OpenStack community more user friendly:<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><a href="http://boris-42.me/thoughts-on-making-openstack-community-more-user-friendly/" target="_blank">http://boris-42.me/thoughts-on-making-openstack-community-more-user-friendly/</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">tl;dr;<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Patches on review are great source of information what is missing in project.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Removing them from queue means losing this essential information. The result<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">of such actions is that project doesn't face users requirements which is quite bad...<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">What if that project team continue work on all "abandoned" patches that are covering <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">valid use cases and finish them?<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Best regards,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Boris Pavlovic <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On Fri, Feb 13, 2015 at 3:52 PM, Flavio Percoco <<a href="mailto:flavio@redhat.com" target="_blank">flavio@redhat.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">On 13/02/15 11:06 +0000, Kuvaja, Erno wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-bottom:12.0pt">Hi all,<br>
<br>
We have almost year old (from last update) reviews still in the queue for<br>
glance. The discussion was initiated on yesterday’s meeting for adopting<br>
abandon policy for stale changes.<br>
<br>
The documentation can be found from <a href="https://etherpad.openstack.org/p/" target="_blank">
https://etherpad.openstack.org/p/</a><br>
glance-cleanout-of-inactive-PS and any input would be appreciated. For your<br>
convenience current state below:<u></u><u></u></p>
</blockquote>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br>
Thanks for putting this together. I missed the meeting yday and this<br>
is important.<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">Glance - Cleanout of inactive change proposals from review<br>
<br>
<br>
We Should start cleaning out our review list to keep the focus on changes that<br>
has momentum. Nova is currently abandoning change proposals that has been<br>
inactive for 4 weeks.<br>
<br>
<br>
<br>
Proposed action (if all of the following is True, abandon the PS):<br>
<br>
1. The PS has -1/-2 (including Jenkins)<u></u><u></u></p>
</blockquote>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br>
I assume you're talking about voting -1/-2 and not Workflow, right?<br>
(you said jenkins afterall but just for the sake of clarity).<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">2. The change is proposed to glance, glance_store or python-glanceclient;<br>
specs should not be abandoned as their workflow is much slower<br>
<br>
3. No activity for 28 days from Author/Owner after the -1/-2<u></u><u></u></p>
</blockquote>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br>
I'd reword this in "No activity". This includes comments, feedback,<br>
discussions and or other committers taking over a patch.<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">4. There has been query made to the owner to update the patch between 5 and<br>
10 days before abandoning (comment on PS/Bug or something similar)<br>
<br>
● Let's be smart on this. Flexibility is good on holiday seasons, during<br>
feature freeze, etc.<u></u><u></u></p>
</blockquote>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br>
+2 to the above, I like it.<br>
<br>
Thanks again,<br>
Flavio<span style="color:#888888"><br>
<br>
<span>-- </span><br>
<span>@flaper87</span><br>
<span>Flavio Percoco</span><br>
</span><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">
OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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><u></u><u></u></p>
</blockquote>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div></div></div>
</div>
</div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>