Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Bug]: counterpoll watermark test does not support multi-asic #14753

Open
liamkearney-msft opened this issue Sep 26, 2024 · 1 comment · May be fixed by #14777
Open

[Bug]: counterpoll watermark test does not support multi-asic #14753

liamkearney-msft opened this issue Sep 26, 2024 · 1 comment · May be fixed by #14777
Assignees
@liamkearney-msft

Comments

@liamkearney-msft
Copy link
Contributor

Issue Description

counterpoll watermark tests fail on multi-asic linecards as they do not look at the namespaced database containers. Refactor the tests to support multi-asic

Results you see

COUNTER_DB is not detected as populated in multi asic devices because the test does not look at the database container for each asic

Results you expected to see

COUNTER_DB is populated for each individual linecard, test should look at the proper database container

Is it platform specific

generic

Relevant log output

No response

Output of show version

No response

Attach files (if any)

No response
@liamkearney-msft liamkearney-msft self-assigned this Sep 26, 2024
@auspham
Copy link
Contributor

auspham commented Sep 26, 2024

Hi @liamkearney-msft I'm working on this. Will get your PR review

liamkearney-msft added a commit to liamkearney-msft/sonic-mgmt that referenced this issue Sep 27, 2024
@liamkearney-msft
[platform_tests/counterpoll]: Support multi-asic for counterpoll wate…
8d95d67 
…rmark tests

* Update counterpoll watermark platform test to support multi-asic DUTs

BugLink: sonic-net#14753
Signed-off-by: Liam Kearney <[email protected]>
liamkearney-msft added a commit to liamkearney-msft/sonic-mgmt that referenced this issue Sep 27, 2024
@liamkearney-msft
[platform_tests/counterpoll]: Support multi-asic for counterpoll wate…
46edb3d 
…rmark tests

* Update counterpoll watermark platform test to support multi-asic DUTs

BugLink: sonic-net#14753
Signed-off-by: Liam Kearney <[email protected]>
liamkearney-msft added a commit to liamkearney-msft/sonic-mgmt that referenced this issue Sep 27, 2024
@liamkearney-msft
[platform_tests/counterpoll]: Support multi-asic for counterpoll wate…
ccabaa1 
…rmark tests

* Update counterpoll watermark platform test to support multi-asic DUTs

BugLink: sonic-net#14753
Signed-off-by: Liam Kearney <[email protected]>
liamkearney-msft added a commit to liamkearney-msft/sonic-mgmt that referenced this issue Sep 27, 2024
@liamkearney-msft
[platform_tests/counterpoll]: Support multi-asic for counterpoll wate…
03188f5 
…rmark tests

* Update counterpoll watermark platform test to support multi-asic DUTs

BugLink: sonic-net#14753
Signed-off-by: Liam Kearney <[email protected]>
@liamkearney-msft liamkearney-msft linked a pull request Sep 27, 2024 that will close this issue
Open
6 tasks
liamkearney-msft added a commit to liamkearney-msft/sonic-mgmt that referenced this issue Sep 27, 2024
@liamkearney-msft
Support multi-asic for counterpoll watermark tests
1e66621 
* Update counterpoll watermark platform test to support multi-asic DUTs

BugLink: sonic-net#14753
Signed-off-by: Liam Kearney <[email protected]>
liamkearney-msft added a commit to liamkearney-msft/sonic-mgmt that referenced this issue Sep 29, 2024
@liamkearney-msft
Support multi-asic for counterpoll watermark tests
f89d7e8 
* Update counterpoll watermark platform test to support multi-asic DUTs

BugLink: sonic-net#14753
Signed-off-by: Liam Kearney <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@liamkearney-msft liamkearney-msft

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@auspham @liamkearney-msft