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 <[email protected]> Gerrit-Reviewer: Amador Pahim <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Lee Yarwood <[email protected]> Gerrit-Reviewer: Pablo Iranzo Gómez <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
