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

laravot at redhat.com laravot at redhat.com
Tue Nov 24 08:20:33 UTC 2015


Liron Aravot 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:
  Liron Aravot: Verified; Looks good to me, approved; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4c371e6188cdb93e1986996c2dc9b843889969bd
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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>



More information about the Engine-commits mailing list