----- Original Message ----- > From: "Mooli Tayer" <mtayer(a)redhat.com&gt; > To: devel(a)ovirt.org > Sent: Monday, October 27, 2014 11:16:41 AM > Subject: [ovirt-devel] vdsm make rpm & make check on fe vs el > > Isn't this odd? > > Fedora release 20 (Heisenbug) > ============================= > $ make check 2>&1 | grep -E "^Ran" > Ran 2 tests in 1.006s > Ran 1 test in 0.070s > Ran 691 tests in 63.504s > $ make rpm 2>&1 | grep -E "^Ran" | tail -f > Ran 2 tests in 1.005s > Ran 1 test in 0.069s > Ran 691 tests in 63.937s > $ > > > > Red Hat Enterprise Linux Server release 6.6 (Santiago) > ====================================================== > > $ make check 2>&1 | grep -E "^Ran" > Ran 2 tests in 1.003s > Ran 1 test in 0.082s > Ran 691 tests in 61.114s > $ make rpm 2>&1 | grep -E "^Ran" > $ > > I've also saw a different numbers of tests running for make rpm on el. I have seen this on several 6.5 machines - make rpm on does *not* run the tests. It would be nice if take a look at this.

From: "Nir Soffer" <nsoffer(a)redhat.com&gt; To: "Francesco Romani" <fromani(a)redhat.com&gt; Cc: devel(a)ovirt.org, "Mooli Tayer" <mtayer(a)redhat.com&gt; Sent: Monday, October 27, 2014 12:16:11 PM Subject: Re: [ovirt-devel] vdsm make rpm & make check on fe vs el

> > > I've also saw a different numbers of tests running for make rpm on el. > > > > I have seen this on several 6.5 machines - make rpm on does *not* run > > the tests. > > > > It would be nice if take a look at this. > > It was made on purposes: > > # Skips check since rhel default repos lack pep8 and pyflakes > %if ! 0%{?rhel} > %global with_check 1 > %endif > > http://gerrit.ovirt.org/#/c/29213/ > > I believe time has come for a more robust solution As first aid, we should log something like "Skipping tests because ..." in this case. I think we need to change that so we disable only pep8 in make rpm. Nir

From: "Antoni Segura Puimedon" <asegurap(a)redhat.com&gt; To: "Francesco Romani" <fromani(a)redhat.com&gt; Cc: devel(a)ovirt.org Sent: Monday, October 27, 2014 1:26:47 PM Subject: Re: [ovirt-devel] vdsm make rpm & make check on fe vs el ----- Original Message ----- > From: "Francesco Romani" <fromani(a)redhat.com&gt; > To: devel(a)ovirt.org > Sent: Monday, October 27, 2014 12:20:17 PM > Subject: Re: [ovirt-devel] vdsm make rpm & make check on fe vs el > > ----- Original Message ----- > > From: "Nir Soffer" <nsoffer(a)redhat.com&gt; > > To: "Francesco Romani" <fromani(a)redhat.com&gt; > > Cc: devel(a)ovirt.org, "Mooli Tayer" <mtayer(a)redhat.com&gt; > > Sent: Monday, October 27, 2014 12:16:11 PM > > Subject: Re: [ovirt-devel] vdsm make rpm & make check on fe vs el > > > [...] > > > > > I've also saw a different numbers of tests running for make rpm on > > > > > el. > > > > > > > > I have seen this on several 6.5 machines - make rpm on does *not* run > > > > the tests. > > > > > > > > It would be nice if take a look at this. > > > > > > It was made on purposes: > > > > > > # Skips check since rhel default repos lack pep8 and pyflakes > > > %if ! 0%{?rhel} > > > %global with_check 1 > > > %endif > > > > > > http://gerrit.ovirt.org/#/c/29213/ > > > > > > I believe time has come for a more robust solution > > > > As first aid, we should log something like "Skipping tests because ..." > > in this case. > > > > I think we need to change that so we disable only pep8 in make rpm. > > > > Nir > > +1 > > we can afford to skip local checks (pep8/pyflakes), but the core test suite > should run > anyway. > > I'll post a patch today(ish) if noone is faster/disagrees with the above :) I don't think the tests should run on make rpm. In make check, sure. In make rpm... It's a last resort for making people realize something is broken when they are not running make check.