Change in vdsm[master]: vdsm_hooks: Add support for checkimages VDSM hook

danken at redhat.com danken at redhat.com
Sun Nov 18 14:25:07 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: vdsm_hooks: Add support for checkimages VDSM hook
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm_hooks/checkimages/before_vm_start.py
Line 45:             sys.exit(2)
Line 46: 
Line 47:         out, err = p.communicate()
Line 48:         rc = p.returncode
Line 49:             
oh, there's whitespace problems here.
Line 50:     if rc == 0:
Line 51:         sys.stderr.write('checkimages: Checking image %s returned: %s\n' % \
Line 52:                          (path, out))
Line 53:     else:


--
To view, visit http://gerrit.ovirt.org/9154
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib4d88fb46a725d2c525d10ce2fcc808bba5b0ab4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Jan Lieskovsky <jlieskov at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list