Liron Ar has posted comments on this change.
Change subject: engine: check active vds before activate storage domain
......................................................................
Patch Set 4:
(1 comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageDomainCommandBase.java
Line 210: protected boolean checkActiveVdsInStoragePool() {
Line 211: List<VDS> vdss =
getVdsDAO().getAllForStoragePoolAndStatus(getParameters().getStoragePoolId(),
VDSStatus.Up);
Line 212: if (vdss.isEmpty()) {
Line 213:
addCanDoActionMessage(VdcBllMessages.ACTION_TYPE_FAILED_NO_ACTIVE_VDS_IN_STORAGE_POOL);
Line 214: return false;
1. please use return failCanDoAction
2. please unite the code with the one at
StorageHandlingCommandBase.initalizeVds() which executes the same check
Line 215: }
Line 216: return true;
Line 217: }
Line 218:
--
To view, visit http://gerrit.ovirt.org/17940
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia3e45b53d15b757237af8c76457f213c00a73909
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: plysan <ply...@gmail.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: LiJiansheng <lijiangshe...@gmail.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-Reviewer: plysan <ply...@gmail.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
