Douglas Schilling Landgraf has posted comments on this change.
Change subject: forceVMstart: Initial commit ......................................................................
Patch Set 3: (2 inline comments)
.................................................... File contrib/forceVMstart.py Line 38: sys.exit(1) Line 39: Line 40: Line 41: # Adding vdsm .pyc libraries to python path Line 42: sys.path.append("/usr/share/vdsm") we have moved vdscli to python sitepackage, no need this anymore. Line 43: Line 44: try: Line 45: import vdscli Line 46: except:
Line 41: # Adding vdsm .pyc libraries to python path Line 42: sys.path.append("/usr/share/vdsm") Line 43: Line 44: try: Line 45: import vdscli should be replaced with: from vdsm import vdscli Line 46: except: Line 47: print "Cannot import vdscli, please fix it" Line 48: sys.exit(1) Line 49:
-- 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@redhat.com Gerrit-Reviewer: Amador Pahim apahim@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Lee Yarwood lyarwood@redhat.com Gerrit-Reviewer: Pablo Iranzo Gómez Pablo.Iranzo@redhat.com Gerrit-Reviewer: Shu Ming shuming@linux.vnet.ibm.com Gerrit-Reviewer: Yaniv Bronhaim ybronhei@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server