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

Fix an issue of all lldp entries take some time to be in DB after reboot in scaling setup. #15731

Merged
merged 3 commits into from
Dec 11, 2024
Merged

Fix an issue of all lldp entries take some time to be in DB after reboot in scaling setup. #15731

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8f3e098
Fix an issue of all lldp entries take some time to be in DB after reb…
wumiaont Nov 25, 2024
d31ec15
Update test_lldp_syncd.py
wumiaont Nov 25, 2024
7cee239
Fix based on comments
wumiaont Dec 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions tests/lldp/test_lldp_syncd.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -330,11 +330,6 @@ def test_lldp_entry_table_after_reboot(
):
duthost = duthosts[enum_rand_one_per_hwsku_frontend_hostname]

# Verify LLDP_ENTRY_TABLE keys match show lldp table output at the start of test
keys_match = wait_until(30, 5, 0, check_lldp_table_keys, duthost, db_instance)
if not keys_match:
assert keys_match, "LLDP_ENTRY_TABLE keys do not match 'show lldp table' output"

# reboot
logging.info("Run cold reboot on DUT")
reboot(
Expand All @@ -346,6 +341,12 @@ def test_lldp_entry_table_after_reboot(
safe_reboot=True,
check_intf_up_ports=True
)

# Wait till we have all lldp entries in the DB after reboot. It's found in scaling
# setup this may take some time to happen.
keys_match = wait_until(90, 5, 30, check_lldp_table_keys, duthost, db_instance)
if not keys_match:
assert keys_match, "LLDP_ENTRY_TABLE keys do not match 'show lldp table' output"
lldp_entry_keys = get_lldp_entry_keys(db_instance)
lldpctl_output = get_lldpctl_output(duthost)
show_lldp_table_int_list = get_show_lldp_table_output(duthost)
Expand Down
Loading