vrfstats reports incorrect values for vxlan encapsulation

Bug #1808546 reported by sagarkchitnis
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Fix Committed
Undecided
sagarkchitnis

Bug Description

Change the "Encapsulation Priority Order" to the following:
VxLAN
MPLS Over UDP
MPLS Over GRE

on two different computes, create one container in each compute on the same network.
Run vrfstats and note the vxlan tunnel stats value.
Do a ping from one container to the other.
The "Vxlan Tunnels" value should increase but it does not change.

sagarkchitnis (sagarc)
Changed in juniperopenstack:
assignee: nobody → sagarkchitnis (sagarc)
Revision history for this message
sagarkchitnis (sagarc) wrote :

Upon investigation, the problem is in dp-core.

nh_vxlan_tunnel function (line 1984) , it says : stats = vr_inet_vrf_stats(fmd->fmd_dvrf, pkt->vp_cpu);
   if (stats)
       stats->vrf_udp_mpls_tunnels++;

instead it should be stats->vrf_vxlan_tunnels++

I will make the change , build vrouter msi again and try it on a windows compute. If it works, I will submit the PR.

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/48267
Submitter: sagarkchitnis (<email address hidden>)

sagarkchitnis (sagarc)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/48267
Committed: http://github.com/Juniper/contrail-vrouter/commit/108368ac733d5623d4c256e99c35a30dbd484c74
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 108368ac733d5623d4c256e99c35a30dbd484c74
Author: U-A5S38-NODE2\Administrator <email address hidden>
Date: Wed Dec 19 06:19:06 2018 -0800

fix for vrfstats reporting incorrect values for vxlan encapsulation

Change-Id: Iaf466edfe3ffd67247c451daa3ca79b6dfbf1419
Closes-Bug: #1808546

Changed in juniperopenstack:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.