Skip to content

Generic hash test plan enhancement#11823

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
echuawu:gh_merge_2
May 7, 2024
Merged

Generic hash test plan enhancement#11823
liat-grozovik merged 1 commit intosonic-net:masterfrom
echuawu:gh_merge_2

Conversation

@echuawu
Copy link
Copy Markdown
Contributor

@echuawu echuawu commented Mar 1, 2024

Enhance generic hash testplan

Description of PR

Enhance generic hash testplan:

Add 6 new supported fields
Support algorithm configuration

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

Add testplan for new generic hash implementation

How did you do it?

Add test plan for new 6 new supported fields and algorithm configuration

How did you verify/test it?

Any platform specific information?

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

Documentation

@echuawu echuawu requested review from wangxin and yxieca as code owners March 1, 2024 05:06
@echuawu
Copy link
Copy Markdown
Contributor Author

echuawu commented Mar 1, 2024

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Mar 4, 2024

@prsunny can you assign someone to take a look at this test plan?

Enhance generic hash testplan

Change-Id: I296ecb36b12dbc2728494a1a235b316302e5526d
@echuawu
Copy link
Copy Markdown
Contributor Author

echuawu commented Mar 21, 2024

@wangxin , the test plan has been updated according to the review comments.

@liat-grozovik
Copy link
Copy Markdown
Collaborator

@nazariig please review as the feature lead

@echuawu echuawu mentioned this pull request Apr 24, 2024
8 tasks
@liat-grozovik liat-grozovik merged commit 91f879a into sonic-net:master May 7, 2024
liat-grozovik pushed a commit that referenced this pull request Jun 26, 2024
Enhance Generic Hash test to cover 6 new supported fields and algorithm configuration.
Related test plan: #11823

Summary:
1. Add support for 6 new fields
2. Update show parse function
3. Add algorithm configure and validate test
4. Adapt on dualtor setup
5. Skip cases on SPC1 due to hardware limitation
6. Add list support for option --algorithm and --hash_field according to community review request
7. Update dynamic supported algorithm check

Change-Id: Ie5382f30b70b17c58aaacaf81560ca0716f203d7
(cherry picked from commit 8da9ce0)
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Sep 20, 2024
Enhance Generic Hash test to cover 6 new supported fields and algorithm configuration.
Related test plan: sonic-net#11823

Summary:
1. Add support for 6 new fields
2. Update show parse function
3. Add algorithm configure and validate test
4. Adapt on dualtor setup
5. Skip cases on SPC1 due to hardware limitation
6. Add list support for option --algorithm and --hash_field according to community review request
7. Update dynamic supported algorithm check

Change-Id: Ie5382f30b70b17c58aaacaf81560ca0716f203d7
(cherry picked from commit 8da9ce0)
mssonicbld pushed a commit that referenced this pull request Sep 21, 2024
Enhance Generic Hash test to cover 6 new supported fields and algorithm configuration.
Related test plan: #11823

Summary:
1. Add support for 6 new fields
2. Update show parse function
3. Add algorithm configure and validate test
4. Adapt on dualtor setup
5. Skip cases on SPC1 due to hardware limitation
6. Add list support for option --algorithm and --hash_field according to community review request
7. Update dynamic supported algorithm check

Change-Id: Ie5382f30b70b17c58aaacaf81560ca0716f203d7
(cherry picked from commit 8da9ce0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants