Skip to content

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

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
AntonHryshchuk:pbh_edit_flow
Mar 31, 2022
Merged

[PBH] Add PBH edit flow validation to test#5263
liat-grozovik merged 1 commit intosonic-net:masterfrom
AntonHryshchuk:pbh_edit_flow

Conversation

@AntonHryshchuk
Copy link
Contributor

Signed-off-by: Anton [email protected]

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

@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 <[email protected]>
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