Daniel Erez has posted comments on this change. Change subject: core: Support detach Storage Domain with disks. ......................................................................
Patch Set 13: (2 comments) http://gerrit.ovirt.org/#/c/24286/13/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DetachStorageDomainFromPoolCommand.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DetachStorageDomainFromPoolCommand.java: Line 42: protected DetachStorageDomainFromPoolCommand(Guid commandId) { Line 43: super(commandId); Line 44: } Line 45: Line 46: private boolean isChildReturnValueSucceeded(List<VdcReturnValueBase> vdcReturnValues) { Consider extracting to CommandBase - so it could be reused in MoveDisksCommand -> handleChildReturnValue. Line 47: for (VdcReturnValueBase vdcReturnValue : vdcReturnValues) { Line 48: getReturnValue().getCanDoActionMessages().addAll(vdcReturnValue.getCanDoActionMessages()); Line 49: getReturnValue().setCanDoAction(getReturnValue().getCanDoAction() && vdcReturnValue.getCanDoAction()); Line 50: } http://gerrit.ovirt.org/#/c/24286/13/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java File frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java: Line 675: Line 676: @DefaultStringValue("Cannot ${action} ${type}. The following VMs are not down: ${vms}.") Line 677: String ACTION_TYPE_FAILED_STORAGE_VMS_NOT_DOWN(); Line 678: Line 679: @DefaultStringValue("Cannot ${action} ${type}. The following VMs are attached to pool: ${vms}.") Can the pool name be added? Line 680: String ACTION_TYPE_FAILED_STORAGE_VMS_IN_POOL(); Line 681: Line 682: @DefaultStringValue("Cannot ${action} ${type}. The Storage Domain name is already in use.") Line 683: String ACTION_TYPE_FAILED_STORAGE_DOMAIN_NAME_ALREADY_EXIST(); -- To view, visit http://gerrit.ovirt.org/24286 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I971fe6acd4a2667a09487c5e1108cf7c759587f1 Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Liran Zelkha <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
