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

Test plan for Distributed VoQ architecture #2659

Merged
merged 3 commits into from
Feb 18, 2021
Merged

Conversation

tcusto
Copy link
Contributor

@tcusto tcusto commented Dec 14, 2020

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

Creating test plan and cases for validating the design of the distributed VoQ architecture HLD

How did you do it?

Based on the code changes in the associated PRs

How did you verify/test it?

N/A

Any platform specific information?

This is only applicable for distributed VoQ systems.

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

Only T2 topology as mentioned in #2638

Documentation

@ghost
Copy link

ghost commented Dec 14, 2020

CLA assistant check
All CLA requirements met.

docs/testplan/Distributed-VoQ-Arch-test-plan.md Outdated Show resolved Hide resolved
docs/testplan/Distributed-VoQ-Arch-test-plan.md Outdated Show resolved Hide resolved
docs/testplan/Distributed-VoQ-Arch-test-plan.md Outdated Show resolved Hide resolved
docs/testplan/Distributed-VoQ-Arch-test-plan.md Outdated Show resolved Hide resolved
@anshuv-mfst
Copy link

Eswaran (Arista) - could you please review, thanks.

Copy link

@tarundeepsharma88 tarundeepsharma88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is Tarun from Arista. I reviewed this test plan and it looks good to me. Thanks

@shubav
Copy link
Contributor

shubav commented Feb 9, 2021

@wangxin can this be merged? We have the next set of VoQ TCs ready to be pushed. But it's in the same file. Hence there's a dependency.

@wangxin wangxin merged commit f679ca6 into sonic-net:master Feb 18, 2021
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.

5 participants