-
Notifications
You must be signed in to change notification settings - Fork 727
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 for align watermark flow with port configuration #6907
test for align watermark flow with port configuration #6907
Conversation
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
b89e20d
to
2200675
Compare
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
2200675
to
a18b4b1
Compare
The pre-commit check detected issues in the files touched by this pull request. For old issues, it is not mandatory to fix them because they were not caused by this change. It is unfair to blame Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
a18b4b1
to
ea8dc0a
Compare
ea8dc0a
to
ce69c34
Compare
ce69c34
to
4841b9a
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
What is the motivation for this PR? PR #6907 introduced the following error: _ ERROR collecting tests/platform_tests/counterpoll/test_counterpoll_watermark.py _ Fixture "backup_and_restore_config_db" called directly. Fixtures are not meant to be called directly, but are created automatically when test functions request them as parameters. How did you do it? Remove unnecessary usefixtures. How did you verify/test it? run platform_tests/counterpoll/test_counterpoll_watermark.py
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
add a verification test for feature align watermark flow with port configuration
How did you do it?
new verification test
How did you verify/test it?
run it on several switches
Any platform specific information?
not platform specific
Supported testbed topology if it's a new test case?
any topology
Documentation