June 2014 - Engine-commits - oVirt List Archives

Change in ovirt-host-deploy[ovirt-host-deploy-1.2]: packaging: post ovirt-host-deploy-1.2.2
by Alon Bar-Lev 22 Jun '14

22 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post ovirt-host-deploy-1.2.2 ...................................................................... packaging: post ovirt-host-deploy-1.2.2 Change-Id: I54e255ed9fc517ca38e0c8e3dbccb1a3400c88fa Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29027 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54e255ed9fc517ca38e0c8e3dbccb1a3400c88fa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.2 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.2]: packaging: ovirt-host-deploy-1.2.2
by Alon Bar-Lev 22 Jun '14

22 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: ovirt-host-deploy-1.2.2 ...................................................................... packaging: ovirt-host-deploy-1.2.2 Change-Id: I27bf60e77c5f42aa67e2d9af1f82a0783b3777ca Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M ovirt-host-deploy-offline.spec.in M ovirt-host-deploy.spec.in 4 files changed, 9 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29026 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27bf60e77c5f42aa67e2d9af1f82a0783b3777ca Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.2 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: restapi: IO exception mapper
by oourfali＠redhat.com 21 Jun '14

21 Jun '14

Oved Ourfali has submitted this change and it was merged. Change subject: restapi: IO exception mapper ...................................................................... restapi: IO exception mapper This patch introduces a new message body reader that generates always an IO exception, containing the cause of the error, and a generic exception mapper that takes that IO exception, extracts the cause and generates the corresponding message. Change-Id: Id63e6eee673723ac32e3c2740f338f7ca465c6ac Bug-Url: https://bugzilla.redhat.com/1110295 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit a47e95e9d5eed10a57ef38963450421ccd32a07f) --- M backend/manager/modules/restapi/interface/definition/pom.xml A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/xml/JAXBMessageBodyReader.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/xml/JAXBValidationEventHandler.java M backend/manager/modules/restapi/interface/definition/src/main/resources/META-INF/services/javax.ws.rs.ext.Providers M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/IOExceptionMapper.java D backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/InvalidValueExceptionMapper.java D backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/XmlMessageBodyReader.java M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml 9 files changed, 264 insertions(+), 99 deletions(-) Approvals: Ravi Nori: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28847 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id63e6eee673723ac32e3c2740f338f7ca465c6ac Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: Fix missing macro
by gpadgett＠redhat.com 20 Jun '14

20 Jun '14

Greg Padgett has submitted this change and it was merged. Change subject: Fix missing macro ...................................................................... Fix missing macro %systemd_requires fails on f19 when systemd is not installed. Change-Id: I5ab60bd58233b4776482f2a7aee609d6a823bffd Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, but someone else must approve Greg Padgett: Looks good to me, approved David Caro: Verified -- To view, visit http://gerrit.ovirt.org/28978 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ab60bd58233b4776482f2a7aee609d6a823bffd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: ovirt-node-plugin-vdsm: Bump release to 0.1.2-4
by dougsland＠redhat.com 20 Jun '14

20 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: ovirt-node-plugin-vdsm: Bump release to 0.1.2-4 ...................................................................... ovirt-node-plugin-vdsm: Bump release to 0.1.2-4 - hooks: Add hook for ovirt-node boot Change-Id: I89a82da7480c25ef3a0abf38ea8984f8b4b1d666 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29002 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89a82da7480c25ef3a0abf38ea8984f8b4b1d666 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: fix references to vdcoptions
by sbonazzo＠redhat.com 20 Jun '14

20 Jun '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix references to vdcoptions ...................................................................... packaging: setup: fix references to vdcoptions Fixed references to vdcoptions introduced in commit I4d465766f10a490bd213ca3510d02b3d6fdfa7c6 Making vdcoption acepting statements wich own connection Fixing an issue calling the logger Change-Id: Ibdc415127f5d6467962d2b63afdf9a78df770f99 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine/vdcoption.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/upgrade/asynctasks.py 2 files changed, 29 insertions(+), 19 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Ravi Nori: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28908 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibdc415127f5d6467962d2b63afdf9a78df770f99 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: Fixing websocket proxy key extraction
by sbonazzo＠redhat.com 20 Jun '14

20 Jun '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Fixing websocket proxy key extraction ...................................................................... packaging: setup: Fixing websocket proxy key extraction As for other keys, extract websocket proxy key out of a transaction so it would be already available before the websocket proxy misc plugin. Change-Id: Id8a608383252b50a4d76f73b0bbbe8c3f6c9a692 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py 1 file changed, 11 insertions(+), 18 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28886 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8a608383252b50a4d76f73b0bbbe8c3f6c9a692 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Added vdsm exceptions related to libgfapi
by kmayilsa＠redhat.com 20 Jun '14

20 Jun '14

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Added vdsm exceptions related to libgfapi ...................................................................... engine: Added vdsm exceptions related to libgfapi Added error codes for vdsm exceptions related to libgapi Change-Id: I48c70d5067e7f6e3bfc5007f9fd63fbd0b7d3229 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/AbstractGlusterBrokerCommand.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/VdsmErrors.java 4 files changed, 25 insertions(+), 1 deletion(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/28632 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I48c70d5067e7f6e3bfc5007f9fd63fbd0b7d3229 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine : if setSucceeded(true) is called before exception co...
by oourfali＠redhat.com 20 Jun '14

20 Jun '14

Oved Ourfali has submitted this change and it was merged. Change subject: engine : if setSucceeded(true) is called before exception command is presented as succeeded ...................................................................... engine : if setSucceeded(true) is called before exception command is presented as succeeded if any Command calls: getReturnValue().setSucceeded(true); and after that follows some logic, then if this logic fails, outcome of this command will be presented as successful. Compensation/rollbacks etc. are called, but data sent to user is wrong. Change-Id: Iee7b186314dacdeeb818c6fdf5157876b0b8342e Bug-Url: https://bugzilla.redhat.com/1106464 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28916 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee7b186314dacdeeb818c6fdf5157876b0b8342e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.1]: packaging: setup: write answer file also on error
by sbonazzo＠redhat.com 20 Jun '14

20 Jun '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: write answer file also on error ...................................................................... packaging: setup: write answer file also on error previously answer file was not written if an error occurred at customization or if user did not accept configuration preview. Also the answer file was not updated with late values asked at closeup stage if an error occurred in that stage. Now the answer file is always written. Change-Id: Icf991ffaa44fbb16c7a97c76b9a4e17397aacaca Bug-Url: https://bugzilla.redhat.com/1111506 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/core/answerfile.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/28901 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icf991ffaa44fbb16c7a97c76b9a4e17397aacaca Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.1 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0