On Tue, Sep 19, 2017 at 7:54 PM, Michael Vorburger <vorbur...@redhat.com> wrote:
> On Fri, Sep 15, 2017 at 6:42 PM, Abhijit Kumbhare <abhijitk...@gmail.com> > 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. > > 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... > Sure @Michael. Just reported the following two bugs for these exceptions. https://bugs.opendaylight.org/show_bug.cgi?id=9184 https://bugs.opendaylight.org/show_bug.cgi?id=9185 > > >> >> On Fri, Sep 15, 2017 at 5:28 AM, Michael Vorburger <vorbur...@redhat.com> >> wrote: >> >>> On Fri, Sep 8, 2017 at 8:28 AM, Jozef Bacigál < >>> jozef.baci...@pantheon.tech> 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:vorbur...@redhat.com] >>>> *Sent:* Thursday, September 7, 2017 3:07 PM >>>> *To:* Jozef Bacigál <jozef.baci...@pantheon.tech>; Anil Vishnoi < >>>> vishnoia...@gmail.com>; Vishnoi, Anil <avish...@brocade.com> >>>> *Cc:* Robert Varga <n...@hq.sk>; Tomáš Slušný >>>> <tomas.slu...@pantheon.tech>; Abhijit Kumbhare <abhijitk...@gmail.com>; >>>> openflowplugin-...@lists.opendaylight.org; >>>> controller-dev@lists.opendaylight.org; mdsal-...@lists.opendaylight.org >>>> *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 <vorbur...@redhat.com> >>>> wrote: >>>> >>>> On Wed, Sep 6, 2017 at 4:34 PM, Michael Vorburger <vorbur...@redhat.com> >>>> wrote: >>>> >>>> On Wed, Sep 6, 2017 at 3:59 PM, Jozef Bacigál < >>>> jozef.baci...@pantheon.tech> 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 >>>> vorbur...@redhat.com | IRC: vorburger @freenode | ~ = >>>> http://vorburger.ch >>>> >>> >>> >> >
_______________________________________________ controller-dev mailing list controller-dev@lists.opendaylight.org https://lists.opendaylight.org/mailman/listinfo/controller-dev
