Skip to content

support for dut single stack ipv6 mgmt ip#20292

Merged
wangxin merged 10 commits intosonic-net:masterfrom
xixuej:mgmt_ipv6_skip_and_fix
Nov 3, 2025
Merged

support for dut single stack ipv6 mgmt ip#20292
wangxin merged 10 commits intosonic-net:masterfrom
xixuej:mgmt_ipv6_skip_and_fix

Conversation

@xixuej
Copy link
Copy Markdown
Contributor

@xixuej xixuej commented Aug 18, 2025

  • fix test cases to support ipv6 mgmt ip
  • skip test cases that do not support dut ipv6 yet

Description of PR

Summary: Enhancement of test cases to support dut single stack mgmt ipv6
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?

update test cases and also skip condition files, etc...

How did you verify/test it?

run regression with ipv6 mgmt ip in physical setups

Any platform specific information?

NA

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

Documentation

- fix test cases to support ipv6 mgmt ip
- skip test cases that do not support dut ipv6 yet
@xixuej xixuej requested review from wangxin and yxieca as code owners August 18, 2025 06:48
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Copy Markdown
Collaborator

@wangxin Can you please help review?

skip:
reason: "ipv6 mgmt ip is not supported"
conditions:
- "is_mgmt_ipv6_only==True"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is SNMP supported after snmp_fact change in this PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bingwang-ms, this case will still fail with the changes in this PR. This is becasue the key "lldpLocManAddrLen" does not exist in snmp_facts['snmp_lldp']. This still needs further analysis so I skipped it for now

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

… type and update certificate creation function to utilize it.
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@xixuej
Copy link
Copy Markdown
Contributor Author

xixuej commented Sep 1, 2025

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Aharon Malkin <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Guy Shemesh <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Guy Shemesh <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Guy Shemesh <[email protected]>
vrajeshe pushed a commit to Akshath-17/sonic-mgmt that referenced this pull request Jan 4, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Venkata Gouri Rajesh Etla <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
yifan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 14, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: YiFan Wang <[email protected]>
yifan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 14, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: YiFan Wang <[email protected]>
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jan 20, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
PriyanshTratiya pushed a commit to PriyanshTratiya/sonic-mgmt that referenced this pull request Jan 21, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Priyansh Tratiya <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Guy Shemesh <[email protected]>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Lakshmi Yarramaneni <[email protected]>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Lakshmi Yarramaneni <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Yael Tzur <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Yael Tzur <[email protected]>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Raghavendran Ramanathan <[email protected]>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Raghavendran Ramanathan <[email protected]>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 18, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups

Signed-off-by: Raghavendran Ramanathan <[email protected]>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Zhuohui Tan <[email protected]>
mssonicbld pushed a commit that referenced this pull request Feb 23, 2026
In PR #21045, I made mistakes while resolving the conflicts with #20292. Some code added in #20292 was accidentally reverted.

This change is to add back the code introduced in #20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Abhishek <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
fix test cases to support ipv6 mgmt ip
skip test cases that do not support dut ipv6 yet

What is the motivation for this PR?
We now enabled single stack ipv6 address for switch management port, some test cases do not support it yet, this PR fixed some of the cases or skiped for the remaining cases

How did you do it?
update test cases and also skip condition files, etc...

How did you verify/test it?
run regression with ipv6 mgmt ip in physical setups
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
In PR sonic-net#21045, I made mistakes while resolving the conflicts with sonic-net#20292. Some code added in sonic-net#20292 was accidentally reverted.

This change is to add back the code introduced in sonic-net#20292 for creating SNMP UdpTransportTarget. Then the snmp_facts module will work with IPv6 only scenario again.

Signed-off-by: Xin Wang <[email protected]>
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.

5 participants