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

juan.hernandez at redhat.com juan.hernandez at redhat.com
Mon Aug 22 14:42:54 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/62676
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: 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