<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Brant Knudson <blk@acm.org></font>
<br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">"OpenStack Development
Mailing List (not for usage questions)" <openstack-dev@lists.openstack.org>,
</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">03/14/2014 10:26 AM</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: [openstack-dev]
[Oslo] Improving oslo-incubator update.py</font>
<br>
<hr noshade>
<br>
<br>
<br>
<br>
<br><font size=3>On Wed, Mar 12, 2014 at 11:03 AM, Duncan Thomas <</font><a href=mailto:duncan.thomas@gmail.com target=_blank><font size=3 color=blue><u>duncan.thomas@gmail.com</u></font></a><font size=3>>
wrote:</font>
<br><font size=3>On 15 January 2014 18:53, Brant Knudson <</font><a href=mailto:blk@acm.org><font size=3 color=blue><u>blk@acm.org</u></font></a><font size=3>>
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.</font>
<br>
<br><font size=3>--> Duncan, It is important to know what commits are
being brought over to help provide a pointer to</font>
<br><font size=3>--> the possible cause of subsequent bugs that arise.
 I.E. if we sync up the DB, there is a commit for fixing</font>
<br><font size=3>--> db connection order and suddenly we are getting
intermittent DB connection failures, it give us</font>
<br><font size=3>--> a starting point to fixing the issue.<br>
</font>
<br><font size=3>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:</font>
<br>
<br><font size=3>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>
</font>
<br><font size=3> - 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)</font>
<br><font size=3><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>
 </font>
<br><font size=3> - 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</font>
<br>
<br><font size=3>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.</font>
<br>
<br><font size=3>- Brant<br>
</font><tt><font size=2>_______________________________________________<br>
OpenStack-dev mailing list<br>
OpenStack-dev@lists.openstack.org<br>
</font></tt><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev"><tt><font size=2>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</font></tt></a><tt><font size=2><br>
</font></tt>
<br>