Hello Ravi Nori,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/13392

to review the following change.

Change subject: cli : --vm-os-boot doesn't send the order of devices
......................................................................

cli : --vm-os-boot doesn't send the order of devices

When starting vm with parameter --vm-os-boot
"boot.dev=hd,boot.dev=network" only network device is sent

Change-Id: I681b28926fad32cfe36472a3940df7021bfcbe6b
Bug-Url: https://bugzilla.redhat.com/921464
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M src/ovirtcli/command/command.py
1 file changed, 4 insertions(+), 0 deletions(-)


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

diff --git a/src/ovirtcli/command/command.py b/src/ovirtcli/command/command.py
index 1ae497a..3e0fd90 100644
--- a/src/ovirtcli/command/command.py
+++ b/src/ovirtcli/command/command.py
@@ -125,6 +125,10 @@
                                 for param_period in spplited_param_data:
                                     if spplited_param_data[-1] == param_period:
                                         if hasattr(obj_params_set_cand, 
param_period):
+                                            if getattr(obj_params_set_cand, 
param_period) != None:
+                                                getattr(obj, 
props[i]).append(obj_params_set_cand)
+                                                obj_params_set_cand = 
params_set_cand.factory()
+                                                root_obj_params_set_cand = 
obj_params_set_cand
                                             setattr(obj_params_set_cand, 
param_period,
                                                     
self.__try_parse(param_data[1].strip()))
                                         else:


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I681b28926fad32cfe36472a3940df7021bfcbe6b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: cli_3.2
Gerrit-Owner: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to