Change in vdsm[master]: Convert vm status strings to constants

asegurap at redhat.com asegurap at redhat.com
Wed Nov 27 14:44:55 UTC 2013


Antoni Segura Puimedon has posted comments on this change.

Change subject: Convert vm status strings to constants
......................................................................


Patch Set 2:

(1 comment)

....................................................
File vdsm/API.py
Line 473:         except KeyError:
Line 474:             return errCode['noVM']
Line 475: 
Line 476:         vmParams = v.status()
Line 477:         if vmParams['status'] in (vmstatus.WAIT_FOR_LAUNCHF, vmstatus.DOWN):
s/LAUNCHF/LAUNCH/
Line 478:             return errCode['noVM']
Line 479:         if params.get('mode') == 'file':
Line 480:             if 'dst' not in params:
Line 481:                 params['dst'], params['dstparams'] = \


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I95e6945c2ee9d443e22586f04dc0f9646cc5cbb3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list