<div dir="ltr">Hi,<div><br></div><div>Tal: The &#39;<i>Check merged to previous</i>&#39; hook is checking if the current patch was merged to previous stable branches and i think that it is not related to the one that you&#39;re searching.</div><div><br></div><div>Eyal: Currently there is no hook that checks if branch is enabled, but there is hook that checks if the patch major version branch is [equal / higher / lower] than the bug target milestone major version.</div><div>i&#39;ll explain by giving you few examples and their results:</div><div><br></div><div><div>Example 1:</div><div>tm: &#39;---&#39;    (empty target milestone)</div><div>branch major version: any</div><div>result: <font color="#ff0000"><b>WARN MSG</b></font></div><div><br></div><div>Example 2:</div><div>tm major version: 3.6</div><div>branch major version: 3.6</div><div>result: <font color="#00ff00"><b>OK MSG</b></font></div><div> </div><div>Example 3:</div><div>tm major version: 3.6</div><div>branch major version: 4.0</div><div>result: checks for clone flags </div><div>(if tm major version in flags - <font color="#00ff00"><b>OK MSG</b></font> else <font color="#ff0000"><b>WARN MSG</b></font>)</div><div> </div><div>Example 4:</div><div>tm major version: 4.0</div><div>branch major version: 3.6</div><div>result: <font color="#ff0000"><b>WARN MSG</b></font></div></div><div><br></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 &amp;&amp; 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 9:45 AM, Eyal Edri <span dir="ltr">&lt;<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Shlomi, <div>Can you check if we have the hook for verifying right branch is enabled? </div><div><br></div><div><br></div></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Thu, Oct 27, 2016 at 11:00 AM, Tal Nisan <span dir="ltr">&lt;<a href="mailto:tnisan@redhat.com" target="_blank">tnisan@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This patch was pushed to 4.0.5 with a bug that at the time of the push was 4.1, Gerrit hooks should have marked it as -verified, instead you get:<div><ul class="m_8370991219207361263m_-1851721314793783974gmail-com-google-gwtexpui-safehtml-client-SafeHtmlCss-wikiList" style="color:rgb(53,53,53);font-family:sans-serif"><li>Check merged to previous::IGNORE, Not in stable branch ([&#39;ovirt-engine-3.6&#39;, &#39;ovirt-engine-4.0&#39;, &#39;ovirt-engine-3.6.9&#39;, &#39;ovirt-engine-4.0.2&#39;, &#39;ovirt-engine-4.0.4&#39;])</li></ul></div></div><div class="m_8370991219207361263HOEnZb"><div class="m_8370991219207361263h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 27, 2016 at 9:55 AM, Gil Shinar <span dir="ltr">&lt;<a href="mailto:gshinar@redhat.com" target="_blank">gshinar@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Can you please elaborate?<div><br></div><div>Thanks</div><div>Gil</div></div><div class="gmail_extra"><br><div class="gmail_quote"><span>On Wed, Oct 26, 2016 at 11:50 AM, Tal Nisan <span dir="ltr">&lt;<a href="mailto:tnisan@redhat.com" target="_blank">tnisan@redhat.com</a>&gt;</span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><div dir="ltr">This patch for example:<div><a href="https://gerrit.ovirt.org/#/c/65398/" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5398/</a><br></div><div><br></div></div>
<br></span>______________________________<wbr>_________________<br>
Infra mailing list<br>
<a href="mailto:Infra@ovirt.org" target="_blank">Infra@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/infra</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
Infra mailing list<br>
<a href="mailto:Infra@ovirt.org" target="_blank">Infra@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/infra</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div class="m_8370991219207361263gmail_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&amp;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>
</blockquote></div><br></div></div>