[node-patches] Change in ovirt-node[master]: Add return value when persist command does not return an err...

fabiand at fedoraproject.org fabiand at fedoraproject.org
Thu Mar 13 08:19:06 UTC 2014


Fabian Deutsch has submitted this change and it was merged.

Change subject: Add return value when persist command does not return an error code on failure
......................................................................


Add return value when persist command does not return an error code on failure

Create a new python function "ovirt_store_config_retnum" to have it return
different error number consist with the different error case
In the shell script ovirt-functions.in, the "ovirt_store_config"
function will invoke the new python function "ovirt_store_config_retnum"
The ovirt_store_config will judge the error number from the ovirt_store_config_retnum

Change-Id: I014c41c9919cc9f8c260d85e0ef1a23ad17021a2
Signed-off-by: Changming Bai <baichm at linux.vnet.ibm.com>
Signed-off-by: lizhihui <zhihuili at linux.vnet.ibm.com>
Signed-off-by: MeiLiu <liumbj at linux.vnet.ibm.com>
---
M scripts/ovirt-functions.in
M scripts/ovirt-init-functions.sh.in
M src/ovirtnode/ovirtfunctions.py
3 files changed, 70 insertions(+), 67 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved
  Wenyi Gao: Looks good to me, but someone else must approve
  Mei Liu: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I014c41c9919cc9f8c260d85e0ef1a23ad17021a2
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: zhihui li <liyuran9522 at gmail.com>
Gerrit-Reviewer: Changming Bai <baichm at linux.vnet.ibm.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Mei Liu <liumbj at linux.vnet.ibm.com>
Gerrit-Reviewer: Wenyi Gao <wenyi at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list