Change in ovirt-engine[master]: engine: Add ability to check if a VDS is hosting the HE VM

rgolan at redhat.com rgolan at redhat.com
Mon May 9 10:09:50 UTC 2016


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 at 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 at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list