Re: [openflowplugin-dev] openflowplugin bug-scub

[Andrej Leitner -X (anleitne - PANTHEON TECHNOLOGIES at Cisco)]

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: openflowplugin-dev-boun...@lists.opendaylight.org 
 on behalf of Shuva Jyoti 
Kar 
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
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Re: [openflowplugin-dev] openflowplugin bug-scub

[Abhijit Kumbhare]

Thanks Shuva!

On Thu, Jul 21, 2016 at 10:33 AM, Shuva Jyoti Kar <
shuva.jyoti@ericsson.com> wrote:

> 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
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev