Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EVPN VXLAN] Only one vni to vlan map entry in the output of show vxlan tunnel command #11902

Closed
dellwuchuan opened this issue Aug 31, 2022 · 4 comments
Assignees
Labels
BRCM Triaged this issue has been triaged

Comments

@dellwuchuan
Copy link

Description

When there are many vlan to vni mapping entries, there are only one founded in the output of show vxlan tunnel command.

Steps to reproduce the issue:

  1. Configure vxlan at dut

sudo config vxlan add vtep101032 10.1.0.32
sudo config vxlan evpn_nvo add my-nvo vtep101032
sudo config vxlan map add vtep101032 100 500100
sudo config vxlan map add vtep101032 101 500101
2. Configure vxlan at HA

brctl addbr br_500100
brctl addif br_500100 vtep_500100
brctl stp br_500100 off
sudo ip link set vtep_500100 up
sudo ip link set br_500100 up
sudo ip addr add 100.0.0.3/24 dev br_500100
sudo ip addr add 100::3/24 dev br_500100
brctl addbr br_500101
brctl addif br_500101 vtep_500101
brctl stp br_500101 off
sudo ip link set vtep_500101 up
sudo ip link set br_500101 up
sudo ip addr add 101.0.0.2/24 dev br_500101
sudo ip addr add 101::2/64 dev br_500101
3. Configure IBGP at DUT and HA

!
router bgp 65000
bgp router-id 10.1.0.32
neighbor 30.0.0.2 remote-as 65000
neighbor 40.0.0.3 remote-as 65000
!
address-family ipv4 unicast
network 10.1.0.32/32
exit-address-family
!
address-family l2vpn evpn
neighbor 30.0.0.2 activate
neighbor 40.0.0.3 activate
advertise-all-vni
advertise-svi-ip
vni 500100
rd 10.1.0.32:100
exit-vni
vni 500101
rd 10.1.0.32:101
exit-vni
exit-address-family
!

!
router bgp 65000
bgp router-id 40.0.0.3
neighbor 40.0.0.1 remote-as 65000
!
address-family l2vpn evpn
neighbor 40.0.0.1 activate
advertise-all-vni
advertise-svi-ip
vni 500100
rd 40.0.0.3:100
exit-vni
exit-address-family
!

  1. Check the vni vlan map relationship

show vxlan vlanvnimap
+----------+---------+
| VLAN | VNI |
+==========+=========+
| Vlan100 | 500100 |
+----------+---------+
| Vlan101 | 500101 |
+----------+---------+
Total count : 2

  1. Check the vxlan tunnel by command "show vxlan tunnel", there is only one entry in the vni to vlan map part:

show vxlan tunnel
vxlan tunnel name source ip destination ip tunnel map name tunnel map mapping(vni -> vlan)


vtep101032 10.1.0.32 map_500100_Vlan100 500100 -> Vlan100

Describe the results you received:

Only one vlan-vni map entry - map_500100_Vlan100 500100 -> Vlan100

Describe the results you expected:

It should list all the vlan vni mapping entries

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

There is the analysis result of developer:

The issue is due to the show command parsing code getting only the first vxlan map key and including it in the show command.

    vxlan_map_keys = config_db.keys(config_db.CONFIG_DB,
                                    'VXLAN_TUNNEL_MAP{}{}{}*'.format(config_db.KEY_SEPARATOR, vxlan_name, config_db.KEY_SEPARATOR))
    if vxlan_map_keys:
        vxlan_map_mapping = config_db.get_all(config_db.CONFIG_DB, vxlan_map_keys[0])
        r.append(vxlan_map_keys[0].split(config_db.KEY_SEPARATOR, 2)[2])
        r.append("{} -> {}".format(vxlan_map_mapping.get('vni'), vxlan_map_mapping.get('vlan')))
    table.append(r)
@zhangyanzhao zhangyanzhao added BRCM Triaged this issue has been triaged labels Aug 31, 2022
@zhangyanzhao
Copy link
Collaborator

BRCM will help to take a look. Thanks @adyeung

@skbhava
Copy link

skbhava commented Sep 2, 2022

Will work for the solution and create PR for review. ETA 09/19.

@adyeung
Copy link
Collaborator

adyeung commented Sep 23, 2022

Fix posted to
sonic-net/sonic-utilities#2391

stepanblyschak added a commit to stepanblyschak/sonic-buildimage that referenced this issue Oct 5, 2022
```
99425a8 (HEAD -> 202205, origin/202205) [actions] Support Semgrep by Github Actions (sonic-net#2417)
f41e4d1 Fix for show vxlan tunnel command display issue sonic-net#11902 (sonic-net#2391)
e1d827e [VxLAN]Fix Vxlan delete command to throw error when there are references (sonic-net#2404)
d77acf8 [doc] add documentation on automatic techsupport based on memory (sonic-net#2411)
2cfc75a [doc] update "config feature" section with "--block" option (sonic-net#2409)
9dc8471 [Vxlanmgrd] [CPA] Update the vxlan_tunnel name len to be under IFNAMIZ to overcome netdev creation failure (sonic-net#2398)
342589e Added cisco config platform commands (sonic-net#2242) (sonic-net#2418)
be7da6b [sonic-installer] use host docker startup arguments when running dockerd in chroot (sonic-net#2179) (sonic-net#2407)
d112f7c [202205][auto-ts] add memory check (sonic-net#2116) (sonic-net#2413)
```

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
yxieca added a commit to yxieca/sonic-buildimage that referenced this issue Oct 6, 2022
linkmgrd:
* a5ac7f6 2022-10-05 | [Active-Active] Post link prober stats to state db  (sonic-net#140) (HEAD -> 202205, github/202205) [Jing Zhang]
* f4b0e53 2022-10-05 | [Active-Active] Retry config mux mode standby (sonic-net#139) [Jing Zhang]

utilities:
* a255838 2022-10-04 | [minigraph] new workflow for golden path (sonic-net#2396) (HEAD -> 202205, github/202205) [jingwenxie]
* 99425a8 2022-10-03 | [actions] Support Semgrep by Github Actions (sonic-net#2417) [Mai Bui]
* f41e4d1 2022-09-30 | Fix for show vxlan tunnel command display issue sonic-net#11902 (sonic-net#2391) [Senthil Bhava]
* e1d827e 2022-09-29 | [VxLAN]Fix Vxlan delete command to throw error when there are references (sonic-net#2404) [Sudharsan Dhamal Gopalarathnam]
* d77acf8 2022-09-28 | [doc] add documentation on automatic techsupport based on memory (sonic-net#2411) [Stepan Blyshchak]
* 2cfc75a 2022-09-28 | [doc] update "config feature" section with "--block" option (sonic-net#2409) [Stepan Blyshchak]
* 9dc8471 2022-09-28 | [Vxlanmgrd] [CPA] Update the vxlan_tunnel name len to be under IFNAMIZ to overcome netdev creation failure (sonic-net#2398) [Vivek]
* 342589e 2022-10-03 | Added cisco config platform commands (sonic-net#2242) (sonic-net#2418) [yucgu]

swss:
* 9d9f395 2022-10-04 | [intfmgr]: Enable `accept_untracked_na` kernel param (sonic-net#2436) (HEAD -> 202205, github/202205) [Lawrence Lee]
* 6b6d25d 2022-10-04 |  [orchdaemon]: Fixed sairedis record file rotation (sonic-net#2480) [Bryan Crossland]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
yxieca added a commit that referenced this issue Oct 6, 2022
linkmgrd:
* a5ac7f6 2022-10-05 | [Active-Active] Post link prober stats to state db  (#140) (HEAD -> 202205, github/202205) [Jing Zhang]
* f4b0e53 2022-10-05 | [Active-Active] Retry config mux mode standby (#139) [Jing Zhang]

utilities:
* a255838 2022-10-04 | [minigraph] new workflow for golden path (#2396) (HEAD -> 202205, github/202205) [jingwenxie]
* 99425a8 2022-10-03 | [actions] Support Semgrep by Github Actions (#2417) [Mai Bui]
* f41e4d1 2022-09-30 | Fix for show vxlan tunnel command display issue #11902 (#2391) [Senthil Bhava]
* e1d827e 2022-09-29 | [VxLAN]Fix Vxlan delete command to throw error when there are references (#2404) [Sudharsan Dhamal Gopalarathnam]
* d77acf8 2022-09-28 | [doc] add documentation on automatic techsupport based on memory (#2411) [Stepan Blyshchak]
* 2cfc75a 2022-09-28 | [doc] update "config feature" section with "--block" option (#2409) [Stepan Blyshchak]
* 9dc8471 2022-09-28 | [Vxlanmgrd] [CPA] Update the vxlan_tunnel name len to be under IFNAMIZ to overcome netdev creation failure (#2398) [Vivek]
* 342589e 2022-10-03 | Added cisco config platform commands (#2242) (#2418) [yucgu]

swss:
* 9d9f395 2022-10-04 | [intfmgr]: Enable `accept_untracked_na` kernel param (#2436) (HEAD -> 202205, github/202205) [Lawrence Lee]
* 6b6d25d 2022-10-04 |  [orchdaemon]: Fixed sairedis record file rotation (#2480) [Bryan Crossland]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
wen587 added a commit to wen587/sonic-buildimage that referenced this issue Oct 10, 2022
423779410 [muxcable][config] add CLI support for mux mode detach (sonic-net#2425)
a817896b1 YANG validation for ConfigDB Updates: MGMT_INTERFACE, PORTCHANNEL_MEMBER use cases (sonic-net#2420)
81e2aecca [minigraph] new workflow for golden path (sonic-net#2396)
c1206aac3 ConfigDB Updates with YANG Validation: Include potential for YANG validation even when adhoc validation is used (sonic-net#2412)
57c509a9d [show] vnet endpoint [ip/ipv6] command (sonic-net#2342)
4b2b766ac [actions] Support Semgrep by Github Actions (sonic-net#2417)
156257e2a check for vxlan mapping before removing vlan (sonic-net#2388)
cb0edd310 Fix for show vxlan tunnel command display issue sonic-net#11902 (sonic-net#2391)
ac71d745d [VxLAN]Fix Vxlan delete command to throw error when there are references (sonic-net#2404)
7419c6731 Added cisco config platform commands (sonic-net#2242)
8760bbe80 Add UT to check sonic installer does not depend on database (sonic-net#2401)
6bef65260 [doc] add documentation on automatic techsupport based on memory (sonic-net#2411)
4a783745f [doc] update "config feature" section with "--block" option (sonic-net#2409)
dd6210fcc [Vxlanmgrd] [CPA] Update the vxlan_tunnel name len to be under IFNAMIZ to overcome netdev creation failure (sonic-net#2398)
bdc4a8a60 Fix broken pipeline build URL (sonic-net#2363)
b31681b43 Fix display disorder problem of show vrf  (sonic-net#2392)
123504a85 YANG validation for ConfigDB Updates: portchannel add/remove, loopback interface, VLAN
28f6820c6 [link-local]Modify RIF check to include link-local enabled interfaces (sonic-net#2394)
@adyeung
Copy link
Collaborator

adyeung commented Oct 20, 2022

resolved by sonic-net/sonic-utilities#2391

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BRCM Triaged this issue has been triaged
Projects
None yet
Development

No branches or pull requests

4 participants