Change in vdsm[master]: testValidateBondingOptions: declare root requirement
danken at redhat.com
danken at redhat.com
Fri Oct 18 09:35:41 UTC 2013
Dan Kenigsberg has uploaded a new change for review.
Change subject: testValidateBondingOptions: declare root requirement
......................................................................
testValidateBondingOptions: declare root requirement
Our Bond.validateOptions() and and its test require root permissions.
Change-Id: I43743e2446d014ae1e4e97b84756dc403019c455
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/netmodelsTests.py
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/84/20284/1
diff --git a/tests/netmodelsTests.py b/tests/netmodelsTests.py
index b5b38c3..6181a9e 100644
--- a/tests/netmodelsTests.py
+++ b/tests/netmodelsTests.py
@@ -31,6 +31,7 @@
import neterrors
from testrunner import VdsmTestCase as TestCaseBase
+from testValidation import ValidateRunningAsRoot
from nose.plugins.skip import SkipTest
from monkeypatch import MonkeyPatch
@@ -87,6 +88,7 @@
self.assertEqual(Bond.validateName('bond11'), None)
self.assertEqual(Bond.validateName('bond11128421982'), None)
+ @ValidateRunningAsRoot
def testValidateBondingOptions(self):
if not os.path.exists(netinfo.BONDING_MASTERS):
raise SkipTest("bonding kernel module could not be found.")
--
To view, visit http://gerrit.ovirt.org/20284
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I43743e2446d014ae1e4e97b84756dc403019c455
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
More information about the vdsm-patches
mailing list