Skip to content

Commit

Permalink
AddRack: Skip multi asic & remove hack (#5254)
Browse files Browse the repository at this point in the history
What is the motivation for this PR?
Test failed in unsupported platform. No more hack mode in testing.

How did you do it?
Skip it explicitly. Remove hackmode flag set to True

How did you verify/test it?
Ran the test in master 3 times successfully.
master.76122-5e1acf022
  • Loading branch information
renukamanavalan authored Mar 7, 2022
1 parent 2a5ad8b commit f066d81
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tests/configlet/test_add_rack.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,9 @@ def test_add_rack(configure_dut, tbinfo, duthosts, rand_one_dut_hostname):
global data_dir, orig_db_dir, clet_db_dir, files_dir

duthost = duthosts[rand_one_dut_hostname]
if duthost.is_multi_asic:
pytest.skip('Generic patch updater does not support multiasic')

log_info("sys.version={}".format(sys.version))
do_test_add_rack(duthost, is_storage_backend = 'backend' in tbinfo['topo']['name'],
hack_apply=True)
do_test_add_rack(duthost, is_storage_backend = 'backend' in tbinfo['topo']['name'])

0 comments on commit f066d81

Please sign in to comment.