[CONTROLLER-595] Plugins that register for DataChangeListener get notifications for all nodes and not just the nodes they registered for. Created: 02/Jul/14  Updated: 14/Jul/14  Due: 11/Jul/14  Resolved: 14/Jul/14

Status: Resolved
Project: controller
Component/s: mdsal
Affects Version/s: Helium
Fix Version/s: None

Type: Bug
Reporter: Ramkumar Gowrishankar Assignee: Tony Tkacik
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: Linux
Platform: PC

Attachments: File pingDataListener.tar.gz    
External issue ID: 1284

 Description   

When I register a datachangelistener on a specific node (for example node1 that is defined in the sample-pingDiscovery module), the onDataChanged gets called for all types of nodes (Netconf, flowCapableNode).

It should get called only for the type of node that I registered for.

I have attached a sample module to this bug report. Un-tar the attached file in the controller/opendaylight directory, compile and copy the jar to the plugins folder and run the pingDiscovery command from https://git.opendaylight.org/gerrit/#/c/7249/16.

 Comments   
Comment by Ramkumar Gowrishankar [ 02/Jul/14 ]

Attachment pingDataListener.tar.gz has been added with description: pingdataListener module

Comment by Tony Tkacik [ 04/Jul/14 ]

Could you be more specific about this? Does it happen only with new APIs?

Because we have test-suite and components which listens only
to Openflow augmentations and ones which listens to Netconf Augmentations
and are never invoked when other one is added.

Also created a testsuite (which wrote Netconf augmentation, was listening for Openflow) and event was not triggered on Netconf augmentation write.

Comment by Tony Tkacik [ 04/Jul/14 ]

Further analysis showed that this happens only if augmentation
you are listening for has leaf-nodes only. If augmentation you want to listen for has containers / and or lists, you will not get triggered on all changes of augmented node.

Comment by Ramkumar Gowrishankar [ 08/Jul/14 ]

Tony, do you still want me to test with the new framework? It will take me a day or so to get to it since I do not have a single workspace with the latest framework and OpenFlow 1.3.

Comment by Tony Tkacik [ 09/Jul/14 ]

Bugfix is in: https://git.opendaylight.org/gerrit/8865

Generated at Wed Feb 07 19:53:25 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.