On Sun, Dec 20, 2015 at 8:49 AM, Barak Korren <bkorren@redhat.com> wrote:
> On 19 December 2015 at 13:05, Yaniv Bronheim <ybronhei@redhat.com> wrote:
>> for some runs it is installed - we added it to the requirements for
>> check-patch -
>> https://gerrit.ovirt.org/#/c/48051/29/automation/check-patch.packages
>>
> check_patch runs are done by the "vdsm_master_check-patch" job, not this one.
> If this job's functionality is indeed already covered by check_patch,
> then I vote for it to be removed.
> VDSM maintainers, please confirm.
I think check-patch.sh and check-merge are enough.
I don't know what are the other jobs and what they are doing.
Better remove them since nobody care about them anyway.
If they do something valuable, they should be integrated with
the standard automation scripts.
Nir