Liron Ar has posted comments on this change. Change subject: core: handle 'source' domains when adding a vm from template ......................................................................
Patch Set 5: (2 comments) http://gerrit.ovirt.org/#/c/26262/5/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/DiskImagesValidator.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/DiskImagesValidator.java: Line 190: * is null, it will be checked that there are no derived disks on any storage domain. Line 191: */ Line 192: public ValidationResult diskImagesOnAnyApplicableDomains(Map<Guid, Set<Guid>> validDomainsForDisk, Line 193: Map<Guid, StorageDomain> storageDomains, Line 194: VdcBllMessages message, > Can the method provide a default message and accept a custom one for re-usa I prefer that it won't, add the message is specific for templates i prefer to keep this validator as generic as possible and pass it from the relevant usecase. Line 195: Set<StorageDomainStatus> applicableStatuses) { Line 196: Line 197: List<String> disksInfo = new LinkedList<>(); Line 198: for (DiskImage diskImage : diskImages) { Line 207: nonApplicableStorageInfo.add(String.format("%s - %s", domain.getName(), domain.getStatus() Line 208: .toString())); Line 209: } Line 210: Line 211: disksInfo.add(String.format("%s (%s) %n", > Consider adding a comment with an example illustrating the final result of I'll add it. Line 212: diskImage.getDiskAlias(), Line 213: StringUtils.join(nonApplicableStorageInfo, " / "))); Line 214: } Line 215: -- To view, visit http://gerrit.ovirt.org/26262 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8d02cfded41a3588dc944d9dfcd5a3eec88c45ab Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[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
