Change in ovirt-engine-cli[master]: cli: Try to locate objects using id, name and alias
juan.hernandez at redhat.com
juan.hernandez at redhat.com
Fri Jul 15 08:35:29 UTC 2016
Juan Hernandez has submitted this change and it was merged.
Change subject: cli: Try to locate objects using id, name and alias
......................................................................
cli: Try to locate objects using id, name and alias
Currently when the CLI tries to locate an object it tries to do it by
id, by name and by alias, but only if the --id, --name or --alias
options are part of the command line. This is not correct for commands
like "show", which don't require any of those options, as the identifier
of the object is passed as an argument, not as an option. The result is
that the "show" command doesn't work. To fix that issue this patch
changes the CLI so that it will always try the id, name and alias,
regardless of the options. If one try fails, for whatever the reason, it
will be ignored and the next will be used.
Change-Id: I1ef5dbefe35cc9ac82884df7d51f912ca171e776
Bug-Url: https://bugzilla.redhat.com/1356607
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M src/ovirtcli/command/command.py
1 file changed, 30 insertions(+), 24 deletions(-)
Approvals:
Ondra Machacek: Looks good to me, but someone else must approve
Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/60762
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1ef5dbefe35cc9ac82884df7d51f912ca171e776
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list