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

fabiand at redhat.com fabiand at redhat.com
Tue Sep 9 13:36:14 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
  Douglas Schilling Landgraf: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I014c41c9919cc9f8c260d85e0ef1a23ad17021a2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: node-3.0
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Changming Bai <baichm at linux.vnet.ibm.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Mei Liu <liumbj at linux.vnet.ibm.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: zhihui li <liyuran9522 at gmail.com>



More information about the node-patches mailing list