Change in ovirt-engine[ovirt-engine-3.6]: core: DeactivateStorageDomainCommand - refresh all hosts

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: core: DeactivateStorageDomainCommand - refresh all hosts
......................................................................


core: DeactivateStorageDomainCommand - refresh all hosts

Currently on DeactivateStorageDomain in a DC that uses the disk backend
we refresh the hosts that are used to move domains to Active status
while in dc that uses the memory backend we refresh also hosts in
statuses that we check before moving a domain to maintenance.
As currently the refresh operation always required an access to the
master domain, this patch unifies the behavior - the operation might
take long time if there's no access to the master domain, but on most
cases we'll benefit from it as domain will move to Maintenance status
faster if there is a non operational host

Change-Id: I4c371e6188cdb93e1986996c2dc9b843889969bd
Bug-Url: https://bugzilla.redhat.com/1256841
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainCommand.java
1 file changed, 1 insertion(+), 9 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved; Passed CI tests
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4c371e6188cdb93e1986996c2dc9b843889969bd
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Liron Aravot <laravot 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>