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