Change in ovirt-engine[master]: core[BLL]: Adding external status to command
by emesika@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: restapi: move InstanceType remove from collection to entity
by juan.hernandez@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: core[DAO]: Adding external status to DAOs
by emesika@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: core[BE]: Adding external status to BEs
by emesika@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: core[Enum]: Adding external status enum
by emesika@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: core[DB]: Adding external status to DB
by emesika@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: Remove UpdatePartialVdsDynamicCalc stored procedure
by rgolan@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: core: simplified refreshVdsmFileList unsuccessful call to vd...
by ahadas@redhat.com
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
9 years, 6 months
Change in ovirt-engine[ovirt-engine-3.5-gluster]: restapi: Adds disperse, redundancy info to glustervolumes res...
by sabose@redhat.com
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
9 years, 6 months
Change in ovirt-engine[master]: core: refactored overgrown methods and calls
by ahadas@redhat.com
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
9 years, 6 months