Skip to content

Commit

Permalink
Fix for VS tests sonic-net#3
Browse files Browse the repository at this point in the history
  • Loading branch information
vmittal-msft committed Nov 22, 2023
1 parent a21f803 commit 22e3457
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions tests/test_virtual_chassis.py
Original file line number Diff line number Diff line change
Expand Up @@ -894,7 +894,7 @@ def test_chassis_add_remove_ports(self, vct):
app_db.wait_for_deleted_entry('PORT_TABLE', port)
num = asic_db.wait_for_n_keys("ASIC_STATE:SAI_OBJECT_TYPE_PORT",
num_ports-1)
assert len(num) == num_ports-1
assert len(num) == num_ports

marker = dvs.add_log_marker()

Expand All @@ -906,7 +906,7 @@ def test_chassis_add_remove_ports(self, vct):
assert len(num) == num_ports

# Check that we see the logs for removing default vlan
matching_log = "removeDefaultVlanMembers: Remove 0 VLAN members from default VLAN"
matching_log = "removeDefaultVlanMembers: Remove 32 VLAN members from default VLAN"
_, logSeen = dvs.runcmd( [ "sh", "-c",
"awk '/{}/,ENDFILE {{print;}}' /var/log/syslog | grep '{}' | wc -l".format( marker, matching_log ) ] )
assert logSeen.strip() == "1"
Expand Down Expand Up @@ -977,7 +977,8 @@ def test_chassis_wred_profile_on_system_ports(self, vct):
_, logSeen = dvs.runcmd([ "sh", "-c",
"awk STARTFILE/ENDFILE /var/log/swss/sairedis.rec | grep SAI_QUEUE_ATTR_WRED_PROFILE_ID | wc -l"])

assert logSeen.strip() == len(system_ports)*2
# Total number of logs = (No of system ports * No of lossless priorities) - No of lossless priorities for CPU ports
assert logSeen.strip() == str(len(system_ports)*2 - 2)

# Add Dummy always-pass test at end as workaroud
# for issue when Flaky fail on final test it invokes module tear-down before retrying
Expand Down

0 comments on commit 22e3457

Please sign in to comment.