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); > Implement it "in an awkward way"? How can you judge that using REST to allo Using rest to allocate nics to gain control over mac address allocation - is awkward. It is just a method to by pass the engine inability to support various mac pool manager behavior. The correct approach would have been to let the user provide his own logic for mac pool manager (i referred to it in previous comment as "a pluggable mac address mechanism"). And all the other flows should have been look the same. 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
