Change in ovirt-engine[master]: engine: Add ability to check if a VDS is hosting the HE VM
Roy Golan has submitted this change and it was merged.
Change subject: engine: Add ability to check if a VDS is hosting the HE VM
......................................................................
engine: Add ability to check if a VDS is hosting the HE VM
Add a property to the VDS entity that tracks whether a given host is
hosting the hosted engine VM. The property is calculated and
populated in the VDS view and is not persisted.
This approach provides the simplest way to get this information
into the VDS entity since the Java 1.7 limitation on classes in
the common package prevents using convenience methods available
elsewhere in the code base. The change to the VDS view created
no significant performance issues when tested locally and at
scale.
Change-Id: I7e2609f59e588c6ab9279b74312b68dc5af06d5a
Bug-Url: https://bugzilla.redhat.com/1308861
Signed-off-by: Phillip Bailey <phbailey(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDaoImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDaoTest.java
M packaging/dbscripts/create_views.sql
4 files changed, 127 insertions(+), 2 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Roy Golan: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/55669
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7e2609f59e588c6ab9279b74312b68dc5af06d5a
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>