Omer Frenkel has posted comments on this change. Change subject: core: adjustments needed to support instance types ......................................................................
Patch Set 21: Code-Review+1 (1 comment) looks ok, just the remove on export domain bothers me http://gerrit.ovirt.org/#/c/23828/21/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmTemplateFromImportExportCommand.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmTemplateFromImportExportCommand.java: Line 48: @Override Line 49: protected boolean canDoAction() { Line 50: boolean isInstanceType = getVmTemplate().getTemplateType() != VmEntityType.INSTANCE_TYPE; Line 51: Line 52: if (getVdsGroup() == null && !isInstanceType) { i dont understand this, why we need any cluster check when removing template from export domain, its not in the system, we might not have the cluster in the db. Line 53: addCanDoActionMessage(VdcBllMessages.ACTION_TYPE_FAILED_CLUSTER_CAN_NOT_BE_EMPTY); Line 54: return false; Line 55: } Line 56: -- To view, visit http://gerrit.ovirt.org/23828 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifaebd849061efa1637f9fa21d8584212ba0e51f2 Gerrit-PatchSet: 21 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Lior Vernia <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Tomas Jelinek <[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
