Skip to content

Commit

Permalink
Fix routes/test_route_perf.py (sonic-net#15620)
Browse files Browse the repository at this point in the history
Description of PR
Summary:
Fixes sonic-net#323

Approach
What is the motivation for this PR?
Regression due to sonic-net#15452

How did you do it?
Added missing quotes to the command.

How did you verify/test it?
Ran route/test_route_perf.py on Arista 7260CX3 platform with dualtor topology.

co-authorized by: [email protected]
  • Loading branch information
vivekverma-arista authored Nov 20, 2024
1 parent 7136df6 commit f803ac2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/route/test_route_perf.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ def check_config(duthosts, enum_rand_one_per_hwsku_frontend_hostname, enum_rand_

if (asic == "broadcom"):
broadcom_cmd = "bcmcmd -n " + str(asic_id) if duthost.is_multi_asic else "bcmcmd"
alpm_cmd = "{} {}".format(broadcom_cmd, "conf show l3_alpm_enable")
alpm_cmd = "{} {}".format(broadcom_cmd, '"conf show l3_alpm_enable"')
alpm_enable = duthost.command(alpm_cmd)["stdout_lines"][2].strip()
logger.info("Checking config: {}".format(alpm_enable))
pytest_assert(alpm_enable == "l3_alpm_enable=2", "l3_alpm_enable is not set for route scaling")
Expand Down

0 comments on commit f803ac2

Please sign in to comment.