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