Tomas, looks like the patch fix has introduced a different openflowplugin issue, so not sure if the original issue is fixed. Can you take a look at that? Is [2] a patch that would help there? If so, should we add both patches to a gate job to verify the issues are fixed?
Thanks, Sam [2] https://git.opendaylight.org/gerrit/#/c/60243/3 2017-07-12 10:04:15,998 | WARN | entLoopGroup-9-1 | OFDecoder | 345 - org.opendaylight.openflowplugin.openflowjava.openflow-protocol-impl - 0.5.0.SNAPSHOT | Message deserialization failed java.lang.ClassCastException: Cannot cast org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.flow.MatchBuilder to org.opendaylight.yang.gen.v1.urn.opendaylight.packet.service.rev130709.packet.in.message.MatchBuilder On Wed, Jul 12, 2017 at 5:57 AM, Tomáš Slušný <[email protected]> wrote: > Here is patch that should solve the issue: https://git. > opendaylight.org/gerrit/#/c/60234/. Currently netvirt CSIT tests are > running on it, so hopefully they will pass. > > > Regards, > > Tomas > > > ------------------------------ > *Od:* Sam Hague <[email protected]> > *Odoslané:* utorok, 11. júla 2017 23:30 > *Komu:* Tomáš Slušný > *Kópia:* Jozef Bacigál; odl netvirt dev; openflowplugin-dev > > *Predmet:* Re: [openflowplugin-dev] openflowplugin regression on master > introduced last couple days > > Great, Thanks for finding this. You can add test-openflow-netvirt as a > comment on the patch to fire off the csit to see if it fixes the issue. Or > just commit the fix and we will run csit. > > On Tue, Jul 11, 2017 at 1:54 PM, Tomáš Slušný <[email protected]> > wrote: > >> Yes, it is caused by new packet in serializer I implemented in Nitrogen. >> I will create patch tomorrow that will resolve this. Sorry for trouble. >> >> Regards, >> Tomas >> >> ------------------------------ >> *Od:* Jozef Bacigál <[email protected]> >> *Odoslané:* utorok, 11. júla 2017 16:36 >> *Komu:* Sam Hague; odl netvirt dev; openflowplugin-dev >> *Predmet:* Re: [openflowplugin-dev] openflowplugin regression on master >> introduced last couple days >> >> >> Yes it would. Let us check it. >> >> >> Jozef >> >> ------------------------------ >> *Od:* Sam Hague <[email protected]> >> *Odoslané:* utorok, 11. júla 2017 15:50:37 >> *Komu:* odl netvirt dev; openflowplugin-dev >> *Predmet:* [openflowplugin-dev] openflowplugin regression on master >> introduced last couple days >> >> Would any of the patches merged in the past couple days on master >> openflowplugin cause exceptions like below? >> >> The netvirt csit [1] is failing with these issues only on master. carbon >> is passing 100%. The code in netvirt is the same on master and carbon since >> the branch has been locked. >> >> 017-07-10 23:52:22,632 | WARN | entLoopGroup-9-2 | OFDecoder >> | 328 - org.opendaylight.openflowjava.openflow-protocol-impl - >> 0.10.0.SNAPSHOT | Message deserialization failed >> org.opendaylight.openflowplugin.extension.vendor.nicira.convertor.CodecPreconditionException: >> Augmentation for path PACKETINMESSAGE_MATCH is not implemented. >> at >> org.opendaylight.openflowplugin.extension.vendor.nicira.convertor.match.RegConvertor.resolveAugmentation(RegConvertor.java:133)[341:org.opendaylight.openflowplugin.extension-nicira:0.5.0.SNAPSHOT] >> >> >> [1] https://jenkins.opendaylight.org/releng/view/netvirt- >> csit/job/netvirt-csit-1node-openstack-newton-nodl-v2- >> upstream-stateful-nitrogen/117/ >> >> Jozef Bacigál >> Senior Software Engineer >> >> 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 >> >> >> >> 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 >> >> >> > > > 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
