Skip to content

Conversation

@harjotsinghpawra
Copy link
Contributor

Description of PR

This is HLD for snappi test case written to test icmp_hw_offload capabilities on hardware and orchagent.

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

Approach

What is the motivation for this PR?

We wanted to isolate and test state detection of icmp sessions and creation of sessions as well.

How did you do it?

We chose snappi to send dummy icmp echo reply data and programmed dummy sessions on dut side.

How did you verify/test it?

Tested it on local snappi setups

Any platform specific information?

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

Snappi topology

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@github-actions github-actions bot requested review from mihirpat1, r12f and wangxin January 8, 2026 05:46
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@harjotsinghpawra harjotsinghpawra changed the title Create icmp_hw_offload_asic_orchagent_test.md icmp_hw_offload_asic_orchagent_test HLD Jan 8, 2026
@harjotsinghpawra harjotsinghpawra changed the title icmp_hw_offload_asic_orchagent_test HLD HLD: icmp_hw_offload_asic_orchagent_test for snappi Jan 8, 2026
@harjotsinghpawra harjotsinghpawra changed the title HLD: icmp_hw_offload_asic_orchagent_test for snappi HLD: icmp_hw_offload_asic_orchagent_test (snappi) Jan 8, 2026
@harjotsinghpawra harjotsinghpawra marked this pull request as ready for review January 8, 2026 22:58
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