andrejleitner Are you still looking at this bug ? If not, can you please un-assign it so we can assign it to someone else?
Avishnoi, Arunprakash I had a look at this. Andrej Leitner'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):
- a) Whenever the lldp pkt is received for a link (lldp interval confgured as 5s), the link is updated with the latest age
- 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 > currentTime), only then the entry is removed.
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.
There are still some failures in the job "openflowplugin-csit-1node-gate-flow-services-all-sodium" of openflowplugin CSIT for this review.
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.
ecelgp Could you please guide on how to run only a particular set of sets from the CSIT.
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.
ecelgpThanks a lot 
. But I have already figured out the reasons for the failures.