https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/OPNFLWPLUG-1027 OpenFlowPlugin <p>In the current implementation of the TopologyManager, it's listening for linkAdded and linkRemoved notification and whenever it receive those notification, it updates the data store. linkAdded and linkRemoved notificaitons are published by topology-lldp-discovery module. Topology-lldp-discovery module will publish this whenever it will receive the lldp packet from the switch. This is not a right behavior, because it's publishing the same linkAdded event everytime it receives the lldp packet. This can cause performance issue when topology is large.</p> <p>Topology-lldp-discovery module should only publish the linkAdded notification if the link is added first time or removed and added back.</p> OPNFLWPLUG-1027

Topology manager writes link information everytime topology-lldp-discovery publish the linkAdded/linkRemove notificaiton

Bug High Resolved Done Gobinath Suganthan Anil Vishnoi Thu, 12 Jul 2018 02:05:38 +0000 Mon, 17 Jun 2019 15:49:12 +0000 Mon, 17 Jun 2019 15:49:12 +0000 Neon Sodium topology-lldp-discovery topology-manager 0 3 <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=andrejleitner" class="user-hover" rel="andrejleitner">andrejleitner</a> Are you still looking at this bug ? If not, can you please un-assign it so we can assign it to someone else?</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=Avishnoi" class="user-hover" rel="Avishnoi">Avishnoi</a>, <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=Arunprakash" class="user-hover" rel="Arunprakash">Arunprakash</a> I had a look at this. <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=andrejleitner" class="external-link" rel="nofollow">Andrej Leitner</a>'s patch had some CSIT failures related to topology. In the patch, the "linkToDate" cache (already existing) is used to decide whether to send the notification. So, the "linkAdded" notification is sent only when the "linkToDate" cache doesnt have the discovered link. This cache is regularly updated in the following manner(LinkAger):</p> <ol> <li>a) Whenever the lldp pkt is received for a link (lldp interval confgured as 5s), the link is updated with the latest age</li> <li>b) All the entries are checked for every interval (topology-lldp-expiration-interval which is currently 60s). In the event of the age being expired (time link updated + expiration-interval &gt; currentTime), only then the entry is removed.</li> </ol> <p> </p> <p>So, even after the link is removed it would take 60s for the link to e removed from the cache. But if we dont wait for this removal and add the link back, the cache would contain the link after the expiry. So no lldp would be sent thinking that the link was never removed in the first place. So the flapped links wont be discovered.</p> <p> </p> <p>There are still some failures in the job "openflowplugin-csit-1node-gate-flow-services-all-sodium" of openflowplugin CSIT for this review.</p> <p>I added some logging to debug them but I'm finding difficult to isolate the issues as similar TCs are repeated across the suite and the job itself is time consuming. It would be easier to debug if only the failed TCs can be run.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=ecelgp" class="user-hover" rel="ecelgp">ecelgp</a> Could you please guide on how to run only a particular set of sets from the CSIT.</p> <p>Tell me what you want to run as test and I can prepare test patch and job in sandbox you can use to validate your patch.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=ecelgp" class="user-hover" rel="ecelgp">ecelgp</a>Thanks a lot <img class="emoticon" src="https://jira.opendaylight.org/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/>. But I have already figured out the reasons for the failures.</p> Development Rank 0|i03gpr: