Change in vdsm[master]: tests: temporarily exclude few tests to restore CI
danken at redhat.com
danken at redhat.com
Mon Aug 24 12:35:06 UTC 2015
Dan Kenigsberg has submitted this change and it was merged.
Change subject: tests: temporarily exclude few tests to restore CI
......................................................................
tests: temporarily exclude few tests to restore CI
A few network tests require the TUN device, which is not available
under mock environments.
Furthermore, when run under the new CI infrastructure which leverages mock,
we can see this error:
======================================================================
ERROR: testGetBondingOptions (netinfoTests.TestNetinfo)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/tmp/run/tests/monkeypatch.py", line 133, in wrapper
return f(*args, **kw)
File "/tmp/run/tests/testValidation.py", line 105, in wrapper
return f(*args, **kwargs)
File "/tmp/run/tests/testValidation.py", line 145, in wrapper
return f(*args, **kwargs)
File "/tmp/run/tests/netinfoTests.py", line 290, in
testGetBondingOptions
with open(netinfo.BONDING_MASTERS, 'w') as bonds:
IOError: [Errno 13] Permission denied: '/sys/class/net/bonding_masters'
-------------------- >> begin captured logging << --------------------
root: DEBUG: /sbin/modprobe bonding (cwd None)
root: DEBUG: FAILED: <err> = ''; <rc> = 1
--------------------- >> end captured logging << ---------------------
----------------------------------------------------------------------
This happens if the system cannot load modules inside the chrooted
mock environments.
We temporarily NOSE_EXCLUDE away the problematic jobs in order to
restore CI automated tests. We will restore those tests ASAP once
fixed to run properly in mocked environments.
Change-Id: I1721a0e2fccb9f8fff216f866843fa1daf192185
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45134
Reviewed-by: Ondřej Svoboda <osvoboda at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Dan Kenigsberg <danken at redhat.com>
---
M automation/check-patch.sh
1 file changed, 3 insertions(+), 9 deletions(-)
Approvals:
Ondřej Svoboda: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved; Passed CI tests
Francesco Romani: Verified
--
To view, visit https://gerrit.ovirt.org/45134
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1721a0e2fccb9f8fff216f866843fa1daf192185
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the vdsm-patches
mailing list