Skip to content

[VRF] enable/disable net.ipv6.ip_nonlocal_bind for TestVrfLoopbackIntf.#2369

Merged
wangxin merged 3 commits intosonic-net:masterfrom
Minkang-Tsai:test_vrf1
Oct 20, 2020
Merged

[VRF] enable/disable net.ipv6.ip_nonlocal_bind for TestVrfLoopbackIntf.#2369
wangxin merged 3 commits intosonic-net:masterfrom
Minkang-Tsai:test_vrf1

Conversation

@Minkang-Tsai
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

Socket bind the IP address in VRF will get the following error message.

root@as7726-32x-1:~# ping6 fc00:168::2 -I Vrf2 -I fc00:1::32
ping: bind icmp socket: Cannot assign requested address

How did you do it?

  1. enable net.ipv6.ip_nonlocal_bind during setup.
  2. disable net.ipv6.ip_nonlocal_bind during teardown.

How did you verify/test it?

Run the TestVrfLoopbackIntf test case of test_vrf.py.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Oct 20, 2020

This pull request fixes 1 alert when merging 593ef94 into 3b152b4 - view on LGTM.com

fixed alerts:

  • 1 for Unnecessary pass

@wangxin wangxin merged commit f2922bf into sonic-net:master Oct 20, 2020
@Minkang-Tsai Minkang-Tsai deleted the test_vrf1 branch October 21, 2020 05:38
yxieca pushed a commit that referenced this pull request Nov 2, 2020
Description of PR
#2369 make use of duthost in TestVrfLoopbackIntf setup while there is no such fixture parameter in the setup function, adding the parameter.

Approach
What is the motivation for this PR?
the bug introduced in #2369 pr

How did you do it?
just added the lacking parameter

How did you verify/test it?
did a run of the test

Signed-off-by: Volodymyr Boyko <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
swss:
* 7841930 2022-07-15 | [vxlan]Fixing L2MC vlan member caching issue (sonic-net#2378) (HEAD -> 202205) [Sudharsan Dhamal Gopalarathnam]
* b8cd435 2022-07-14 | [muxorch] Always use direct link for SoC IPs (sonic-net#2369) [Longxiang Lyu]
* 6158d5c 2022-07-08 | Add BGP profile to Vnet routes (sonic-net#2337) [Prince Sunny]
* bdb7ffd 2022-07-06 | [teammgr]: Waiting MACsec ready before doLagMemberTask (sonic-net#2286) [Ze Gan]

sairedis:
* 58359d4 2022-06-30 | [sairedis] Perform log rotate on request (sonic-net#1058) (HEAD -> 202205, github/202205) [Kamil Cudnik]
* cad0268 2022-07-13 | Enable cisco debug shell by default (sonic-net#1078) [VenkatCisco]

Signed-off-by: Ying Xie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants