Michael Pasternak 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 yes it is, this code exist for backward compatibility, before i wrote command line parsing parameter type recognition, users always had to pass quoted UUIDs, integers, longs, etc. so this code makes sure if someone wrote a script in old format, it will continue to run for him on new cli. to workaround ambiguity mentioned in this BZ, users have to use id only when the name of the entity is digits, thanks. -- 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
