Sure will take care of it as an enhancement From: Andrej Leitner -X (anleitne - PANTHEON TECHNOLOGIES at Cisco) [mailto:[email protected]] Sent: Friday, July 22, 2016 11:29 AM To: Shuva Jyoti Kar; Abhijit Kumbhare Cc: openflowplugin-dev Subject: Re: openflowplugin bug-scub
Hi Shuva, your patch works for me for now, but consider implementing BindingAwareProvider with onSessionInitiated(ProviderContext) method. You can find this approach used in org.opendaylight.openflowplugin.applications.frsync.impl.ForwardingRulesSyncProvider. More elegant (if can be used) than SimpleTaskRetryLooper I think. -al- ________________________________ From: [email protected]<mailto:[email protected]> <[email protected]<mailto:[email protected]>> on behalf of Shuva Jyoti Kar <[email protected]<mailto:[email protected]>> Sent: Thursday, July 21, 2016 7:33 PM To: Abhijit Kumbhare Cc: openflowplugin-dev Subject: [openflowplugin-dev] openflowplugin bug-scub Hi Devs, I have updated the bugzilla with the status for Bug4117. Only one review is pending to be merged. https://git.opendaylight.org/gerrit/#/c/39967/ - Probably Jozef/Andrej can review. Regarding node-added notifications I donot seem to find them . Went through 6117, I am looking at introducing inJeopardy() into the older-plugin perspective(working on a change). [thanks Anil for correcting me, it wasn't the issue I was thinking, my bad] It would be great if someone could look into it for the lithium-plugin which already supports inJeopardy(). Need to determine whether it's a base clustering problem. Also for the unclassified bugs, did a quick analysis, we need to test and verify few of those(mostly they have been raised on the He-plugin), probably will put a note in the xcel sheet. A few of them have been fixed, will update the Bugzilla. Thanks Shuva
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
