Change in ovirt-engine[master]: engine: ConnectStorageServer on attach FC SD

amureini at redhat.com amureini at redhat.com
Tue Jul 26 10:19:20 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: engine: ConnectStorageServer on attach FC SD
......................................................................


engine: ConnectStorageServer on attach FC SD

After creating a new storage domain, we will attach the SD to a pool.
As part of the AttachStorageDomainToPoolCommand, we connect all the hosts
to the storage server.

In case of Fiber Channel, the connectStorageServer command was ignored
since the SD was not connected to any pool.

Change-Id: I9aedd2cd177a0697c8096f025d0fe636ac04557f
Bug-Url: https://bugzilla.redhat.com/1353430
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/connection/FCPStorageHelper.java
1 file changed, 1 insertion(+), 4 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, approved; Passed CI tests
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9aedd2cd177a0697c8096f025d0fe636ac04557f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot 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