[NETVIRT-654] New netvirt classifier not setting tun id to 0 when egressing to SFF Created: 05/May/17 Updated: 12/Aug/17 Resolved: 12/Aug/17 |
|
| Status: | Resolved |
| Project: | netvirt |
| Component/s: | General |
| Affects Version/s: | Carbon |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Jaime Caamaño Ruiz | Assignee: | Jaime Caamaño Ruiz |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 8387 |
| Description |
|
SFC terminating service action matches by a fixed tunnel key of 0, and classifier needs to ensure this is properly set. Currently, the classifier is not rewriting the tunnel key, thus the tunnel key set by netvirt related to the original destination of the packet remains. This is a sample of the ACL flow where the tunnel key would be set but is not: cookie=0xf005ba1100000002, duration=989.743s, table=101, n_packets=7, n_bytes=518, priority=500,tcp,in_port=2,nw_src=10.0.0.0/24,tp_dst=80 actions=push_nsh,load:0x1->NXM_NX_NSH_MDTYPE[],load:0x3->NXM_NX_NSH_NP[],load:0x8->NXM_NX_NSP[0..23],load:0xff->NXM_NX_NSI[],load:0->NXM_NX_NSH_C1[],load:0->NXM_NX_NSH_C2[],load:0xac130002->NXM_NX_REG0[],resubmit(,17) This is the terminating service action on the SFF: cookie=0x9000000, duration=1055.050s, table=36, n_packets=422, n_bytes=47686, priority=5,tun_id=0 actions=goto_table:83 This is the terminating service action on the original destination: cookie=0x9000008, duration=1398.995s, table=36, n_packets=3, n_bytes=182, priority=5,tun_id=0x8 actions=load:0x800->NXM_NX_REG6[],resubmit(,220) And the vxlan gpe nsh packet: Virtual eXtensible Local Area Network |
| Comments |
| Comment by Jaime Caamaño Ruiz [ 05/May/17 ] |
|
Solved by setting tunnel id to 0 in the classifier next hop table: cookie=0xf005ba1100000004, duration=1793.555s, table=222, n_packets=0, n_bytes=0, priority=260,nshc1=0,nshc2=0 actions=move:NXM_NX_TUN_IPV4_DST[] Proposed patches [1] and [2] for carbon and nitrogen respectively. [1] https://git.opendaylight.org/gerrit/56608 |