Skip to content

Test plan for next hop split#3459

Merged
wangxin merged 2 commits intosonic-net:masterfrom
TACappleman:master
Oct 9, 2021
Merged

Test plan for next hop split#3459
wangxin merged 2 commits intosonic-net:masterfrom
TACappleman:master

Conversation

@TACappleman
Copy link
Contributor

What is the motivation for this PR?
This is the test plan for the next hop group split enhancement, as covered in PR 712 (sonic-net/SONiC#712)

Description of PR

Added test plan for next hop group split enhancement (sonic-net/SONiC#712)

Type of change

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

Approach

What is the motivation for this PR?

Present test plan for enhancement before writing the tests.

How did you do it?

N/A

How did you verify/test it?

N/A

Any platform specific information?

N/A

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

N/A

Documentation

Commit is documentation

What is the motivation for this PR?
This is the test plan for the next hop group split enhancement, as covered in PR 712 (sonic-net/SONiC#712)
@yxieca yxieca requested a review from qiluo-msft May 16, 2021 04:11
@yxieca
Copy link
Collaborator

yxieca commented Sep 9, 2021

@TACappleman there is a minor type in the HLD, can you fix that so that we can get the PR merged?

@TACappleman
Copy link
Contributor Author

@yxieca Thanks, fixed

Copy link
Contributor

@qiluo-msft qiluo-msft left a comment

Choose a reason for hiding this comment

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

LGTM. Please check with other reviewers.

@wangxin wangxin merged commit 0bf0a8e into sonic-net:master Oct 9, 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.

4 participants