Change in ovirt-engine[ovirt-engine-3.6.1]: core: Auto import Hosted Engine Storage Domain

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: core: Auto import Hosted Engine Storage Domain
......................................................................


core: Auto import Hosted Engine Storage Domain

The hosted engine storage domain is a critical part of the
managing that VM with the engine.

In order to reduce manual intervention, we can collect all
data and import that domain automatically.

The flow would be:

1. Analyze the VM (VM monitoring) and check if its MANAGED
2. if not MANAGED, try to import the SD
3. detect the SD details by crossmatching the domain info (we have the
   name in config) with the output of getDeviceList, to get connection
   details of ISCSI
4. import the SD
5. if SD imported, import the VM

Change-Id: Ib1a240e8e5ee2502a5ce11ec586a0eacec9f83fe
Bug-Url: https://bugzilla.redhat.com/1269768
Signed-off-by: Roy Golan <rgolan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HostedEngineImporter.java
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportHostedEngineStorageDomainCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostedengine/HostedEngineHelper.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetExistingStorageDomainListQuery.java
A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImportHostedEngineStorageDomainCommandTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/CommonTestMocks.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/SchedulingManagerTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
9 files changed, 516 insertions(+), 12 deletions(-)

Approvals:
  Tal Nisan: Passed CI tests
  Roy Golan: Verified
  Maor Lipchuk: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/50077
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1a240e8e5ee2502a5ce11ec586a0eacec9f83fe
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6.1
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>