Michael Kublin has uploaded a new change for review. Change subject: engine: Removing HandleFailedStorageDomainCommand ......................................................................
engine: Removing HandleFailedStorageDomainCommand The following command is removed because of it was used only at one place: VdsEventListener.storageDomainNotOperational, the logic of command was: if domain is master run reconstruct, otherwise run deactivate storage domain. You can see that before call to VdsEventListener.storageDomainNotOperational the following check is performed and if domain is master we are calling to VdsEventListener.masterDomainNotOperational otherwise VdsEventListener.storageDomainNotOperational, so no need to duplicate a logic Change-Id: I61e1674056f9060440989d70516109dd5ee37f9a Signed-off-by: Michael Kublin <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/HandleFailedStorageDomainCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 3 files changed, 6 insertions(+), 56 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/79/6179/1 -- To view, visit http://gerrit.ovirt.org/6179 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I61e1674056f9060440989d70516109dd5ee37f9a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
