Ravi Nori has uploaded a new change for review.

Change subject: cli: collection-based-options could be passed in 2 ways(#859684)
......................................................................

cli: collection-based-options could be passed in 2 ways(#859684)

Collection based options can be passwd without the xml type prefix.
Both boot.dev=hd and dev=hd as value for --os-boot work. Only the first
one bood.dev=hd is a valid value for the option --os-boot.

Change-Id: I1c8d29264727c3b1d9e786514cd6f11e58030b5f
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=859684
Signed-off-by: Ravi Nori <[email protected]>
---
M src/ovirtcli/command/command.py
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine-cli refs/changes/23/12323/1

diff --git a/src/ovirtcli/command/command.py b/src/ovirtcli/command/command.py
index 02a4534..ce810f7 100644
--- a/src/ovirtcli/command/command.py
+++ b/src/ovirtcli/command/command.py
@@ -119,6 +119,9 @@
 
                         for param in val.split(','):
                             obj_params_set_cand = root_obj_params_set_cand
+                            if not param.startswith(props[i] + '.'):
+                               
self.error(Messages.Error.INVALID_OPTION_SEGMENT % \
+                                                       (param, prop))
                             param_data = param.replace(props[i] + '.', 
'').split('=')
                             if len(param_data) == 2:
                                 spplited_param_data = param_data[0].split('.')


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c8d29264727c3b1d9e786514cd6f11e58030b5f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to