<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Feb 29, 2016 at 6:42 PM, Clark Boylan <span dir="ltr"><<a href="mailto:cboylan@sapwetik.org" target="_blank">cboylan@sapwetik.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Feb 29, 2016, at 09:09 AM, Steven Dake (stdake) wrote:<br>
><br>
><br>
> On 2/29/16, 12:26 AM, "Andreas Jaeger" <<a href="mailto:aj@suse.com">aj@suse.com</a>> wrote:<br>
</span><span class="">> >This is not needed, the CI system always rebases if you run tests. To<br>
> >get current tests, a simple "recheck" is enough.<br>
> ><br>
> >Also, we test in the gate before merging - again after rebasing to head.<br>
> >That should take care of not merging anything broken. Running recheck<br>
> >after a larger change will ensure that you have recent results.<br>
><br>
> Andreas,<br>
><br>
> Thanks for the recheck information.  I thought the gate ran against what<br>
> it was submitted with as head.  We don't have any gate jobs at present<br>
> (or<br>
> many) they are mostly check jobs, so its pre-merge checking that we need<br>
> folks to do.<br>
><br>
</span>To clarify the check pipeline changes are merged into their target<br>
branches before testing and the gate pipeline changes are merged into<br>
the forward looking state of the world that Zuul generates as part of<br>
gate testing. This means you do not need to rebase and create new<br>
patchsets to get updated test results, just recheck.<br>
<br></blockquote><div><br></div><div>Unfortunately we do not have voting gates in Kolla that do building and deploying. This will change soon, but that would be where the confusion in the thread is coming from I believe. Our only indication is a "Green" check job at this time. This is why Steven is asking for a rebase to make the check gates green before we merge new patches.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You should only need to rebase and create a new patchset if a merge<br>
conflict exists.<br>
<span class="HOEnZb"><font color="#888888"><br>
Clark<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>