Yes Vishal !!! Its done :)
Br,shuva On Tue, Jun 20, 2017 at 9:34 PM, Vishal Thapar <[email protected]> wrote: > OFP devs, can we get this merged? > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *N Vivekanandan > *Sent:* 20 June 2017 20:03 > *To:* Sam Hague <[email protected]> > *Cc:* odl netvirt dev <[email protected]>; Jamo Luhrsen < > [email protected]>; Tomáš Slušný <[email protected]>; > [email protected]; Michael Vorburger < > [email protected]> > *Subject:* Re: [netvirt-dev] [openflowplugin-dev] > NxActionRegLoadNodesNodeTableFlowApplyActionsCase for version 4 not found. > > > > Hi Sam, > > > > *From:* Sam Hague [mailto:[email protected] <[email protected]>] > *Sent:* Tuesday, June 20, 2017 7:40 PM > *To:* N Vivekanandan <[email protected]> > *Cc:* Michael Vorburger <[email protected]>; odl netvirt dev < > [email protected]>; Tomáš Slušný < > [email protected]>; [email protected]; > Jamo Luhrsen <[email protected]> > *Subject:* RE: [openflowplugin-dev] > NxActionRegLoadNodesNodeTableFlowApplyActionsCase > for version 4 not found. > > > > > > > > On Jun 20, 2017 8:25 AM, "N Vivekanandan" <[email protected]> > wrote: > > Thanks a lot Tomas/Michael.. > > > > We hope Nitrogen CSITs to pass out going forward, so I fired a NetVirt > CSIT run for the review: > > This likely doesn't work, since it uses a integration distro which does > have NetVirt in it yet. Jamo has a workaround to use the NetVirt distro but > that is only for the NetVirt periodic job and not the gate job. > > > > Thanks for this clarification.. > > Is there an easier way to see if this change fixes the problem? > > > > -- > > Thanks, > > > > Vivek > > https://git.opendaylight.org/gerrit/#/c/59214/ > > > > -- > > Thanks, > > > > Vivek > > > > *From:* Michael Vorburger [mailto:[email protected]] > *Sent:* Tuesday, June 20, 2017 5:26 PM > *To:* N Vivekanandan <[email protected]>; Sam Hague < > [email protected]>; Jamo Luhrsen <[email protected]> > *Cc:* Tomáš Slušný <[email protected]>; odl netvirt dev < > [email protected]>; openflowplugin-dev@lists. > opendaylight.org > *Subject:* Re: [openflowplugin-dev] > NxActionRegLoadNodesNodeTableFlowApplyActionsCase > for version 4 not found. > > > > Vivek, Sam, Jamo: > > on this thread, NB Tomas' https://git.opendaylight.org/gerrit/#/c/59214/ > ... for https://bugs.opendaylight.org/show_bug.cgi?id=8718. > > Tx a lot Tomas! > > > > Tx, > > M. > > -- > > Michael Vorburger, Red Hat > [email protected] | IRC: vorburger @freenode | ~ = http://vorburger.ch > > > > On Mon, Jun 19, 2017 at 4:11 PM, Michael Vorburger <[email protected]> > wrote: > > > > On Mon, Jun 19, 2017 at 3:06 PM, Tomáš Slušný <[email protected]> > wrote: > > Hi Vivek, > > it looks like feature 'odl-openflowplugin-nxm-extensions' was not > installed properly on your side for some reason, because serializer for > this interface seen in warning message > > is on latest Nitrogen and on Carbon registered properly in > NiciraExtensionProvider. For this part of plugin, it should work in same > way on both Nitrogen and Carbon, so maybe > > there is another issue that came with Karaf 4 here. > > a quick check in netvirt/vpnservice/features/odl-netvirt-api/pom.xml > shows that odl-openflowplugin-nxm-extensions *IS* listed as a dependency, > so it's not like that was just "forgotten" in the Karaf 4 feature > migration... would have been nice if it was that easy. ;) > > I just had a quick look through the [1] logs (linked in below), and can > spot lines like this, so it looks like that feature did get installed, > doesn't it: > > 2017-06-18 13:42:00,807 | INFO | pool-2-thread-1 | > FeaturesServiceImpl | 9 - org.apache.karaf.features.core - > 4.0.9 | mvn:org.opendaylight.openflowplugin/openflowplugin- > extension-nicira/0.5.0-SNAPSHOT > > So this probably means that something is broken with the registration of > that serializer (whatever that actually is...), specifically under Karaf 4, > could that be? Any tips how to pursue on this, like perhaps what to grep > for or what additional logs to enable to see more about the registration of > such serializers, does that make any sense? Where in the code does that > happen? > > [1] https://logs.opendaylight.org/releng/jenkins092/netvirt- > csit-1node-openstack-newton-nodl-v2-upstream-stateful- > nitrogen/69/odl1_karaf.log.gz > > > > Regards, > > Tomas Slusny > > > ------------------------------ > > *Od:* N Vivekanandan <[email protected]> > *Odoslané:* pondelok, 19. júna 2017 14:43 > *Komu:* Sam Hague; odl netvirt dev; Jamo Luhrsen; > [email protected] > *Predmet:* Re: [openflowplugin-dev] > NxActionRegLoadNodesNodeTableFlowApplyActionsCase > for version 4 not found. > > > > 'included openflowplugin-dev > > -----Original Message----- > From: N Vivekanandan > Sent: Monday, June 19, 2017 6:12 PM > To: 'Sam Hague' <[email protected]>; odl netvirt dev <netvirt-dev@lists. > opendaylight.org>; Jamo Luhrsen <[email protected]> > Subject: RE: NxActionRegLoadNodesNodeTableFlowApplyActionsCase for > version 4 not found. > > Hi Sam, > > have you seen the below warning before? > No, I have not seen this warning in Karaf 3 runs. > Here is the latest karaf log for stable/carbon run which is on Karaf 3 and > that doesn't have the warning: > https://logs.opendaylight.org/releng/jenkins092/netvirt- > csit-1node-openstack-newton-nodl-v2-upstream-stateful- > carbon/703/odl1_karaf.log.gz > > Any idea what it is? it comes out repeatedly in the csit [1]. > > Looks to be an OFPlugin breakage for OpenFlow version 1.4 Nicira > ResubmitAction. > We could request Openflowplugin folks to look into this : > > 2017-06-18 13:49:43,284 | WARN | entLoopGroup-7-1 | > ActionUtil | 343 - org.opendaylight.openflowplugin.impl > - 0.5.0.SNAPSHOT | Serializer for action interface > org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension. > nicira.action.rev140714.nodes.node.table.flow.instructions. > instruction.instruction.apply.actions._case.apply.actions.action.action. > NxActionResubmitNodesNodeTableFlowApplyActionsCase for version 4 not > found. > > -- > Thanks, > > Vivek > > -----Original Message----- > From: Sam Hague [mailto:[email protected] <[email protected]>] > Sent: Monday, June 19, 2017 4:34 PM > To: odl netvirt dev <[email protected]>; N Vivekanandan < > [email protected]>; Jamo Luhrsen <[email protected]> > Subject: NxActionRegLoadNodesNodeTableFlowApplyActionsCase for version 4 > not found. > > Vivek, > > have you seen the below warning before? Any idea what it is? it comes out > repeatedly in the csit [1]. This is the karaf 4 job with the 77 errors. > Wonder if it is related? There is a later flow datastore exception that > might be related. > > 2017-06-18 14:32:25,116 | WARN | entLoopGroup-7-1 | ActionUtil > | 343 - org.opendaylight.openflowplugin.impl - > 0.5.0.SNAPSHOT | Serializer for action interface > org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension. > nicira.action.rev140714.nodes.node.table.flow.instructions. > instruction.instruction.apply.actions._case.apply.actions.action.action. > NxActionRegLoadNodesNodeTableFlowApplyActionsCase > for version 4 not found. > > Thanks ,Sam > > [1] https://logs.opendaylight.org/releng/jenkins092/netvirt- > csit-1node-openstack-newton-nodl-v2-upstream-stateful- > nitrogen/69/odl1_karaf.log.gz > _______________________________________________ > openflowplugin-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev > > > > *Tomáš Slušný* > > Software Developer > > > > *PANTHEON* *technologies s.r.o.* > > Janka Kráľa 9, 974 01 Banská Bystrica > > Slovakia > > Tel / +421 220 665 111 > > > > *MAIL **/* [email protected] > > *WEB **/* https://pantheon.tech > > > > > > > _______________________________________________ > 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 > >
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
