HC2VPP-78 - subnet validation fix

Validation removed, more descriptive expcetion handling to be added
after VPP-649

Change-Id: I6e0a84b2dc3f3c9d3d943874baa508636a1df808
Signed-off-by: Jan Srnicek <jsrnicek@cisco.com>

3 files changed, 3 insertions, 157 deletions

diff --git a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/subnet/validation/Ipv4SubnetValidator.java b/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/subnet/validation/Ipv4SubnetValidator.java
deleted file mode 100644
index 28cd70440..000000000
--- a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/subnet/validation/Ipv4SubnetValidator.java
+++ /dev/null
@@ -1,80 +0,0 @@
-/*
- * Copyright (c) 2016 Cisco and/or its affiliates.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at:
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package io.fd.hc2vpp.v3po.interfaces.ip.subnet.validation;
-
-import static com.google.common.base.Preconditions.checkNotNull;
-
-import com.google.common.base.Function;
-import com.google.common.collect.Multimap;
-import com.google.common.collect.Multimaps;
-import io.fd.hc2vpp.v3po.interfaces.ip.IpWriter;
-import java.util.List;
-import javax.annotation.Nonnull;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.ip.rev140616.interfaces._interface.ipv4.Address;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.ip.rev140616.interfaces._interface.ipv4.address.Subnet;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.ip.rev140616.interfaces._interface.ipv4.address.subnet.Netmask;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.ip.rev140616.interfaces._interface.ipv4.address.subnet.PrefixLength;
-
-/**
- * Validator for detecting if there is an attempt to assign multiple addresses from same subnet
- */
-public class Ipv4SubnetValidator implements IpWriter {
-
-    /**
-     * Checks whether data provided for writing are not in collision with already existing data
-     */
-    public void checkNotAddingToSameSubnet(@Nonnull final List<Address> addresses)
-            throws SubnetValidationException {
-
-        final Multimap<Short, Address> prefixLengthRegister = Multimaps.index(addresses, toPrefixLength());
-        final int keySetSize = prefixLengthRegister.keySet().size();
-
-        if (keySetSize == 0 || keySetSize == addresses.size()) {
-            //this means that every key is unique(has only one value) or no addresses were prefix-length based ,so there is no conflict
-            return;
-        }
-
-        //finds conflicting prefix
-        final Short conflictingPrefix = prefixLengthRegister.keySet()
-                .stream()
-                .filter(a -> prefixLengthRegister.get(a).size() > 1)
-                .findFirst()
-                .get();
-
-        //and reports it with affected addresses
-        throw SubnetValidationException
-                .forConflictingData(conflictingPrefix, prefixLengthRegister.get(conflictingPrefix));
-    }
-
-    private Function<Address, Short> toPrefixLength() {
-        return (final Address address) -> {
-            final Subnet subnet = address.getSubnet();
-
-            if (subnet instanceof PrefixLength) {
-                return ((PrefixLength) subnet).getPrefixLength();
-            }
-
-            if (address.getSubnet() instanceof Netmask) {
-                return (short) getSubnetMaskLength(
-                        checkNotNull(((Netmask) subnet).getNetmask(), "No netmask defined for %s", subnet)
-                                .getValue());
-            }
-
-            throw new IllegalArgumentException("Unsupported subnet : " + subnet);
-        };
-    }
-}
diff --git a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/subnet/validation/SubnetValidationException.java b/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/subnet/validation/SubnetValidationException.java
deleted file mode 100644
index 001923093..000000000
--- a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/subnet/validation/SubnetValidationException.java
+++ /dev/null
@@ -1,50 +0,0 @@
-/*
- * Copyright (c) 2016 Cisco and/or its affiliates.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at:
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package io.fd.hc2vpp.v3po.interfaces.ip.subnet.validation;
-
-import static com.google.common.base.Preconditions.checkNotNull;
-
-import java.util.Collection;
-import javax.annotation.Nonnull;
-import org.apache.commons.lang3.StringUtils;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.ip.rev140616.interfaces._interface.ipv4.Address;
-
-/**
- * Thrown as negative result of subnet validation
- */
-public class SubnetValidationException extends Exception {
-
-    private SubnetValidationException(@Nonnull final String message) {
-        super(message);
-    }
-
-    public static SubnetValidationException forConflictingData(@Nonnull final Short prefix, @Nonnull Collection<Address> addresses) {
-        return new SubnetValidationException(
-                "Attempt to define multiple addresses for same subnet[prefixLen = " + prefixToString(prefix) + "], "
-                        + "addresses : " + addressesToString(addresses));
-    }
-
-    private static String prefixToString(final Short prefix) {
-        return checkNotNull(prefix, "Cannot create " + SubnetValidationException.class.getName() + " for null prefix")
-                .toString();
-    }
-
-    private static String addressesToString(final Collection<Address> addresses) {
-        return StringUtils.join(checkNotNull(addresses,
-                "Cannot create " + SubnetValidationException.class.getName() + " for null address list"), " | ");
-    }
-}
diff --git a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/v4/Ipv4AddressCustomizer.java b/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/v4/Ipv4AddressCustomizer.java
index 0860436d3..d6cc6a00f 100644
--- a/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/v4/Ipv4AddressCustomizer.java
+++ b/v3po/v3po2vpp/src/main/java/io/fd/hc2vpp/v3po/interfaces/ip/v4/Ipv4AddressCustomizer.java
@@ -23,8 +23,6 @@ import com.google.common.base.Optional;
 import io.fd.hc2vpp.common.translate.util.FutureJVppCustomizer;
 import io.fd.hc2vpp.common.translate.util.NamingContext;
 import io.fd.hc2vpp.v3po.interfaces.ip.IpWriter;
-import io.fd.hc2vpp.v3po.interfaces.ip.subnet.validation.Ipv4SubnetValidator;
-import io.fd.hc2vpp.v3po.interfaces.ip.subnet.validation.SubnetValidationException;
 import io.fd.honeycomb.translate.spi.write.ListWriterCustomizer;
 import io.fd.honeycomb.translate.util.RWUtils;
 import io.fd.honeycomb.translate.write.WriteContext;
@@ -51,19 +49,11 @@ public class Ipv4AddressCustomizer extends FutureJVppCustomizer
 
     private static final Logger LOG = LoggerFactory.getLogger(Ipv4AddressCustomizer.class);
     private final NamingContext interfaceContext;
-    private final Ipv4SubnetValidator subnetValidator;
-
-    @VisibleForTesting
-    Ipv4AddressCustomizer(@Nonnull final FutureJVppCore futureJVppCore, @Nonnull final NamingContext interfaceContext,
-                          @Nonnull final Ipv4SubnetValidator subnetValidator) {
-        super(futureJVppCore);
-        this.interfaceContext = checkNotNull(interfaceContext, "Interface context cannot be null");
-        this.subnetValidator = checkNotNull(subnetValidator, "Subnet validator cannot be null");
-    }
 
     public Ipv4AddressCustomizer(@Nonnull final FutureJVppCore futureJVppCore,
                                  @Nonnull final NamingContext interfaceContext) {
-        this(futureJVppCore, interfaceContext, new Ipv4SubnetValidator());
+        super(futureJVppCore);
+        this.interfaceContext = checkNotNull(interfaceContext, "Interface context cannot be null");
     }
 
     @Override
@@ -72,21 +62,7 @@ public class Ipv4AddressCustomizer extends FutureJVppCustomizer
 
         final String interfaceName = id.firstKeyOf(Interface.class).getName();
         final int interfaceIndex = interfaceContext.getIndex(interfaceName, writeContext.getMappingContext());
-
-        // checks whether address is not from same subnet of some address already defined on this interface
-        try {
-
-            final InstanceIdentifier<Ipv4> parentId = RWUtils.cutId(id, InstanceIdentifier.create(Ipv4.class));
-            final Optional<Ipv4> ipv4Optional = writeContext.readAfter(parentId);
-
-            //no need to check isPresent() - we are inside of address customizer, therefore there must be Address data
-            //that is being processed by infrastructure
-
-            subnetValidator.checkNotAddingToSameSubnet(ipv4Optional.get().getAddress());
-        } catch (SubnetValidationException e) {
-            throw new WriteFailedException(id, e);
-        }
-
+        // TODO - HC2VPP-92 - Add more descriptive exception handling after https://jira.fd.io/browse/VPP-649
         setAddress(true, id, interfaceName, interfaceIndex, dataAfter, writeContext);
     }