Change in ovirt-engine[master]: core: Change auto-recovery code for storage domains
Michael Kublin has submitted this change and it was merged.
Change subject: core: Change auto-recovery code for storage domains
......................................................................
core: Change auto-recovery code for storage domains
Because of change which was done at error storage flows
these is no need for activating of storage domain in auto-recovery, when domain will be
seen by vdsm it will be automaticlly switched to Active status.
These is possible because of no more disconnect and deactivate is
send when domain in error state.
At any case we will try to reconnect storage domain if it was disconnected.
The status of domain to Active will be switch by vdsm
Change-Id: I04f25dca8888b19711a840aed150227437972d65
Signed-off-by: Michael Kublin <mkublin(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoRecoveryManager.java
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectDomainToStorageCommand.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AutoRecoveryManagerTest.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
4 files changed, 29 insertions(+), 3 deletions(-)
Approvals:
Michael Kublin: Verified
Yair Zaslavsky: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8015
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I04f25dca8888b19711a840aed150227437972d65
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>