<div dir="ltr">Hi Eyal,<div><br></div><div>Most of the hooks are updated not to use the STABLE_BRANCHES parameter, but there are still few hooks that are using this parameter such as: 'patchset-created.warn_if_not_merged_to_previous_branch' hook.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Best Regards,</div><div dir="ltr"><br></div><div dir="ltr">Shlomi Ben-David | DevOps Engineer <span style="font-size:small">| </span><span style="font-size:12.8px">Red Hat ISRAEL</span></div><div>RHCSA | <span style="font-size:small">RHCE</span><span style="font-size:small"> </span><span style="font-size:12.8px"> </span></div><div dir="ltr">IRC: shlomibendavid <span style="font-size:small">(on #rhev-integ, #rhev-dev, #rhev-ci)</span><br><br>OPEN SOURCE - 1 4 011 && 011 4 1<br></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Sun, Oct 30, 2016 at 5:25 PM, Eyal Edri <span dir="ltr"><<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Oct 10, 2016 at 6:24 PM, Francesco Romani <span dir="ltr"><<a href="mailto:fromani@redhat.com" target="_blank">fromani@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>----- Original Message -----<br>
> From: "Dan Kenigsberg" <<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>><br>
> To: "Francesco Romani" <<a href="mailto:fromani@redhat.com" target="_blank">fromani@redhat.com</a>><br>
> Cc: "Nir Soffer" <<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>>, <a href="mailto:devel@ovirt.org" target="_blank">devel@ovirt.org</a><br>
> Sent: Monday, October 10, 2016 5:11:26 PM<br>
> Subject: Re: [vdsm] branch ovirt-4.0.5 created<br>
><br>
> On Mon, Oct 10, 2016 at 10:30:49AM -0400, Francesco Romani wrote:<br>
> > Hi everyone,<br>
> ><br>
> > this time I choose to create the ovirt-4.0.5 branch.<br>
> > I already merged some patches for 4.0.6.<br>
> ><br>
> > Unfortunately I branched a bit too early (from last tag :))<br>
> ><br>
> > So patches<br>
> > <a href="https://gerrit.ovirt.org/#/c/65303/1" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5303/1</a><br>
> > <a href="https://gerrit.ovirt.org/#/c/65304/1" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5304/1</a><br>
> > <a href="https://gerrit.ovirt.org/#/c/65305/1" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5305/1</a><br>
> ><br>
> > Should be trivially mergeable - the only thing changed from ovirt-4.0<br>
> > counterpart<br>
> > is the change-id. Please have a quick look just to doublecheck.<br>
><br>
> Change-Id should be the same for a master patch and all of its backport.<br>
> It seems that it was NOT changed, at least for<br>
> <a href="https://gerrit.ovirt.org/#/q/I5cea6ec71c913d74d95317ff7318259d64b40969" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/q/I<wbr>5cea6ec71c913d74d95317ff731825<wbr>9d64b40969</a><br>
> which is a GOOD thing.<br>
<br>
</span>Yes, sorry, indeed it is (and indeed it should not change).<br>
<span><br>
> I think we want to enable CI on the new 4.0.5 branch, right? Otherwise<br>
> we'd need to fake the CI+1 flag until 4.0.5 is shipped.<br>
<br>
</span>We should, but it is not urgently needed - just regular priority.<br>
For the aforementioned first three patches especially I'm just overly<br>
cautious.<br>
<span class="m_-4056231478165911074HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div></div></div><div>Was CI enabled for 4.0.5 branch?</div><div>Adding infra as well.</div><div><br></div><div>Shlomi, Did we enabled the regex for stable branch already and we don't need to manually update conf files?</div><span class=""><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="m_-4056231478165911074HOEnZb"><font color="#888888">
--<br>
Francesco Romani<br>
Red Hat Engineering Virtualization R & D<br>
Phone: 8261328<br>
IRC: fromani<br>
</font></span><div class="m_-4056231478165911074HOEnZb"><div class="m_-4056231478165911074h5">______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br>
<br>
<br>
</div></div></blockquote></span></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div class="m_-4056231478165911074gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Eyal Edri<br>Associate Manager</div><div>RHV DevOps<br>EMEA ENG Virtualization R&D<br>Red Hat Israel<br><br>phone: <a href="tel:%2B972-9-7692018" value="+97297692018" target="_blank">+972-9-7692018</a><br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div></div></div></div></div></div></div>
</font></span></div></div>
</blockquote></div><br></div>