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

rgolan at redhat.com rgolan at redhat.com
Tue Jan 12 23:46:05 UTC 2016


Roy Golan 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/storage/domain/ImportHostedEngineStorageDomainCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/domain/ImportHostedEngineStorageDomainCommandTest.java
2 files changed, 42 insertions(+), 3 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Roy Golan: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I790d1d22ffe5358b8a0daeb85b004d5a484a4745
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez 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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list