Skip to content

[route_check.py] Skip route check for tun0 interfaces#1399

Merged
lguohan merged 1 commit intosonic-net:masterfrom
prsunny:tun0
Feb 3, 2021
Merged

[route_check.py] Skip route check for tun0 interfaces#1399
lguohan merged 1 commit intosonic-net:masterfrom
prsunny:tun0

Conversation

@prsunny
Copy link
Contributor

@prsunny prsunny commented Feb 3, 2021

- What I did
As part of dual-tor features, kernel tunnel interface (tun0) is being created. The routes over this is intended for kernel forwarding and not expected to be installed in ASIC.

Ref PR - sonic-net/sonic-swss#1615

- How I did it
Added to the list of 'skipped' interfaces

- How to verify it

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

@lguohan lguohan merged commit 9ac47e5 into sonic-net:master Feb 3, 2021
daall pushed a commit that referenced this pull request Feb 6, 2021
As part of dual-tor features, kernel tunnel interface (tun0) is being created. The routes over this is intended for kernel forwarding and not expected to be installed in ASIC.

Ref PR - sonic-net/sonic-swss#1615
anand-kumar-subramanian pushed a commit to anand-kumar-subramanian/sonic-utilities that referenced this pull request Mar 2, 2021
As part of dual-tor features, kernel tunnel interface (tun0) is being created. The routes over this is intended for kernel forwarding and not expected to be installed in ASIC.

Ref PR - sonic-net/sonic-swss#1615
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
 [sonic-swsss] Fix the issue of field "next_hop_ip" not getting updated
 in state DB in ERSPAN Mirror (sonic-net#1375)
[vlanmgr] Support Jumbo Frame By Default (sonic-net#1393)
[fec] added logic that put port down before applying fec
onfiguration (sonic-net#1399)
 [fec] Get FEC mode when port is already admin down (sonic-net#1403)
 Refine getDbId() calling to fix build after swss-common change
 (sonic-net#1245)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants