June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core[BLL]: Adding external status to command
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core[BLL]: Adding external status to command ...................................................................... core[BLL]: Adding external status to command This patch handles the required changes in the AddExternalEventCommand Change-Id: I271aa3f841a3820482d89a1675e28dcb75540277 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddExternalEventCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddExternalEventParameters.java 2 files changed, 38 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Jenkins CI: Verified -- To view, visit https://gerrit.ovirt.org/40995 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I271aa3f841a3820482d89a1675e28dcb75540277 Gerrit-PatchSet: 22 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: move InstanceType remove from collection to entity
by juan.hernandez＠redhat.com 10 Jun '15

10 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: move InstanceType remove from collection to entity ...................................................................... restapi: move InstanceType remove from collection to entity This patch moves the method that implements the DELETE operation from the collection interface to the entity interface. This is needed to avoid issues with newer versions of Resteasy. Change-Id: Iadcf27decc9f50942bf6b7567eacbc265954fd6d Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/InstanceTypeResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/InstanceTypesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendInstanceTypeResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendInstanceTypesResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendInstanceTypeResourceTest.java 5 files changed, 36 insertions(+), 11 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42081 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iadcf27decc9f50942bf6b7567eacbc265954fd6d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core[DAO]: Adding external status to DAOs
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core[DAO]: Adding external status to DAOs ...................................................................... core[DAO]: Adding external status to DAOs This patch adds external status handling to the DAOs I added here also an interface for external status aware DAOs in t6he same manner it was done for status Change-Id: Ie4dd9a1bc0aa99604f74a07155a7b596585e3ccf Signed-off-by: emesika <emesika(a)redhat.com> --- A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/ExternalStatusAwareDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDynamicDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDynamicDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDynamicDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml 6 files changed, 58 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Jenkins CI: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40994 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4dd9a1bc0aa99604f74a07155a7b596585e3ccf Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core[BE]: Adding external status to BEs
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core[BE]: Adding external status to BEs ...................................................................... core[BE]: Adding external status to BEs This patch adds the new field to the VDS and VDSDynamic BE Change-Id: Id7d4804a2909b10ec278cf8564e2c9b1993bb022 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsDynamic.java 2 files changed, 20 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified Jenkins CI: Verified Moti Asayag: Looks good to me, approved Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40993 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id7d4804a2909b10ec278cf8564e2c9b1993bb022 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core[Enum]: Adding external status enum
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core[Enum]: Adding external status enum ...................................................................... core[Enum]: Adding external status enum This patch adds a new enum: ExternalStatus Change-Id: Ia46af5206938025ad3c487cc1998b32301783cbf Signed-off-by: emesika <emesika(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ExternalStatus.java 1 file changed, 34 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40992 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia46af5206938025ad3c487cc1998b32301783cbf Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core[DB]: Adding external status to DB
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core[DB]: Adding external status to DB ...................................................................... core[DB]: Adding external status to DB This patch set adds external status for the host entity The Health Status field will be returned as part of the retrieved entity when a call to display the entity is done using the REST API. The main use-case for this new status is to provide plugins / external systems the ability to trigger issues, and allow the administrator to clearly see there is an issue through the UI For more information please see the feature page wiki http://www.ovirt.org/Features/EntityHealthStatus Change-Id: Ia61e8bef40bdb5d575b75ea1433a7a00e19c9aae Signed-off-by: emesika <emesika(a)redhat.com> --- M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/03_06_1510_add_host_external_status.sql M packaging/dbscripts/vds_sp.sql 3 files changed, 17 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40991 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia61e8bef40bdb5d575b75ea1433a7a00e19c9aae Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: Remove UpdatePartialVdsDynamicCalc stored procedure
by rgolan＠redhat.com 10 Jun '15

10 Jun '15

Roy Golan has submitted this change and it was merged. Change subject: Remove UpdatePartialVdsDynamicCalc stored procedure ...................................................................... Remove UpdatePartialVdsDynamicCalc stored procedure This procedure is no longer needed and was removed from the VdsDAO when the new pending resource tracking mechanism was introduced. Change-Id: I2b61d7465c3a7182fef398a6b65ea5555a68aa94 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M packaging/dbscripts/vds_sp.sql 1 file changed, 0 insertions(+), 30 deletions(-) Approvals: Martin Sivák: Verified Eli Mesika: Looks good to me, approved Jenkins CI: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41821 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b61d7465c3a7182fef398a6b65ea5555a68aa94 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: simplified refreshVdsmFileList unsuccessful call to vd...
by ahadas＠redhat.com 10 Jun '15

10 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: core: simplified refreshVdsmFileList unsuccessful call to vdsm as well as returning null are considered to be invalid return values. Taking advantage of it, if vds call is unsuccessful, it's pretended as if null was returned instead simplifying further pr ...................................................................... core: simplified refreshVdsmFileList unsuccessful call to vdsm as well as returning null are considered to be invalid return values. Taking advantage of it, if vds call is unsuccessful, it's pretended as if null was returned instead simplifying further processing. Bug-Url: https://bugzilla.redhat.com/1200952 Change-Id: Ic8b53afcd6c0e87f4f206a60735cecf9a51480c5 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java 1 file changed, 17 insertions(+), 11 deletions(-) Approvals: Martin Mucha: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41275 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8b53afcd6c0e87f4f206a60735cecf9a51480c5 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: restapi: Adds disperse, redundancy info to glustervolumes res...
by sabose＠redhat.com 10 Jun '15

10 Jun '15

Sahina Bose has submitted this change and it was merged. Change subject: restapi: Adds disperse,redundancy info to glustervolumes restapi ...................................................................... restapi: Adds disperse,redundancy info to glustervolumes restapi Disperse count, redundancy count informations of disperse volumes are added to glustervolumes restapi. Change-Id: I722b50e81c5118146d3cd4fb9c522a44aae17362 Bug-Url: https://bugzilla.redhat.com/1213379 Signed-off-by: Karnan TC <kchidamb(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/GlusterVolumeType.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GlusterVolumeMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/GlusterVolumeMapperTest.java 4 files changed, 29 insertions(+), 1 deletion(-) Approvals: Karnan t c: Verified Kanagaraj M: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41916 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I722b50e81c5118146d3cd4fb9c522a44aae17362 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Karnan t c <kchidamb(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]: core: refactored overgrown methods and calls
by ahadas＠redhat.com 10 Jun '15

10 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: core: refactored overgrown methods and calls ...................................................................... core: refactored overgrown methods and calls Bug-Url: https://bugzilla.redhat.com/1200952 Change-Id: I0ec9c8ab12069851c623776ecdde69f07978b0ed Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java 1 file changed, 13 insertions(+), 10 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41274 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ec9c8ab12069851c623776ecdde69f07978b0ed Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0