March 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Corrected geo-replication status sync
by masayag＠redhat.com 04 Mar '15

04 Mar '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Corrected geo-replication status sync ...................................................................... engine: Corrected geo-replication status sync Corrected issues while syncing geo-replication session w.r.t saving slave uuids and retrieving status Change-Id: If30bda56f117fedf3d42b51971201e4557f1c2cf Bug-Url: https://bugzilla.redhat.com/1195699 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterGeoRepSyncJob.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterGeoRepDaoDbFacadeImpl.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java 3 files changed, 15 insertions(+), 8 deletions(-) Approvals: Shubhendu Tripathi: Looks good to me, but someone else must approve Sahina Bose: Verified Moti Asayag: Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38224 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If30bda56f117fedf3d42b51971201e4557f1c2cf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Corrected VDS parsing for geo-rep config
by masayag＠redhat.com 04 Mar '15

04 Mar '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Corrected VDS parsing for geo-rep config ...................................................................... engine: Corrected VDS parsing for geo-rep config Corrected the output parsing for geo-rep config. Refactored the class name Change-Id: Ib293b98daf575f18182f453fab7c7d5aa78dcdc8 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeConfigListXmlRpc.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeGeoRepConfigListXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 6 files changed, 54 insertions(+), 54 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Darshan N: Looks good to me, but someone else must approve Sahina Bose: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38323 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib293b98daf575f18182f453fab7c7d5aa78dcdc8 Gerrit-PatchSet: 2 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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Adding branches support in the ovirt node prm builders
by tlitovsk＠redhat.com 04 Mar '15

04 Mar '15

Anatoly Litovsky has submitted this change and it was merged. Change subject: Adding branches support in the ovirt node prm builders ...................................................................... Adding branches support in the ovirt node prm builders Change-Id: I8e5bbf563b3fac0b1734f5b35db65d739181d7cf Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-node/ovirt-node-job.yaml M jobs/confs/yaml/scms/ovirt-node-scms.yaml M jobs/confs/yaml/templates/ovirt-node-job-templates.yaml 3 files changed, 11 insertions(+), 5 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Anatoly Litovsky: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38332 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e5bbf563b3fac0b1734f5b35db65d739181d7cf Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Max Kovgan <kovganm(a)gmail.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Creating the ovirt node rpm builder
by tlitovsk＠redhat.com 04 Mar '15

04 Mar '15

Anatoly Litovsky has submitted this change and it was merged. Change subject: Creating the ovirt node rpm builder ...................................................................... Creating the ovirt node rpm builder The patch splits the iso and node creation scripts. It also adds configuration params allowing to reuse the same build code in multiuple builders Change-Id: I3d303c3dfd9a285295af0e33a631fa8035e5f997 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- A jobs/confs/shell-scripts/build-local-ovirt-node-iso.sh M jobs/confs/shell-scripts/build-local-ovirt-node.sh M jobs/confs/yaml/builders/ovirt-node-builders.yaml M jobs/confs/yaml/jobs/ovirt-node/ovirt-node-job.yaml M jobs/confs/yaml/templates/ovirt-node-job-templates.yaml 5 files changed, 162 insertions(+), 48 deletions(-) Approvals: Max Kovgan: Looks good to me, but someone else must approve Anatoly Litovsky: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38317 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d303c3dfd9a285295af0e33a631fa8035e5f997 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Max Kovgan <kovganm(a)gmail.com> Gerrit-Reviewer: Max Kovgan <mkovgan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Added compat check for gluster task actions
by masayag＠redhat.com 03 Mar '15

03 Mar '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Added compat check for gluster task actions ...................................................................... engine: Added compat check for gluster task actions Added cluster compatibility checks for actions that depend on gluster task management support in vdsm Change-Id: I70ee8230253b04141133ccb639e00531086fd5bd Bug-Url: https://bugzilla.redhat.com/1164798 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterAsyncCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/AbstractRemoveGlusterVolumeBricksCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/CommitRemoveGlusterVolumeBricksCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/StartRebalanceGlusterVolumeCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/StartRemoveGlusterVolumeBricksCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/StopRebalanceGlusterVolumeCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/StopRemoveGlusterVolumeBricksCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 11 files changed, 141 insertions(+), 12 deletions(-) Approvals: Sahina Bose: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/36986 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70ee8230253b04141133ccb639e00531086fd5bd Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(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: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: json-rpc: fix the Host.getVMList return value
by oourfali＠redhat.com 03 Mar '15

03 Mar '15

Oved Ourfali has submitted this change and it was merged. Change subject: json-rpc: fix the Host.getVMList return value ...................................................................... json-rpc: fix the Host.getVMList return value Hack response type processing to accept both map with status and vmid and an array of vmids. Bug-Url: https://bugzilla.redhat.com/1196327 Change-Id: Iade8c1d5954eaae4339e95d9d5031a7b98cf831b Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Related-To: https://gerrit.ovirt.org/#/c/38172 --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VMListReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/jsonrpc/MarshallingTestCase.java 3 files changed, 39 insertions(+), 4 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38321 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iade8c1d5954eaae4339e95d9d5031a7b98cf831b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5.2]: aaa: Missing namespace and principal in RSDL (users.add)
by oourfali＠redhat.com 03 Mar '15

03 Mar '15

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: Missing namespace and principal in RSDL (users.add) ...................................................................... aaa: Missing namespace and principal in RSDL (users.add) Change-Id: I3ac8f4f22132e37d58bae65b8d7fafd4ec2fd121 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1188176 Topic: AAA Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38320 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ac8f4f22132e37d58bae65b8d7fafd4ec2fd121 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5.2 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: aaa: Missing namespace and principal in RSDL (users.add)
by oourfali＠redhat.com 03 Mar '15

03 Mar '15

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: Missing namespace and principal in RSDL (users.add) ...................................................................... aaa: Missing namespace and principal in RSDL (users.add) Change-Id: I3ac8f4f22132e37d58bae65b8d7fafd4ec2fd121 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1188176 Topic: AAA Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38319 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ac8f4f22132e37d58bae65b8d7fafd4ec2fd121 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5.2]: json-rpc: fix the Host.getVMList return value
by oourfali＠redhat.com 03 Mar '15

03 Mar '15

Oved Ourfali has submitted this change and it was merged. Change subject: json-rpc: fix the Host.getVMList return value ...................................................................... json-rpc: fix the Host.getVMList return value Hack response type processing to accept both map with status and vmid and an array of vmids. Bug-Url: https://bugzilla.redhat.com/1196327 Change-Id: Iade8c1d5954eaae4339e95d9d5031a7b98cf831b Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Related-To: https://gerrit.ovirt.org/#/c/38172 --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VMListReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/jsonrpc/MarshallingTestCase.java 3 files changed, 39 insertions(+), 4 deletions(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38304 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iade8c1d5954eaae4339e95d9d5031a7b98cf831b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5.2 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: json-rpc: fix the Host.getVMList return value
by oourfali＠redhat.com 03 Mar '15

03 Mar '15

Oved Ourfali has submitted this change and it was merged. Change subject: json-rpc: fix the Host.getVMList return value ...................................................................... json-rpc: fix the Host.getVMList return value Hack response type processing to accept both map with status and vmid and an array of vmids. Bug-Url: https://bugzilla.redhat.com/1196327 Change-Id: Iade8c1d5954eaae4339e95d9d5031a7b98cf831b Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Related-To: https://gerrit.ovirt.org/#/c/38172 --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VMListReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/jsonrpc/MarshallingTestCase.java 3 files changed, 39 insertions(+), 4 deletions(-) Approvals: Michal Skrivanek: Verified; Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38271 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iade8c1d5954eaae4339e95d9d5031a7b98cf831b Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0