<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 20, 2015 at 9:46 AM, Nir Soffer <span dir="ltr"><<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@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 class="">On Sun, Dec 20, 2015 at 8:49 AM, Barak Korren <<a href="mailto:bkorren@redhat.com">bkorren@redhat.com</a>> wrote:<br>
> On 19 December 2015 at 13:05, Yaniv Bronheim <<a href="mailto:ybronhei@redhat.com">ybronhei@redhat.com</a>> wrote:<br>
>> for some runs it is installed - we added it to the requirements for<br>
>> check-patch -<br>
>> <a href="https://gerrit.ovirt.org/#/c/48051/29/automation/check-patch.packages" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/48051/29/automation/check-patch.packages</a><br>
>><br>
> check_patch runs are done by the "vdsm_master_check-patch" job, not this one.<br>
> If this job's functionality is indeed already covered by check_patch,<br>
> then I vote for it to be removed.<br>
> VDSM maintainers, please confirm.<br>
<br>
</span>I think check-patch.sh and check-merge are enough.<br>
<br>
I don't know what are the other jobs and what they are doing.<br>
Better remove them since nobody care about them anyway.<br>
<br>
If they do something valuable, they should be integrated with<br>
the standard automation scripts.<br></blockquote><div> </div><div>ack</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
Nir<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><span style="font-size:12.8px"><b>Yaniv Bronhaim.</b></span><br></div></div></div></div></div>
</div></div>