ovsdb enable checkstyle on error

[ovsdb.git] / southbound / southbound-impl / src / main / java / org / opendaylight / ovsdb / southbound / transactions / md / OvsdbPortRemoveCommand.java

diff --git a/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbPortRemoveCommand.java b/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbPortRemoveCommand.java
index 763ae27f91227f465e39df9c50b146154d9ba4c2..756fd49616535b2054e7f500a6325634757dacd1 100644 (file)
--- a/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbPortRemoveCommand.java
+++ b/southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbPortRemoveCommand.java
@@ -9,47 +9,64 @@
 package org.opendaylight.ovsdb.southbound.transactions.md;
 
 import java.util.Collection;
+import java.util.Map;
 
 import org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.ovsdb.lib.message.TableUpdates;
+import org.opendaylight.ovsdb.lib.notation.UUID;
 import org.opendaylight.ovsdb.lib.schema.DatabaseSchema;
 import org.opendaylight.ovsdb.lib.schema.typed.TyperUtils;
+import org.opendaylight.ovsdb.schema.openvswitch.Bridge;
 import org.opendaylight.ovsdb.schema.openvswitch.Port;
+import org.opendaylight.ovsdb.southbound.OvsdbConnectionInstance;
 import org.opendaylight.ovsdb.southbound.SouthboundMapper;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeName;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.ovsdb.node.attributes.ConnectionInfo;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.TpId;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPoint;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPointKey;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class OvsdbPortRemoveCommand extends AbstractTransactionCommand {
-    public OvsdbPortRemoveCommand(ConnectionInfo key, TableUpdates updates,
+    private static final Logger LOG = LoggerFactory.getLogger(OvsdbPortRemoveCommand.class);
+
+    public OvsdbPortRemoveCommand(OvsdbConnectionInstance key, TableUpdates updates,
             DatabaseSchema dbSchema) {
         super(key, updates, dbSchema);
     }
 
     @Override
     public void execute(ReadWriteTransaction transaction) {
-        String bridgeName = null;
         String portName = null;
         Collection<Port> portRemovedRows = TyperUtils.extractRowsRemoved(
                 Port.class, getUpdates(), getDbSchema()).values();
-        Collection<Port> portUpdatedRows = TyperUtils.extractRowsUpdated(
-                Port.class, getUpdates(), getDbSchema()).values();
-        for (Port bridge : portUpdatedRows) {
-            bridgeName = bridge.getName();
-            for (Port port : portRemovedRows) {
-                portName = port.getName();
-                final InstanceIdentifier<TerminationPoint> nodePath = SouthboundMapper
-                        .createInstanceIdentifier(getConnectionInfo(),
-                                new OvsdbBridgeName(bridgeName)).child(
-                                TerminationPoint.class,
-                                new TerminationPointKey(new TpId(portName)));
-                transaction.delete(LogicalDatastoreType.OPERATIONAL, nodePath);
-                transaction.submit();
+        Map<UUID,Bridge> bridgeUpdatedRows = TyperUtils.extractRowsUpdated(
+                Bridge.class, getUpdates(), getDbSchema());
+        Map<UUID,Bridge> bridgeUpdatedOldRows = TyperUtils.extractRowsOld(
+                Bridge.class, getUpdates(), getDbSchema());
+        for (Port port : portRemovedRows) {
+            Bridge updatedBridgeData = null;
+            for (UUID bridgeUuid : bridgeUpdatedOldRows.keySet()) {
+                Bridge oldBridgeData = bridgeUpdatedOldRows.get(bridgeUuid);
+                if (oldBridgeData.getPortsColumn() != null
+                        && oldBridgeData.getPortsColumn().getData().contains(port.getUuidColumn().getData())
+                        && (! bridgeUpdatedRows.isEmpty())) {
+                    updatedBridgeData = bridgeUpdatedRows.get(bridgeUuid);
+                    break;
+                }
+            }
+            if (updatedBridgeData == null) {
+                LOG.warn("Bridge not found for port {}",port);
+                continue;
             }
+            portName = port.getName();
+            final InstanceIdentifier<TerminationPoint> nodePath = SouthboundMapper
+                    .createInstanceIdentifier(getOvsdbConnectionInstance(),
+                            updatedBridgeData).child(
+                            TerminationPoint.class,
+                            new TerminationPointKey(new TpId(portName)));
+            transaction.delete(LogicalDatastoreType.OPERATIONAL, nodePath);
         }
     }