Skip to content

Commit

Permalink
Merge pull request #17277 from FRRouting/mergify/bp/dev/10.2/pr-17157
Browse files Browse the repository at this point in the history
tests: logger masked in topotest.py (backport #17157)
  • Loading branch information
Jafaral authored Oct 28, 2024
2 parents e144cc3 + 9a830b3 commit 354537a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions tests/topotests/lib/topotest.py
Original file line number Diff line number Diff line change
Expand Up @@ -1244,8 +1244,8 @@ def _sysctl_assure(commander, variable, value):
def sysctl_atleast(commander, variable, min_value, raises=False):
try:
if commander is None:
logger = logging.getLogger("topotest")
commander = micronet.Commander("sysctl", logger=logger)
topotest_logger = logging.getLogger("topotest")
commander = micronet.Commander("sysctl", logger=topotest_logger)

return _sysctl_atleast(commander, variable, min_value)
except subprocess.CalledProcessError as error:
Expand All @@ -1262,8 +1262,8 @@ def sysctl_atleast(commander, variable, min_value, raises=False):
def sysctl_assure(commander, variable, value, raises=False):
try:
if commander is None:
logger = logging.getLogger("topotest")
commander = micronet.Commander("sysctl", logger=logger)
topotest_logger = logging.getLogger("topotest")
commander = micronet.Commander("sysctl", logger=topotest_logger)
return _sysctl_assure(commander, variable, value)
except subprocess.CalledProcessError as error:
logger.warning(
Expand Down

0 comments on commit 354537a

Please sign in to comment.