Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[PBH] Add PBH edit flow validation to test #5263

Merged
merged 1 commit into from
Mar 31, 2022

Conversation

AntonHryshchuk
Copy link
Contributor

Signed-off-by: Anton antonh@nvidia.com

Description of PR

Summary:
Added pbh edit flow validation to the test test_inner_hashing.py.
Also added: 1) warmboot finalizer checker to pbh tests with WR. 2)removed acl dependency in WR lag test

Type of change

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

Back port request

  • 201911
  • 202012

Approach

What is the motivation for this PR?

To cover the pbh edit flow feature

How did you verify/test it?

all tests are passed

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

t0

Documentation

Signed-off-by: Anton <antonh@nvidia.com>
@AntonHryshchuk AntonHryshchuk requested a review from a team as a code owner March 1, 2022 22:01
@AntonHryshchuk
Copy link
Contributor Author

@nazariig, @anish-n - please review

@AntonHryshchuk
Copy link
Contributor Author

the conflicts were because the changes were done over reverted commit #5282.
must be ok, when the revert of revert will be merged #5310

@liat-grozovik liat-grozovik changed the title [pbh edit flow] add pbh edit flow validation to test [PBH] Add PBH edit flow validation to test Mar 30, 2022
@liat-grozovik
Copy link
Collaborator

@nazariig could you please help to review as well?

@liat-grozovik liat-grozovik merged commit 2159aeb into sonic-net:master Mar 31, 2022
@liat-grozovik
Copy link
Collaborator

@anish-n FYI

@AntonHryshchuk AntonHryshchuk deleted the pbh_edit_flow branch April 4, 2022 06:07
xwjiang-ms pushed a commit to xwjiang-ms/sonic-mgmt that referenced this pull request Apr 13, 2022
Added pbh edit flow validation to the test test_inner_hashing.py.
Also added: 1) warmboot finalizer checker to pbh tests with WR. 2)removed acl dependency in WR lag test

Signed-off-by: Anton <antonh@nvidia.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants