Skip to content

[action] [PR:18465] [TESTGAP] Implement new test case to check the crm nexthop_group#18467

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/18465
May 23, 2025
Merged

[action] [PR:18465] [TESTGAP] Implement new test case to check the crm nexthop_group#18467
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/18465

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Fixes # (issue)
This is a new test case to verify the CRM nexthop_group threshold, which is typically set to at least 256. However, for some HWSKUs, such as 7215 and 720dt, the value is lower. This is acceptable due to their specific application scenarios, which have lower requirements for ECMP group capacity.

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • [] 202405
  • 202411

Approach

What is the motivation for this PR?

To address the test gap related to nexthop group requirements.

How did you do it?

To verify the total number of nexthop groups supported by the platform and ensure it meets the minimum required threshold.

How did you verify/test it?

Run the test case on device.

Any platform specific information?

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

't0', 't1', 'm0', 'mx', 'm1', 'm2', 'm3'

Documentation

…ic-net#18465)

This is a new test case to verify the CRM nexthop_group threshold, which is typically set to at least 256. However, for some HWSKUs, such as 7215 and 720dt, the value is lower. This is acceptable due to their specific application scenarios, which have lower requirements for ECMP group capacity.

What is the motivation for this PR?
To address the test gap related to nexthop group requirements.

How did you do it?
To verify the total number of nexthop groups supported by the platform and ensure it meets the minimum required threshold.

How did you verify/test it?
Run the test case on device.

Any platform specific information?
Supported testbed topology if it's a new test case?
't0', 't1', 'm0', 'mx', 'm1', 'm2', 'm3'
@mssonicbld
Copy link
Collaborator Author

Original PR: #18465

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 6c1c706 into sonic-net:202411 May 23, 2025
17 checks passed
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