Change in ovirt-engine-cli[cli_3.6]: cli: Try to locate objects using id, name and alias

juan.hernandez at redhat.com juan.hernandez at redhat.com
Fri Jul 15 08:37:58 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>
(cherry picked from commit 184ab531adb4c7ef994004563b3d74e59435d5d9)
---
M src/ovirtcli/command/command.py
1 file changed, 30 insertions(+), 24 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/60776
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: cli_3.6
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list