Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Choose ARP scale based on available NH entries for CISCO 8000 platforms #15842

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sridhartalari
Copy link
Contributor

Description of PR

Summary:
ARP stress test is failing for certain CISCO 8000 platforms as the tested scale exceeds the HW availablity

Fixes # (issue)
Choose ARP scale based on available NH entries for CISCO 8000 platforms

Type of change

  • [ x] Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Pass all sonic mgmt tests on CISCO 8000 platforms

How did you do it?

Run all sonic-mgmt testcases with CISCO 8000 as DUT and verify

How did you verify/test it?

Make sure all learnt ARP's are installed by comparing the neighbor entries installed in HW with ARP requests sent

Any platform specific information?

In certain CISCO platforms, Each Neighbor needs one NH entry. As a general fix, we limit the number of Neighbor entries tested to the available NH entry on the platform

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

Documentation

@sridhartalari
Copy link
Contributor Author

/lgtm

@bpar9
Copy link
Contributor

bpar9 commented Dec 11, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

bpar9
bpar9 previously approved these changes Dec 12, 2024
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

3 participants