<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 12, 2014 at 11:03 AM, Duncan Thomas <span dir="ltr"><<a href="mailto:duncan.thomas@gmail.com" target="_blank">duncan.thomas@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="">On 15 January 2014 18:53, Brant Knudson <<a href="mailto:blk@acm.org">blk@acm.org</a>> wrote:<br>

<br>
> At no point do I care what are the different commits that are being brought<br>
> in from oslo-incubator. If the commits are listed in the commit message then<br>
> I feel an obligation to verify that they got the right commits in the<br>
> message and that takes extra time for no gain.<br>
<br>
</div>I find that I very much *do* want a list of what changes have been<br>
pulled in, so I've so idea of the intent of the changes. Some of the<br>
OSLO changes can be large and complicated, and the more clues as to<br>
why things changed, the better the chance I've got of spotting<br>
breakages or differing assumptions between cinder and OSLO (of which<br>
there have been a number)<br>
<br>
I don't very often verify that the version that has been pulled in is<br>
the very latest or anything like that - generally I want to know:<br></blockquote><div><br></div><div>One thing that I think we should be verifying is that the changes being brought over have actually been committed to oslo-incubator. I'm sure there have been times where someone eager to get the fix in has not waited for the oslo-incubator merge before syncing their change over.<br>
<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
 - What issue are you trying to fix by doing an update? (The fact OSLO<br>
is ahead of us is rarely a good enough reason on its own to do an<br>
update - preferably reference a specific bug that exists in cinder)<br></blockquote><div><br>When I sync a change from oslo-incubator to fix a bug I put Closes-Bug on the commit message to indicate 
what bug is being fixed. If the sync tool was enhanced to pick out the 
*-Bug references from the oslo commits to include in the sync commit 
message that would be handy.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
 - What other incidental changes are being pulled in (by intent, not<br>
just the code)<br>
 - If I'm unsure about one of the incidental changes, how do I go find<br>
the history for it, with lots of searching (hense the commit ID or the<br>
change ID) - this lets me find bugs, reviews etc<br></blockquote><div><br></div><div>How does one get the list of commits that are being brought over from oslo-incubator? You'd have to know what the previous commit was that was synced.<br>
</div><div><br></div><div>- Brant<br><br></div></div></div></div>