Change in ovirt-engine[master]: core: Host shouldn't move to UP if it's already connected to...

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: core: Host shouldn't move to UP if it's already connected to other pool
......................................................................


core: Host shouldn't move to UP if it's already connected to other pool

When host fails to connect to the pool, it shouldn't move to UP if it's
already connected to other pool regardless of the master domain status.

Regardless of this change, the error handling in
ConnectStoragePoolVdsCommand should be changed as well as the error
handling in it's callers, this change handles InitVdsOnUp.

Change-Id: Ifd3d8268516b010f7831b59bbc6331cfd70efec1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1023732
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
1 file changed, 14 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/21095
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifd3d8268516b010f7831b59bbc6331cfd70efec1
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server