Change in vdsm[master]: BZ#849315 vdsm-reg: return proper exit code from upgrade scr...

2012-08-19 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: BZ#849315 vdsm-reg: return proper exit code from upgrade script
..


Patch Set 2: Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4d0c3d60c3d80ea50ede6b22f931b091ec625836
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Michael Burns mbu...@redhat.com
Gerrit-Reviewer: Yaniv Kaul yk...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: BZ#849315 vdsm-reg: return proper exit code from upgrade scr...

2012-08-19 Thread danken
Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#849315 vdsm-reg: return proper exit code from upgrade script
..


BZ#849315 vdsm-reg: return proper exit code from upgrade script

main returns boolean, while exit code of True is actually program
status of failure.

Change-Id: I4d0c3d60c3d80ea50ede6b22f931b091ec625836
Signed-off-by: Alon Bar-Lev alo...@redhat.com
---
M vdsm_reg/vdsm-upgrade
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alon Bar-Lev: Verified
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d0c3d60c3d80ea50ede6b22f931b091ec625836
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Michael Burns mbu...@redhat.com
Gerrit-Reviewer: Yaniv Kaul yk...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: BZ#849315 vdsm-reg: return proper exit code from upgrade scr...

2012-08-18 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: BZ#849315 vdsm-reg: return proper exit code from upgrade script
..


Patch Set 2:

Build Successful 

http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/505/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4d0c3d60c3d80ea50ede6b22f931b091ec625836
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Michael Burns mbu...@redhat.com
Gerrit-Reviewer: Yaniv Kaul yk...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: BZ#849315 vdsm-reg: return proper exit code from upgrade scr...

2012-08-18 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change.

Change subject: BZ#849315 vdsm-reg: return proper exit code from upgrade script
..


Patch Set 2: Verified

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4d0c3d60c3d80ea50ede6b22f931b091ec625836
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf dougsl...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Michael Burns mbu...@redhat.com
Gerrit-Reviewer: Yaniv Kaul yk...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches