Fix container flow logic in forwarding rules manager 45/545/1
authorAlessandro Boch <aboch@cisco.com>
Tue, 2 Jul 2013 21:13:06 +0000 (14:13 -0700)
committerAlessandro Boch <aboch@cisco.com>
Tue, 2 Jul 2013 21:13:06 +0000 (14:13 -0700)
commit162da091eb6080acbf0128c27124eea1389c687c
tree4f9d5f3712d564fcacd46756a7799733b1ac6fce
parenta881650131c718f5189d975e95f44b3f4450903a
Fix container flow logic in forwarding rules manager

- In Activator.java:
   + All instances of FRM must be IContainerListener
   + Pass the right cache name in order to have the save config event working in the cluster
- In ForwardingRulesManaerImpl.java:
   + Add container check filtering in the IContainerListener methods
   + In updateFlowsContainerFlow() first uninstall all current flows, then reinstall instead
     of interleave remove/add which could cause conflict failures during installation phase
     as the new entries could conflict with existing old container flow merged flows present
     on the netwrok node
   + minor logs and style changes
- In FlowEntryInstall.java:
   + Fixed a bug where container flow merging was altering the original flow entry

Signed-off-by: Alessandro Boch <aboch@cisco.com>
opendaylight/forwardingrulesmanager/api/src/main/java/org/opendaylight/controller/forwardingrulesmanager/FlowEntryInstall.java
opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/Activator.java
opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/ForwardingRulesManagerImpl.java