Pablo Iranzo Gómez has posted comments on this change.
Change subject: forceVMstart: Initial commit
......................................................................
Patch Set 3: (1 inline comment)
....................................................
File contrib/forceVMstart.py
Line 89: return 0
Line 90:
Line 91: def checkRoot(self):
Line 92: """check if the user running the script is
root"""
Line 93: if os.geteuid() != 0:
Regaring the assert, I'm trying the code on my computer and:
import os
assert(os.geteuid() == 0,'sudo it')
gives:
[iranzo@x201 ~]$ python prueba.py
prueba.py:2: SyntaxWarning: assertion is always true, perhaps remove parentheses?
assert(os.geteuid() == 0,'sudo it')
¿Is something I'm doing wrong?
Thanks!
Line 94: print "You must be root to run this script."
Line 95: sys.exit(2)
Line 96:
Line 97: def getIpManagementIP(self):
--
To view, visit
http://gerrit.ovirt.org/9473
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a70b31ce0730194880406701316f219c9f92ceb
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Pablo Iranzo Gómez <Pablo.Iranzo(a)redhat.com>
Gerrit-Reviewer: Amador Pahim <apahim(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com>
Gerrit-Reviewer: Pablo Iranzo Gómez <Pablo.Iranzo(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server