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

[QoS] J2C+ _qos_yaml for 400 and 100G #13319

Closed
wants to merge 3 commits into from

Conversation

ansrajpu-git
Copy link
Contributor

Description of PR

Updated qos test params for 400g and 100g port speeds for the new vsq threshold introduced in PR#sonic-net/sonic-buildimage#18239

Summary:
Fixes # (issue)

Type of change

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

How did you do it?

How did you verify/test it?

Executed Qos test for j2C+ t2 topology

Any platform specific information?

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

Documentation

@ansrajpu-git
Copy link
Contributor Author

@vmittal-msft .PR updated . Please review

@vmittal-msft vmittal-msft self-assigned this Jun 28, 2024
@vmittal-msft
Copy link
Contributor

vmittal-msft commented Jun 28, 2024

@kenneth-arista we will need similar value for Arista HWSKUs.
sonic-net/sonic-buildimage#18500

@arlakshm
Copy link
Contributor

@kenneth-arista, to create another PR for J2

@ansrajpu-git
Copy link
Contributor Author

This PR changes are reflected now in the master PR -#13069.
Hence closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants