Change in ovirt-engine-cli[master]: cli: Generate empty list for empty option

juan.hernandez at redhat.com juan.hernandez at redhat.com
Fri Sep 12 08:29:47 UTC 2014


Juan Hernandez has submitted this change and it was merged.

Change subject: cli: Generate empty list for empty option
......................................................................


cli: Generate empty list for empty option

Currently when a command receives an option that represents a list of
values but without any value the CLI automatically adds the string
"None" as the only value. This can cause issues. What the CLI should
generate in this cases is an empty list.

Change-Id: Ic95dadcbfd7f77b49524df0ffd05dcdcf7af8f1e
Bug-Url: https://bugzilla.redhat.com/1085463
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M src/ovirtcli/command/command.py
1 file changed, 3 insertions(+), 2 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic95dadcbfd7f77b49524df0ffd05dcdcf7af8f1e
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