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.

Sign up for GitHub

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

Jump to bottom

[test gap][dhcpv6_relay] Add testcase for multi-vlan #9122

Closed
kellyyeh opened this issue Jul 26, 2023 · 1 comment
Closed

[test gap][dhcpv6_relay] Add testcase for multi-vlan #9122

kellyyeh opened this issue Jul 26, 2023 · 1 comment
Assignees
@yaqiangz @w1nda
Labels
P0 Test gap

Comments

@kellyyeh
Copy link
Contributor

kellyyeh commented Jul 26, 2023 •
edited
Loading

Description
Previously dhcp6relay was not always forwarding packets with correct vlan ip because one client socket was used for multiple interfaces and vlan interfaces were not filtered

Steps to reproduce the issue:

  1. Configure multiple Vlans and dhcp servers
  2. Send dhcp requests
  3. Check dhcp packets relayed contains expected Vlan IP

Describe the results you received:

Describe the results you expected:

Additional information you deem important:

**Output of `show version`:**

```
(paste your output here)
```

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@Blueve Blueve added the P0 label Jul 26, 2023
@kellyyeh kellyyeh self-assigned this Jul 26, 2023
@StormLiangMS StormLiangMS assigned yaqiangz and unassigned kellyyeh Jun 6, 2024
@yaqiangz
Copy link
Contributor

sonic-net/sonic-dhcp-relay#27

@w1nda w1nda mentioned this issue Sep 25, 2024
Merged
8 tasks
wangxin pushed a commit that referenced this issue Oct 22, 2024
@w1nda
[dhcpv6_relay_test] test if dhcpv6_relay will set correct link addres…
5737e5c 
…s in packet on multiple vlans host (#14732)

What is the motivation for this PR?
This code is to fix test gap: #9122

How did you do it?
Configure multiple Vlans and dhcp servers
Send dhcp requests
Check dhcp packets relayed contains expected Vlan IP

How did you verify/test it?
Tested on mx, m0, dualtor and dualtor-aa.
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this issue Oct 23, 2024
@w1nda @mssonicbld
[dhcpv6_relay_test] test if dhcpv6_relay will set correct link addres…
75bf1ca 
…s in packet on multiple vlans host (sonic-net#14732)

What is the motivation for this PR?
This code is to fix test gap: sonic-net#9122

How did you do it?
Configure multiple Vlans and dhcp servers
Send dhcp requests
Check dhcp packets relayed contains expected Vlan IP

How did you verify/test it?
Tested on mx, m0, dualtor and dualtor-aa.
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this issue Oct 23, 2024
@w1nda @mssonicbld
[dhcpv6_relay_test] test if dhcpv6_relay will set correct link addres…
9423932 
…s in packet on multiple vlans host (sonic-net#14732)

What is the motivation for this PR?
This code is to fix test gap: sonic-net#9122

How did you do it?
Configure multiple Vlans and dhcp servers
Send dhcp requests
Check dhcp packets relayed contains expected Vlan IP

How did you verify/test it?
Tested on mx, m0, dualtor and dualtor-aa.
This was referenced Oct 23, 2024
Open
Open
@w1nda w1nda closed this as completed Nov 1, 2024
sreejithsreekumaran pushed a commit to sreejithsreekumaran/sonic-mgmt that referenced this issue Nov 15, 2024
@w1nda @sreejithsreekumaran
[dhcpv6_relay_test] test if dhcpv6_relay will set correct link addres…
0fa8d2b 
…s in packet on multiple vlans host (sonic-net#14732)

What is the motivation for this PR?
This code is to fix test gap: sonic-net#9122

How did you do it?
Configure multiple Vlans and dhcp servers
Send dhcp requests
Check dhcp packets relayed contains expected Vlan IP

How did you verify/test it?
Tested on mx, m0, dualtor and dualtor-aa.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yaqiangz yaqiangz

@w1nda w1nda

Labels
P0 Test gap
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Blueve @yaqiangz @w1nda @kellyyeh