Shuva, I have +2'ed it but there is a build failure (so I cannot merge) - though I don't see why we should have a build failure on this small change. Once there is no build failure - you can yourself merge it pointing to my +2.
Thanks, Abhijit On Sun, Aug 7, 2016 at 2:35 AM, Shuva Jyoti Kar < [email protected]> wrote: > Alon, > > Have pushed a patch for [3]. > https://git.opendaylight.org/gerrit/#/c/43277/ > > thanks > shuva > > -----Original Message----- > From: Kochba, Alon [mailto:[email protected]] > Sent: Sunday, August 07, 2016 1:20 PM > To: Jamo Luhrsen; Luis Gomez; Shuva Jyoti Kar > Cc: [email protected]; openflowplugin-dev@lists. > opendaylight.org > Subject: RE: [netvirt-dev] [openflowplugin-dev] problem reading table > feature propery message? > > Hi all, > > I wanted to reiterate this issue - this message is really flooding the > logs. > I see the bug to support this [1] is still opened, and a bug for log > flooding was closed [2]. > > root@devstack6-1:~# grep "Problem during reading table feature property" > /opt/stack/logs/screen-karaf.log | wc -l > 13716 > > This is on a devstack setup with two compute nodes - is there any reason > not to change this to DEBUG? > The default debug levels should not have flooded logs in my opinion. It > hides other important issues. > > On a related note, this message is also flooding the log very badly, I > opened bug [3] (it was introduced in Anil's commit [4]): > 2016-08-07 10:45:03,262 | INFO | entLoopGroup-9-6 | > DeviceFlowRegistryImpl | 279 - org.opendaylight.openflowplugin.impl > - 0.3.0.SNAPSHOT | Triggered the loop > > root@devstack6-1:~# grep "Triggered the loop" > /opt/stack/logs/screen-karaf.log | wc -l > 506775 > > [1] https://bugs.opendaylight.org/show_bug.cgi?id=4473 > [2] https://bugs.opendaylight.org/show_bug.cgi?id=5173 > [3] https://bugs.opendaylight.org/show_bug.cgi?id=6354 > [4] https://git.opendaylight.org/gerrit/#/c/42510/ > > --alon > > > -----Original Message----- > From: [email protected] [mailto: > [email protected]] On Behalf Of Jamo Luhrsen > Sent: Wednesday, 13 July 2016 19:42 > To: Luis Gomez <[email protected]>; Shuva Jyoti Kar < > [email protected]> > Cc: [email protected]; openflowplugin-dev@lists. > opendaylight.org > Subject: Re: [netvirt-dev] [openflowplugin-dev] problem reading table > feature propery message? > > this makes it miserable for me to debug what I think could be a legit bug > in netvirt. > > :/ > > > > On 07/13/2016 09:38 AM, Luis Gomez wrote: > > AFAIR, the logs are generated by a lack of code implementation in > openflowplugin, the solutions are: 1) write the code or 2) hide the ERRORs. > In Be release there was no time for 1) and some devs did not like 2). I > hope in Boron we can do better, specially looking at the ofplugin users > experience. > > > > BR/Luis > > > > > > > >> On Jul 13, 2016, at 3:02 AM, Shuva Jyoti Kar < > [email protected]> wrote: > >> > >> I am aware of the discussions that happened as a part of > https://bugs.opendaylight.org/show_bug.cgi?id=4473 > >> > >> > >> Thanks > >> Shuva > >> > >> -----Original Message----- > >> From: [email protected] [mailto: > [email protected]] On Behalf Of Jamo > Luhrsen > >> Sent: Wednesday, July 13, 2016 11:59 AM > >> To: Luis Gomez; [email protected]; > [email protected] > >> Subject: [openflowplugin-dev] problem reading table feature propery > message? > >> > >> Hi OFP, > >> > >> I forget the reason we are getting a lot of these messages in our log? > I know I've been told before, but can you remind me please :) > >> > >> 2016-07-13 05:50:47,758 | WARN | entLoopGroup-9-2 | ListDeserializer > | 272 - > >> org.opendaylight.openflowjava.openflow-protocol-impl - 0.8.0.SNAPSHOT > | Problem during reading table feature property. > >> Skipping unknown feature property: msgVersion: 4 objectClass: > >> org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm. > rev150225.match.entries.grouping.MatchEntry msgType: 1 oxm_field: > >> 31 experimenterID: null.If more information is needed, set > org.opendaylight.openflowjava do DEBUG log level. > >> > >> I'm seeing it a bunch in a job I'm trying to stabilize for > netvirt-vpnservice which involves devstack on the mitaka branch. > >> > >> Thanks, > >> JamO > >> _______________________________________________ > >> openflowplugin-dev mailing list > >> [email protected] > >> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev > > > _______________________________________________ > netvirt-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/netvirt-dev > _______________________________________________ > openflowplugin-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev >
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
