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 LAG on Distributed VOQ System #3033

Merged
merged 3 commits into from Apr 12, 2021

Conversation

oxygen980
Copy link
Contributor

@oxygen980 oxygen980 commented Feb 24, 2021

Description of PR

The PR is to introduced LAG on VOQ

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

This is the test plan for LAG on SONIC Distributed VOQ System, as described in the VOQ LAG HLD

The PR covered in this test plan is Distributed VOQ LAG HLD PR 697

The scope of this test plan is as follows:

  • Check for unique SYSTEM_LAG_ID per LAG in a chassis with multiple line cards
  • Check if LAG is learned by local and remote ASIC

This test plan does not cover functioning of LAG as an IP/Layer3 endpoint as that is outside the scope of the code PR.

How did you do it?

Based on the code changes based on the associated PR

How did you verify/test it?

Any platform specific information?

This is valid only on VOQ systems.

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

Documentation

@oxygen980
Copy link
Contributor Author

retest vs image please

@oxygen980 oxygen980 changed the title Test plan for VOQ LAG feature Test plan for LAG on Distributed VOQ System Feb 25, 2021
@anshuv-mfst
Copy link

@eswaranb - please add Arista reviewers

@oxygen980
Copy link
Contributor Author

retest vs image please

@oxygen980
Copy link
Contributor Author

retest vs image please

2 similar comments
@oxygen980
Copy link
Contributor Author

retest vs image please

@oxygen980
Copy link
Contributor Author

retest vs image please

@shubav
Copy link
Contributor

shubav commented Mar 22, 2021

@yxieca, Could you help merge this test plan? The failures in the test case are unrelated to this commit. And this was presented in the test community last week. Thanks.

@shubav
Copy link
Contributor

shubav commented Mar 30, 2021

@yxieca - could you please review?
@eswaranb - could you please add Arista reviewers?
Thanks

@anshuv-mfst
Copy link

@yxieca @wangxin , @saravanansv - could you please take a look, thanks.


Verify that when a LAG is dynamically deleted via CLI on an ASIC, it is removed in remote ASIC_DB.
### Test Steps
* Delete an existing LAG from any ASIC in the OC topology
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is OC topology?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OC topology was to refer Open community topology, I have removed 'OC topology'.

@wangxin wangxin merged commit 84f004f into sonic-net:master Apr 12, 2021
saravanansv pushed a commit to saravanansv/sonic-mgmt that referenced this pull request May 6, 2021
What is the motivation for this PR?
This is the test plan for LAG on SONIC Distributed VOQ System, as described in the VOQ LAG HLD (https://github.com/Azure/SONiC/blob/2e05c6b8ac570fd237484a18e732a58eec004b9c/doc/voq/lag_hld.md)

The PR covered in this test plan is Distributed VOQ LAG HLD PR 697 (https://github.com/Azure/SONiC/pull/697/files#diff-77ea0c16b4ae9885fa0e388e81f6343c6bda0f24b999f93e64fcee8467df63fc)

The scope of this test plan is as follows:
* Check for unique SYSTEM_LAG_ID per LAG in a chassis with multiple line cards
* Check if LAG is learned by local and remote ASIC

This test plan does not cover functioning of LAG as an IP/Layer3 endpoint as that is outside the scope of the code PR.

How did you do it?
Based on the code changes based on the associated PR

Any platform specific information?
This is valid only on VOQ systems.

Co-authored-by: falodiya <renu.falodiy@nokia.com>
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
What is the motivation for this PR?
This is the test plan for LAG on SONIC Distributed VOQ System, as described in the VOQ LAG HLD (https://github.com/Azure/SONiC/blob/2e05c6b8ac570fd237484a18e732a58eec004b9c/doc/voq/lag_hld.md)

The PR covered in this test plan is Distributed VOQ LAG HLD PR 697 (https://github.com/Azure/SONiC/pull/697/files#diff-77ea0c16b4ae9885fa0e388e81f6343c6bda0f24b999f93e64fcee8467df63fc)

The scope of this test plan is as follows:
* Check for unique SYSTEM_LAG_ID per LAG in a chassis with multiple line cards
* Check if LAG is learned by local and remote ASIC

This test plan does not cover functioning of LAG as an IP/Layer3 endpoint as that is outside the scope of the code PR.

How did you do it?
Based on the code changes based on the associated PR

Any platform specific information?
This is valid only on VOQ systems.

Co-authored-by: falodiya <renu.falodiy@nokia.com>
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.

None yet

4 participants