<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 16, 2014 at 3:19 PM, Ben Nemec <span dir="ltr"><<a href="mailto:openstack@nemebean.com" target="_blank">openstack@nemebean.com</a>></span> wrote:<br>
<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 class="im">On 2014-01-16 13:48, John Griffith wrote:<br>

<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">
Hey Everyone,<br>
<br>
A review came up today that cherry-picked a specific commit to OSLO<br>
Incubator, without updating the rest of the files in the module.  I<br>
rejected that patch, because my philosophy has been that when you<br>
update/pull from oslo-incubator it should be done as a full sync of<br>
the entire module, not a cherry pick of the bits and pieces that you<br>
may or may not be interested in.<br>
<br>
As it turns out I've received a bit of push back on this, so it seems<br>
maybe I'm being unreasonable, or that I'm mistaken in my understanding<br>
of the process here.  To me it seems like a complete and total waste<br>
to have an oslo-incubator and common libs if you're going to turn<br>
around and just cherry pick changes, but maybe I'm completely out of<br>
line.<br>
<br>
Thoughts??<br>
</blockquote>
<br></div>
I suppose there might be exceptions, but in general I'm with you.  For one thing, if someone tries to pull out a specific change in the Oslo code, there's no guarantee that code even works.  Depending on how the sync was done it's possible the code they're syncing never passed the Oslo unit tests in the form being synced, and since unit tests aren't synced to the target projects it's conceivable that completely broken code could get through Jenkins.<br>

<br>
Obviously it's possible to do a successful partial sync, but for the sake of reviewer sanity I'm -1 on partial syncs without a _very_ good reason (like it's blocking the gate and there's some reason the full module can't be synced).</blockquote>
<div><br></div><div><div class="gmail_default" style="font-size:small">I agree. Cherry picking a single (or even partial) commit really should be avoided.</div><div class="gmail_default" style="font-size:small"><br></div>
<div class="gmail_default" style="font-size:small">The update tool does allow syncing just a single module, but that should be used very VERY carefully, especially because some of the changes we're making as we work on graduating some more libraries will include cross-dependent changes between oslo modules.</div>
<div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Doug</div><br></div><div> </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 class=""><font color="#888888"><br>
<br>
-Ben</font></span><div class=""><div class="h5"><br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>