Skip to content

Commit

Permalink
[tacacs] Remove useless params in test_ro_disk (#7584)
Browse files Browse the repository at this point in the history
What is the motivation for this PR?
There are useless params in function do_reboot()

How did you do it?
Remove useless params in do_reboot()

How did you verify/test it?
Run tests

Signed-off-by: Yaqiang Zhu <[email protected]>
  • Loading branch information
yaqiangz authored Feb 28, 2023
1 parent 37273ae commit c2a86d2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/tacacs/test_ro_disk.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ def chk_ssh_remote_run(localhost, remote_ip, username, password, cmd):
return rc == 0


def do_reboot(duthost, localhost, duthosts, rw_user="", rw_pass=""):
def do_reboot(duthost, localhost, duthosts):
# occasionally reboot command fails with some kernel error messages
# Hence retry if needed.
#
Expand Down Expand Up @@ -213,6 +213,6 @@ def test_ro_disk(localhost, ptfhost, duthosts, enum_rand_one_per_hwsku_hostname,
finally:
logger.debug("START: reboot {} to restore disk RW state".
format(enum_rand_one_per_hwsku_hostname))
do_reboot(duthost, localhost, duthosts, rw_user, rw_pass)
do_reboot(duthost, localhost, duthosts)
logger.debug(" END: reboot {} to restore disk RW state".
format(enum_rand_one_per_hwsku_hostname))

0 comments on commit c2a86d2

Please sign in to comment.