Change in ovirt-engine-cli[master]: cli: Ignore "id" and "name" options for the "show" command

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Aug 30 07:39:47 UTC 2016


Juan Hernandez has submitted this change and it was merged.

Change subject: cli: Ignore "id" and "name" options for the "show" command
......................................................................


cli: Ignore "id" and "name" options for the "show" command

Currently auto-completion for the "show" command includes the "id" and
"name" options. This is wrong, as these options aren't used to specify
the object to be displayed. The reason they are included in
auto-completion is that the underlying Python SDK method is "get", and
that method usually has both the "id" and "name" parameters. We simply
and blindly extract the parameters from the method and assume that they
are valid options. This patch changes the CLI so that it ignores these
parameters, thus they won't appear in the auto-completion list.

Change-Id: I068686a6c600d0aac582adc239439c5f7dba0f13
Bug-Url: https://bugzilla.redhat.com/1358227
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M src/ovirtcli/shell/showcmdshell.py
1 file changed, 1 insertion(+), 0 deletions(-)

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



-- 
To view, visit https://gerrit.ovirt.org/61095
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I068686a6c600d0aac582adc239439c5f7dba0f13
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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list