On Sun, Dec 20, 2015 at 9:46 AM, Nir Soffer <[email protected]> wrote:
> On Sun, Dec 20, 2015 at 8:49 AM, Barak Korren <[email protected]> wrote: > > On 19 December 2015 at 13:05, Yaniv Bronheim <[email protected]> > 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. > ack > Nir > -- *Yaniv Bronhaim.*
_______________________________________________ Devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/devel
