Skip to content

[action] [PR:19639] Add is_ipv6_only_topology utilities and IPv6 topology check to skip …#20412

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/19639
Aug 28, 2025
Merged

[action] [PR:19639] Add is_ipv6_only_topology utilities and IPv6 topology check to skip …#20412
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/19639

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

…IPv4 tests in test_ip_suite

Description of PR

Summary:
Fixes #19638

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?

To support IPv6 only topo.
Remove unnecessary fail and reduce total test run time.

How did you do it?

Enable topo check when it tries to run IPv4 test.
Added the is_ipv6_only_topology in utilities.
If topo is IPv6 but test is IPv4, the test will skip.

How did you verify/test it?

Local run on IPv6 only testbed:
image

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #19639

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit cf9862f into sonic-net:202505 Aug 28, 2025
17 checks passed
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…D automatically (sonic-net#20412)

#### Why I did it
src/sonic-platform-daemons
```
* 01aeddd - (HEAD -> 202311, origin/202311) [xcvrd] Modify to support regular expression when parsing the key in media_settings.json (sonic-net#471) (4 hours ago) [DennisChiuEC]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

2 participants