I believe upstream openflowplugin CSIT uses 2.8.0/1 version and that did
not contain the new NSH support and that's working fine. Seems like the new
NSH support is causing this issue. Looking like in your CSIT someone is
trying to install the flow with the NSH match or your switch has NSH based
flow. MatchConvertor comes into play when either you are pushing flow down
to the switch or reading and deserializing the stats/packet_in coming from
switch.

We already have patches pushed by Jamie to implement these new NSH fields
and remove the old one, hopefully that should fix it.

On Sun, Jul 8, 2018 at 5:26 AM Sam Hague <sha...@redhat.com> wrote:

> Adding Daya to see if they have been doing any testing with ovs 2.8.2 or
> later. Also forgot netvirt-dev and openflowplugin-dev lists.
>
> On Sat, Jul 7, 2018 at 5:05 PM Sam Hague <sha...@redhat.com> wrote:
>
>> Jamo,
>>
>> did we ever have any runs using ovs 2.8.2 - or really anything later than
>> 2.7.2? [1] is a job using ovs 2.8.2 and it fails a few tests because of an
>> openflow deserialization error with some NSH packets - but this is just our
>> normal csit and not anything related to sfc. My 2.9 jobs also had some
>> failures but I didn't inspect them before the sandbox cleared up this
>> weekend.
>>
>> NSH support was added in 2.8 so that makes sense this could cause a
>> problem. The exception below is coming out when the test fails - a ping
>> fails, so I guess an arp or other punt packet is being mapped to nsh and
>> not being decoded by openflowplugin.
>>
>> Jaime, Venkat,
>>
>> have you seen issues like this? Is it because openflowplugin has not been
>> updated to use the new nsh and maybe some older nsh support is causing
>> problems?
>>
>> Thanks, Sam
>>
>> [1]
>> https://jenkins.opendaylight.org/sandbox/job/shague-queens-netvirt-csit-1node-openstack-queens-upstream-stateful-fluorine/3/
>>
>> 2018-07-07T15:28:56,461 | WARN  | epollEventLoopGroup-9-1 | 
>> MatchExtensionHelper             | 364 - org.opendaylight.openflowplugin - 
>> 0.7.0.SNAPSHOT | Convertor for 
>> MatchEntry{_matchEntryValue=NshNpCaseValue{_nshNpValues=NshNpValues{_value=41,
>>  augmentation=[]}, augmentation=[]}, _oxmClass=interface 
>> org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.Nxm1Class,
>>  _oxmMatchField=interface 
>> org.opendaylight.yang.gen.v1.urn.opendaylight.openflowjava.nx.match.rev140421.NxmNxNshNp,
>>  _hasMask=false, augmentation=[]} for version 4 with match path 
>> PACKET_IN_MESSAGE_MATCH not found.
>> 2018-07-07T15:28:56,462 | WARN  | epollEventLoopGroup-9-1 | OFDecoder        
>>                 | 386 - 
>> org.opendaylight.openflowplugin.openflowjava.openflow-protocol-impl - 
>> 0.7.0.SNAPSHOT | Message deserialization 
>> failedjava.lang.NullPointerException: null       at 
>> org.opendaylight.openflowplugin.openflow.md.core.extension.MatchExtensionHelper.injectExtension(MatchExtensionHelper.java:83)
>>  [364:org.opendaylight.openflowplugin:0.7.0.SNAPSHOT]   at 
>> org.opendaylight.openflowplugin.impl.protocol.deserialization.match.MatchDeserializer.deserializeEntry(MatchDeserializer.java:104)
>>
>>
>> _______________________________________________
> netvirt-dev mailing list
> netvirt-...@lists.opendaylight.org
> https://lists.opendaylight.org/mailman/listinfo/netvirt-dev
>


-- 
Thanks
Anil
_______________________________________________
openflowplugin-dev mailing list
openflowplugin-dev@lists.opendaylight.org
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Reply via email to