Martin Betak has posted comments on this change.

Change subject: cli: Enable update of VM with integer name
......................................................................


Patch Set 1:

(1 comment)

....................................................
File src/ovirtcli/command/command.py
Line 360
Line 361
Line 362
Line 363
Line 364
I purposely removed only this check at line 362 and not the one on line 365 so 
the user would have to explicitly quote the name as in '12345'. When I tried 
entering 12345 the type of candidate was  'int' so I assumed the command line 
parsing provided arguments already converted. Is this behavior still incorrect?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c54b646f057d2f39b85ab0ca6c448308c06ecce
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <[email protected]>
Gerrit-Reviewer: Martin Betak <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to