Skip to content

Adjust test /telemetry/test_events.py to handle IPV6 only topologies #21898

Closed
gshemesh2 wants to merge 9 commits into
sonic-net:masterfrom
gshemesh2:test_events_telementry_ipv6
Closed

Adjust test /telemetry/test_events.py to handle IPV6 only topologies #21898
gshemesh2 wants to merge 9 commits into
sonic-net:masterfrom
gshemesh2:test_events_telementry_ipv6

Conversation

@gshemesh2

Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Adjust test /telemetry/test_events.py to handle IPV6 only topologies

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
  • 202511

Approach

What is the motivation for this PR?

Adjust test /telemetry/test_events.py to handle IPV6 only topologies

How did you do it?

using is_ipv6_only_topology to detect and adjust ipov6 sonic commands to handle cases of ipv6 only topologies

How did you verify/test it?

ran it on SN5640 Nvidia platforms over IPV6 only topologies

Any platform specific information?

No platform specific information

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

Documentation

…nected to optical module with the supported CMIS

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
@gshemesh2
gshemesh2 requested review from a team and wangxin as code owners January 12, 2026 12:01
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2
gshemesh2 force-pushed the test_events_telementry_ipv6 branch from 71053e5 to 4cb4209 Compare January 12, 2026 12:03
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2
gshemesh2 force-pushed the test_events_telementry_ipv6 branch from 4cb4209 to 16214ef Compare January 12, 2026 12:04
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp 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 successfully started running 1 pipeline(s).

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
@gshemesh2
gshemesh2 force-pushed the test_events_telementry_ipv6 branch from 7d4dacc to 05cd239 Compare January 12, 2026 12:05
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@github-actions
github-actions Bot requested review from wsycqyz and zbud-msft January 12, 2026 12:06
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2
gshemesh2 force-pushed the test_events_telementry_ipv6 branch from d4f8346 to e09e4be Compare January 12, 2026 12:08
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp 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 successfully started running 1 pipeline(s).

@gshemesh2

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines 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 successfully started running 1 pipeline(s).

@gshemesh2
gshemesh2 force-pushed the test_events_telementry_ipv6 branch from d71fd3b to 88082f9 Compare February 2, 2026 12:30
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
@gshemesh2
gshemesh2 force-pushed the test_events_telementry_ipv6 branch from 88082f9 to dad0bd6 Compare February 3, 2026 08:59
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gshemesh2 gshemesh2 closed this Feb 15, 2026
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.

4 participants