Moti Asayag has posted comments on this change. Change subject: core: dont fail on empty ranges ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/35391/1/backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/MacAddressPoolRangesValueHelper.java File backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/MacAddressPoolRangesValueHelper.java: Line 25: @Override Line 26: public ValidationResult validate(ConfigKey key, String value) { Line 27: Line 28: if (StringUtils.isBlank(value)) { Line 29: return new ValidationResult(true); > Full control over MAC address allocation. All vNIC creation should be done in that case the user shouldn't mess at all with the mac pool at all. that also implies that for ovirt to be a scale solution, the user will have to implement its own mac pool manager solution and use ovirt via api only. this doesn't comply with the nature of ovirt as a manager. Line 30: } Line 31: Line 32: try { Line 33: List<String[]> rangesBoundaries = MacAddressRangeUtils.rangeStringToStringBoundaries(value); -- To view, visit http://gerrit.ovirt.org/35391 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I314ff9e62d7515eae334c461a67ce609ca188359 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Martin Mucha <[email protected]> Gerrit-Reviewer: Lior Vernia <[email protected]> Gerrit-Reviewer: Martin Mucha <[email protected]> Gerrit-Reviewer: Moti Asayag <[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
