<div dir="ltr"><div class="gmail_default" style="font-family:courier new,monospace"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 24, 2014 at 8:31 AM, John Dennis <span dir="ltr"><<a href="mailto:jdennis@redhat.com" target="_blank">jdennis@redhat.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">When a change is complex good practice is to break the change into a<br>
series of smaller individual patches that show the individual<br>
incremental steps needed to get to the final goal. When partitioned into<br>
small steps each change is easier to review and hopefully illustrates<br>
the progression.<br></blockquote><div class="gmail_default" style="font-family:'courier new',monospace">Definitely agree, however I've noticed people aren't necessarily very *good* about breaking these into logical pieces sometimes.  In other words it becomes random changes throughout multiple patches; in most cases it seems to be after-thoughts or just what the submitter managed to work on at the time.</div>

<div class="gmail_default" style="font-family:'courier new',monospace"><br></div><div class="gmail_default" style="font-family:'courier new',monospace">Personally I'd love to see these be a bit more well thought out and organized for my own sake as a reviewer.  While we're at it (I realize this isn't the case you're talking about) I also would REALLY like to not see 5 individual patches all dependent on each other and all just changing one or two lines (I was seeing this quite a bit this cycle, and the only thing I can think of is perhaps it's developers getting some sort of points for number of commits).</div>

<div class="gmail_default" style="font-family:'courier new',monospace"></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In most cases such a series of patches are interdependent and order<br>
dependent, jenkins cannot run tests on any patch unless the previous<br>
patch has been applied.<br>
<br>
I was under the impression gerrit review supported multiple commits. In<br>
fact you can submit multiple commits with a single "git review" command.<br>
<br>
But from that point forward it appears as if each commit is handled<br>
independently rather than being an ordered list of commits that are<br>
grouped together sharing a single review where their relationship is<br>
explicit. Also the jenkins tests either needs to apply all the commits<br>
in sequence and run the test or it needs to run the test after applying<br>
the next commit in the sequence.<br>
<br>
Can someone provide some explanation on how to handle this situation?<br></blockquote><div class="gmail_default" style="font-family:'courier new',monospace"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
Or perhaps I'm just not understanding how the tools work when multiple<br>
commits are submitted.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
John<br>
<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>
</font></span></blockquote></div><br></div></div>