In-line. On Tue, Sep 19, 2017 at 7:24 AM, Michael Vorburger <[email protected]> wrote:
> On Fri, Sep 15, 2017 at 6:42 PM, Abhijit Kumbhare <[email protected]> > wrote: > >> Thanks Michael! Looks like Luis is running test-openflowplugin-core on >> the patch. Also I understand one of the 3 exceptions is fixed - there are >> the other two exceptions still remaining. With one major exception fixed, >> does this mean the priority can be reduced from blocker to critical? >> > > Abhijit, IMHO we would be causing less confusion ;) if we closed 9038 > (after it's gone into master also, that was forgotten...) and then had 2 > separate new bugs for those other exceptions. > Sure - no problem. > > Sridhar, were you going to open those 2 other new bugs? It doesn't matter > IMHO if you don't have the environment anymore to be able to provide more > logs in the short term - at least have the issues, so that the openfllow > plugin team can start pondering what to do about those, some time... > > >> >> On Fri, Sep 15, 2017 at 5:28 AM, Michael Vorburger <[email protected]> >> wrote: >> >>> On Fri, Sep 8, 2017 at 8:28 AM, Jozef Bacigál < >>> [email protected]> wrote: >>> >>>> Thanks Michael, thanks for information. :) >>>> >>>> >>>> >>>> I do investigations further for the Illegal state bug. >>>> >>> >>> Jozef, Abhijit, Anil: Have you seen https://bugs.opendaylight.org/ >>> show_bug.cgi?id=9038#c19 - great news - thanks a lot, Sridhar and >>> everyone involved! >>> >>> So... will you merge https://git.opendaylight.org/gerrit/#/c/62674/6 >>> ASAP, cherry-pick it from stable/carbon to stable/nitrogen and master, >>> and... done! >>> >>> >>>> But at least we got some issue solved :) >>>> >>>> >>>> >>>> Jozef >>>> >>>> >>>> >>>> P.S.: Let you know if something new >>>> >>>> >>>> >>>> *From:* Michael Vorburger [mailto:[email protected]] >>>> *Sent:* Thursday, September 7, 2017 3:07 PM >>>> *To:* Jozef Bacigál <[email protected]>; Anil Vishnoi < >>>> [email protected]>; Vishnoi, Anil <[email protected]> >>>> *Cc:* Robert Varga <[email protected]>; Tomáš Slušný >>>> <[email protected]>; Abhijit Kumbhare <[email protected]>; >>>> [email protected]; >>>> [email protected]; [email protected] >>>> *Subject:* Re: [mdsal-dev] [openflowplugin-dev] Fwd: Bug 9038 - >>>> IllegalStateException: Attempted to close chain with outstanding >>>> transaction PingPongTransaction at org.opendaylight.openflowplugi >>>> n.impl.device.TransactionChainManager.createTxChain >>>> >>>> >>>> >>>> All, >>>> >>>> >>>> >>>> On Wed, Sep 6, 2017 at 7:26 PM, Michael Vorburger <[email protected]> >>>> wrote: >>>> >>>> On Wed, Sep 6, 2017 at 4:34 PM, Michael Vorburger <[email protected]> >>>> wrote: >>>> >>>> On Wed, Sep 6, 2017 at 3:59 PM, Jozef Bacigál < >>>> [email protected]> wrote: >>>> >>>> HI Michael, >>>> >>>> are you able to test it on this patch [1] ? Or it needed to be merged ? >>>> >>>> Jozef - thanks a lot! I'll discuss with Daniel to understand how >>>> easy/hard it would be to get a carbon RPM with this, and let you know ASAP. >>>> >>>> >>>> >>>> Jozef, FYI we've just been able to get a new full ODL dist build + RPM >>>> incl. this (unmerge) change, and will re-run this through our scale test >>>> environment ASAP, and let you know how it looks. >>>> >>>> >>>> >>>> There is Good and Bad news - which one do you want first? ;-) >>>> >>>> The Good: I’m (very) happy to hereby confirm that on a re-test of a >>>> custom distribution of latest (yesterday's) carbon/stable together with >>>> c/62674/5 (NB Patch Set 5; it’s still WIP) pulled in, we do not see >>>> https://bugs.opendaylight.org/show_bug.cgi?id=9034 (OOM due to huge >>>> Map in ShardDataTree) anymore! This is great. NB: It may have been c/62674 >>>> or anything else that went in recently (anywhere incl. Bug 9028, dunno if >>>> that could have helped for this?) which did the trick, but it works now, so >>>> great! FYI this has been verified by both "wow, now it didn't blew up >>>> anymore with OOM" as well as confirmed via HPROF analysis through MAT >>>> (which now also revealed new Bug 9114; but that's another story). I'll >>>> therefore close blocker Bug 9034 as soon as you've agreed a final version >>>> of c/62674 and merged it (into carbon, nitrogen and master). >>>> >>>> The FYI: c/62674/5 does fix https://bugs.opendaylight.org/ >>>> show_bug.cgi?id=9070, so I'll also close that after you merge. >>>> >>>> >>>> >>>> The Bad: The https://bugs.opendaylight.org/show_bug.cgi?id=9038 >>>> "IllegalStateException: Attempted to close chain with outstanding >>>> transaction PingPongTransaction" still appears in logs - sorry. So >>>> c/62674/5 >>>> *DOES NOT* fix blocker Bug 9038. You can also find related information >>>> on the new attachment on umbrella https://bugs.opendaylight.org/ >>>> show_bug.cgi?id=9096 re. other Tx leaks. I'm hoping this information >>>> is useful to openflowplug-dev to find another solution for Bug 9038. >>>> >>>> >>>> Tx, >>>> >>>> M. >>>> >>>> PS: Wrote about some blurb about the new tooling which came out of this >>>> on http://blog2.vorburger.ch/2017/09/how-to-find-transaction-re >>>> lated-memory.html >>>> >>>> -- >>>> >>>> Michael Vorburger, Red Hat >>>> [email protected] | IRC: vorburger @freenode | ~ = >>>> http://vorburger.ch >>>> >>> >>> >> >
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
