[OPNFLWPLUG-531] Li: Enable Yang notifications for backward compatibility Created: 11/Aug/15  Updated: 27/Sep/21  Resolved: 22/Jul/16

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

Type: Bug
Reporter: Kamal Rameshan Assignee: Kamal Rameshan
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

External issue ID: 4117

 Description   

Applications have been created over He plugin and are subscribed to yang notifications

These apps will have a hard time moving to the new Li plugin, if we do not support yang notifications.

In a sense of backward compatibility notifications with respect to node and node-connectors should be exposed.

They can be deprecated and slowly tapered off.

 Comments   
Comment by Vaclav Demcak [ 17/Aug/15 ]

We need new Project in Openflow. Project has to be marked as depricated, because
it has to exist as long as long He-plugin exist only. Notification emiter has to
be implemented as DataChangeListener for a relevant wildcarded path in DataStore.

Supported notification yang files (default):
path: controller/opendaylight/md-sal/model/model-inventoty
aug: openflowplugin/model/model-flow-service
Node (wildcardPath: InstanceIdentifier.create(Nodes.class).child(Node.class).augmentation(FlowCapableNode.class)):
(create/update notif)
org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeUpdated
org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNodeUpdated
(remove notif)
org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeRemoved

NodeConnector (wildcardPath: InstanceIdentifier.create(Nodes.class).child(Node.class).child(NodeConnector.class).augmentation(FlowCapableNodeConnector.class)):
(create/update notif)
org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeConnectorUpdated
org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNodeConnectorUpdated
(remove notif)
org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeConnectorRemoved

Supported notification (choose parameter by configSubsystem) TODO: needs to be checked with comunity
Port - PortUpdata & PortRemoved
Flow - FlowAdded & FlowUpdated & FlowRemoved & SwitchFlowRemoved
Meter - MeterAdded & MeterUpdated & MeterRemoved
Group - GroupAdded & GroupUpdated & GroupRemoved
GroupStat - GroupStatisticsUpdated & GroupDescStatsUpdated & GroupFeaturesUpdated
MeterStat - MeterStatisticsUpdated & MeterConfigStatsUpdated & MeterFeaturesUpdated
NodeConnector - NodeConnectorStatisctisUpdate
FlowStat - FlowsStatisticsUpdate & AggregateFlowStatisticsUpdate
QueueStat - QueueStatisticsUpdate
FlowTableStat - FlowTableStatisticsUpdate

Comment by Vaclav Demcak [ 28/Aug/15 ]

https://git.opendaylight.org/gerrit/#/c/25965/

Comment by Michal Rehak [ 13/Oct/15 ]

https://git.opendaylight.org/gerrit/#/q/status:open+project:openflowplugin+branch:stable/lithium+topic:BUG-4117

  • huge code has been divided into smaller parts
Comment by Kamal Rameshan [ 10/Mar/16 ]

do we need to implement this, as we have decided that applications need to make changes to listen to DCNs?

Comment by Abhijit Kumbhare [ 11/Apr/16 ]

Kamal,

The applications internal to ODL may make the changes to DCNs - but the TSC thinks the applications/users outside of ODL may be dependent on these & wants to make sure the API compatibility are maintained (and hence this change is needed for backward compatibility).

Abhijit

Comment by Shuva Jyoti Kar [ 20/May/16 ]

https://git.opendaylight.org/gerrit/#/q/project:openflowplugin+branch:master+topic:BUG-4117

Comment by Shuva Jyoti Kar [ 21/May/16 ]

renaming notification supplier:
https://git.opendaylight.org/gerrit/#/c/39204/

Comment by Shuva Jyoti Kar [ 26/May/16 ]

will move the data change listeners in notification-supplier to data tree change listeners

Comment by Andrej Leitner [ 21/Jul/16 ]

merged already in boron

Comment by Shuva Jyoti Kar [ 21/Jul/16 ]

Already merged
https://git.opendaylight.org/gerrit/#/q/project:openflowplugin+status:merged+Bug-4117

pending

https://git.opendaylight.org/gerrit/#/c/39967/

Will close it off post that

Generated at Wed Feb 07 20:32:44 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.