Change in ovirt-engine[ovirt-engine-3.6]: core: hosted-engine: Add connection details explicitly for NFS

tnisan at redhat.com tnisan at redhat.com
Wed Jan 13 15:16:38 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: hosted-engine: Add connection details explicitly for NFS
......................................................................


core: hosted-engine: Add connection details explicitly for NFS

It is not reliable to assume AddExistingFileStorageDomain command
will create the connection details internally, thus the import HE domain
command calls is prior to importing the domain.

Change-Id: I790d1d22ffe5358b8a0daeb85b004d5a484a4745
Bug-Url: https://bugzilla.redhat.com/1293928
Bug-Url: https://bugzilla.redhat.com/1290518
Signed-off-by: Roy Golan <rgolan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportHostedEngineStorageDomainCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImportHostedEngineStorageDomainCommandTest.java
2 files changed, 44 insertions(+), 4 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I790d1d22ffe5358b8a0daeb85b004d5a484a4745
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
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>



More information about the Engine-commits mailing list