Change in ovirt-engine-cli[master]: cli: Try id, name and alias for show

juan.hernandez at redhat.com juan.hernandez at redhat.com
Sat Jan 24 14:11:38 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: cli: Try id, name and alias for show
......................................................................


cli: Try id, name and alias for show

The fix for bug 1148941 (commit 4b654a2) introduced a regression in the
way that the "show" command locates objects, so that objects can't be
located by name. For example, the following command will fail to locate
the a virtual machine, even if it exists:

  # show vm myvm

This patch fixes that regression, so that when given an object
identifier the CLI will try to locate it first by identifier, then by
name and finally by alias (for disks).

Change-Id: I8d197c042e83939470aea93405546a25bdf3f0c0
Related-To: https://bugzilla.redhat.com/1148941
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M src/ovirtcli/command/command.py
1 file changed, 9 insertions(+), 3 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d197c042e83939470aea93405546a25bdf3f0c0
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: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list