Skip to content

Commit

Permalink
[config reload]Modifying config reload tests to align with latest con…
Browse files Browse the repository at this point in the history
…fig reload enhancement (#7558)

### Approach
#### What is the motivation for this PR?
Modifying config reload tests to align with latest config reload enhancement according to HLD sonic-net/SONiC#1203

#### How did you do it?
Removed the timer based checks

#### How did you verify/test it?
Run the tests with config reload enhamcenet changes
  • Loading branch information
dgsudharsan committed Apr 11, 2023
1 parent 708ffc3 commit a0cc0c2
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions tests/common/config_reload.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,16 +43,6 @@ def config_system_checks_passed(duthost):
if int(out['stdout'].strip()) < 120:
return False

logging.info("Checking if delayed services are up")
out = duthost.shell("systemctl list-dependencies sonic-delayed.target --plain |sed '1d'")
status = duthost.shell("systemctl is-enabled {}".format(out['stdout'].replace("\n", " ")))
services = [line.strip() for line in out['stdout'].splitlines()]
state = [line.strip() for line in status['stdout'].splitlines()]
for service in services:
if state[services.index(service)] == "enabled":
out1 = duthost.shell("systemctl show {} --property=LastTriggerUSecMonotonic --value".format(service))
if out1['stdout'].strip() == "0":
return False
logging.info("All checks passed")
return True

Expand Down

0 comments on commit a0cc0c2

Please sign in to comment.