Antoni Segura Puimedon has posted comments on this change.

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


Patch Set 1:

(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: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to