- SAL Inventory needs to be modified to be able to take multiple IPluginInInventorySe...
[controller.git] / opendaylight / protocol_plugins / openflow / src / main / java / org / opendaylight / controller / protocol_plugin / openflow / internal / InventoryService.java
index 7041f447e37608b15f9321f6e2ac82d56520f137..7a2ace5bae6008e0262b06755d1e70bec668219f 100644 (file)
@@ -1,4 +1,3 @@
-
 /*
  * Copyright (c) 2013 Cisco Systems, Inc. and others.  All rights reserved.
  *
@@ -20,27 +19,26 @@ import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
 
 import org.apache.felix.dm.Component;
+import org.opendaylight.controller.protocol_plugin.openflow.IInventoryProvider;
 import org.opendaylight.controller.protocol_plugin.openflow.IInventoryShimInternalListener;
-import org.opendaylight.controller.protocol_plugin.openflow.IOFInventoryService;
 import org.opendaylight.controller.protocol_plugin.openflow.core.IController;
 import org.opendaylight.controller.protocol_plugin.openflow.core.ISwitch;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
+import org.opendaylight.controller.sal.core.Actions;
+import org.opendaylight.controller.sal.core.Buffers;
+import org.opendaylight.controller.sal.core.Capabilities;
 import org.opendaylight.controller.sal.core.ConstructionException;
 import org.opendaylight.controller.sal.core.Node;
 import org.opendaylight.controller.sal.core.Node.NodeIDType;
 import org.opendaylight.controller.sal.core.NodeConnector;
 import org.opendaylight.controller.sal.core.Property;
-import org.opendaylight.controller.sal.core.TimeStamp;
 import org.opendaylight.controller.sal.core.Tables;
-import org.opendaylight.controller.sal.core.Actions;
-import org.opendaylight.controller.sal.core.Buffers;
-import org.opendaylight.controller.sal.core.Capabilities;
+import org.opendaylight.controller.sal.core.TimeStamp;
 import org.opendaylight.controller.sal.core.UpdateType;
 import org.opendaylight.controller.sal.inventory.IPluginInInventoryService;
 import org.opendaylight.controller.sal.inventory.IPluginOutInventoryService;
 import org.opendaylight.controller.sal.utils.GlobalConstants;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * The class describes inventory service protocol plugin. One instance per
@@ -51,14 +49,14 @@ import org.opendaylight.controller.sal.utils.GlobalConstants;
  *
  */
 public class InventoryService implements IInventoryShimInternalListener,
-        IPluginInInventoryService, IOFInventoryService {
+        IPluginInInventoryService, IInventoryProvider {
     protected static final Logger logger = LoggerFactory
             .getLogger(InventoryService.class);
     private Set<IPluginOutInventoryService> pluginOutInventoryServices = Collections
             .synchronizedSet(new HashSet<IPluginOutInventoryService>());
     private IController controller = null;
-    private ConcurrentMap<Node, Map<String, Property>> nodeProps; // properties are maintained in default container only
-    private ConcurrentMap<NodeConnector, Map<String, Property>> nodeConnectorProps; // properties are maintained in default container only
+    private ConcurrentMap<Node, Map<String, Property>> nodeProps; // properties are maintained in global container only
+    private ConcurrentMap<NodeConnector, Map<String, Property>> nodeConnectorProps; // properties are maintained in global container only
     private boolean isDefaultContainer = false;
 
     void setController(IController s) {
@@ -142,7 +140,7 @@ public class InventoryService implements IInventoryShimInternalListener,
         try {
             node = new Node(NodeIDType.OPENFLOW, id);
         } catch (ConstructionException e) {
-            e.printStackTrace();
+            logger.error("", e);
         }
 
         return node;
@@ -157,7 +155,7 @@ public class InventoryService implements IInventoryShimInternalListener,
             return null;
         Map<Long, ISwitch> switches = controller.getSwitches();
         for (Map.Entry<Long, ISwitch> entry : switches.entrySet()) {
-               ISwitch sw = entry.getValue();
+            ISwitch sw = entry.getValue();
             Node node = OFSwitchToNode(sw);
             Map<String, Property> propMap = null;
             if (isDefaultContainer) {
@@ -165,22 +163,22 @@ public class InventoryService implements IInventoryShimInternalListener,
                 byte tables = sw.getTables();
                 Tables t = new Tables(tables);
                 if (t != null) {
-                       propMap.put(Tables.TablesPropName,t);
+                    propMap.put(Tables.TablesPropName, t);
                 }
                 int cap = sw.getCapabilities();
                 Capabilities c = new Capabilities(cap);
                 if (c != null) {
-                       propMap.put(Capabilities.CapabilitiesPropName, c);
+                    propMap.put(Capabilities.CapabilitiesPropName, c);
                 }
                 int act = sw.getActions();
                 Actions a = new Actions(act);
                 if (a != null) {
-                       propMap.put(Actions.ActionsPropName,a);
+                    propMap.put(Actions.ActionsPropName, a);
                 }
                 int buffers = sw.getBuffers();
                 Buffers b = new Buffers(buffers);
                 if (b != null) {
-                       propMap.put(Buffers.BuffersPropName,b);
+                    propMap.put(Buffers.BuffersPropName, b);
                 }
                 Date connectedSince = sw.getConnectedDate();
                 Long connectedSinceTime = (connectedSince == null) ? 0
@@ -197,8 +195,9 @@ public class InventoryService implements IInventoryShimInternalListener,
     @Override
     public ConcurrentMap<NodeConnector, Map<String, Property>> getNodeConnectorProps(
             Boolean refresh) {
-        if (nodeConnectorProps == null)
+        if (nodeConnectorProps == null) {
             return null;
+        }
 
         if (isDefaultContainer && refresh) {
             Map<Long, ISwitch> switches = controller.getSwitches();
@@ -207,8 +206,8 @@ public class InventoryService implements IInventoryShimInternalListener,
                         .OFSwitchToProps(sw);
                 for (Map.Entry<NodeConnector, Set<Property>> entry : ncProps
                         .entrySet()) {
-                    updateNodeConnector(entry.getKey(), UpdateType.ADDED, entry
-                            .getValue());
+                    updateNodeConnector(entry.getKey(), UpdateType.ADDED,
+                            entry.getValue());
                 }
             }
         }
@@ -219,36 +218,32 @@ public class InventoryService implements IInventoryShimInternalListener,
     @Override
     public void updateNodeConnector(NodeConnector nodeConnector,
             UpdateType type, Set<Property> props) {
-        logger.trace("NodeConnector id " + nodeConnector.getID()
-                + " type " + nodeConnector.getType() + " "
-                + type.getName() + " for Node id "
-                + nodeConnector.getNode().getID());
-
-        if (nodeConnectorProps == null)
+        logger.trace("updateNodeConnector {} type {}", nodeConnector,
+                type.getName());
+        if (nodeConnectorProps == null) {
+            logger.trace("nodeConnectorProps is null");
             return;
+        }
 
-        synchronized (nodeConnectorProps) {
-            Map<String, Property> propMap = nodeConnectorProps
-                    .get(nodeConnector);
-            switch (type) {
-            case ADDED:
-            case CHANGED:
-                if (propMap == null)
-                    propMap = new HashMap<String, Property>();
-
-                if (props != null) {
-                    for (Property prop : props) {
-                        propMap.put(prop.getName(), prop);
-                    }
+        Map<String, Property> propMap = nodeConnectorProps.get(nodeConnector);
+        switch (type) {
+        case ADDED:
+        case CHANGED:
+            if (propMap == null) {
+                propMap = new HashMap<String, Property>();
+            }
+            if (props != null) {
+                for (Property prop : props) {
+                    propMap.put(prop.getName(), prop);
                 }
-                nodeConnectorProps.put(nodeConnector, propMap);
-                break;
-            case REMOVED:
-                nodeConnectorProps.remove(nodeConnector);
-                break;
-            default:
-                return;
             }
+            nodeConnectorProps.put(nodeConnector, propMap);
+            break;
+        case REMOVED:
+            nodeConnectorProps.remove(nodeConnector);
+            break;
+        default:
+            return;
         }
 
         // update sal and discovery
@@ -260,9 +255,10 @@ public class InventoryService implements IInventoryShimInternalListener,
     }
 
     private void addNode(Node node, Set<Property> props) {
-        logger.trace("{} added", node);
-        if (nodeProps == null)
+        logger.trace("{} added, props: {}", node, props);
+        if (nodeProps == null) {
             return;
+        }
 
         // update local cache
         Map<String, Property> propMap = new HashMap<String, Property>();
@@ -305,26 +301,31 @@ public class InventoryService implements IInventoryShimInternalListener,
         }
     }
 
-    /*
-     * Function called by other protocol plugin modules to notify Inventory Service
-     * that a property has changed for the specified switch
-     */
-    @Override
-    public void updateSwitchProperty(Long switchId, Set<Property> propSet) {
-        // update local cache
-        Node node = OFSwitchToNode(controller.getSwitches().get(switchId));
-        Map<String, Property> propMap = nodeProps.get(node);
-        if (propMap == null)
-            propMap = new HashMap<String, Property>();
-        for (Property prop : propSet) {
-            propMap.put(prop.getName(), prop);
+    private void updateNode(Node node, Set<Property> properties) {
+        logger.trace("{} updated, props: {}", node, properties);
+        if (nodeProps == null || !nodeProps.containsKey(node) ||
+                properties == null || properties.isEmpty()) {
+            return;
         }
-        nodeProps.put(node, propMap);
 
-        // update sal
-        synchronized (pluginOutInventoryServices) {
-            for (IPluginOutInventoryService service : pluginOutInventoryServices) {
-                service.updateNode(node, UpdateType.CHANGED, propSet);
+        // Update local cache with new properties
+        Set<Property> newProperties = new HashSet<Property>(properties.size());
+        Map<String, Property> propertyMap = nodeProps.get(node);
+        for (Property property : properties) {
+            String name = property.getName();
+            Property currentProperty = propertyMap.get(name);
+            if (!property.equals(currentProperty)) {
+                propertyMap.put(name, property);
+                newProperties.add(property);
+            }
+        }
+
+        // Update SAL if we got new properties
+        if (!newProperties.isEmpty()) {
+            synchronized (pluginOutInventoryServices) {
+                for (IPluginOutInventoryService service : pluginOutInventoryServices) {
+                    service.updateNode(node, UpdateType.CHANGED, newProperties);
+                }
             }
         }
     }
@@ -338,8 +339,12 @@ public class InventoryService implements IInventoryShimInternalListener,
         case REMOVED:
             removeNode(node);
             break;
+        case CHANGED:
+            updateNode(node, props);
+            break;
         default:
             break;
         }
     }
+
 }