[OPNFLWPLUG-57] Not handling switch reconnect and disconnect correctly Created: 26/Jan/14  Updated: 27/Sep/21  Resolved: 06/Feb/14

Status: Resolved
Project: OpenFlowPlugin
Component/s: General
Affects Version/s: None
Fix Version/s: None

Type: Improvement
Reporter: Ed Warnicke Assignee: Ed Warnicke
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: Mac OS
Platform: PC


 Description   

When a switch disconnects and reconnects, we are not handling the sync of configuration correctly.

A correct handling would involve a mark and sweep to bring the switch into sync with the controller. We do not have time to do this correctly.

A workable solution is:

1) On switch disconnect, in plugin, remove all of the flow, group, and meter for the node from the configuration tree.
a) This can be done either by removing the FlowCabableNode Augmentation from the node or by removing all the Tables,Groups, and Meters. It should not be done by removing the Node.
b) In the FlowTransaction, we will need to wrapper the call to the removeFlow RPC to catch the exception if the node is not services by a working switch.
2) On switch connect, in the plugin, send a message to the switch deleting all Flows, Groups, and Meters prior to the initial poll for stats (I'd recommend a barrier on this).

 Comments   
Comment by Michal Rehak [ 03/Feb/14 ]

https://git.opendaylight.org/gerrit/#/c/5001/1

Done for flows, meters and groups need to be added.

But removing flow from config context raises an exception based on invalid session in plugin (this should be caused by rendering flow removal to actual message for particular flow). But at the end - tables are gone, node remains.

Need to examine that closely.

Comment by Michal Rehak [ 06/Feb/14 ]
  • added meters ang groups
  • tested on:
  • cpqd (ok)
  • and ovs (ovs OF-1.3 has currently issues regarding adding meter, but removing flows works, groups are not supported, OF-1.0 supports flows only and it works)
Generated at Wed Feb 07 20:31:27 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.