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.

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

[Bug][T2]: BGP check is too short in wait_bgp_sessions in duthost_utils #16436

Open
Javier-Tan opened this issue Jan 10, 2025 · 0 comments · May be fixed by #16438
Open

[Bug][T2]: BGP check is too short in wait_bgp_sessions in duthost_utils #16436

Javier-Tan opened this issue Jan 10, 2025 · 0 comments · May be fixed by #16438
Assignees

Comments

@Javier-Tan
Copy link
Contributor

Issue Description

Function wait_bgp_sessions in duthost_utils is too short for T2, causing false errors in tests that use them.

Results you see

Test suite test_mgmt_ipv6_only fixture convert_and_restore_config_db_to_ipv6_only uses wait_bgp_sessions, causing the fixture to fail after removing the ipv4 management from the DUTs and doesn't teardown, causing many other tests to fail alongside it.

Results you expected to see

wait_bgp_sessions should wait longer for T2 to give an accurate result

Is it platform specific

generic

Relevant log output

N/A

Output of show version

N/A

Attach files (if any)

N/A

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

1 participant