You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
while i'm playing with my driver fork ... it's probably nothing, but it doesn't make much sense to me. See the if ttl block.
regards
ios.py:L3196 [traceroute()]
if ttl:
if isinstance(ttl, int) and 0 <= timeout <= 255:
command += " ttl 0 {}".format(str(ttl))
if timeout:
# Timeout should be an integer between 1 and 3600
if isinstance(timeout, int) and 1 <= timeout <= 3600:
command += " timeout {}".format(str(timeout))
Yes, I think that looks a bit 🐟 - at least the command += " ttl 0 {}".format(str(ttl)) which should probably be command += " ttl {}".format(ttl) instead. Feel free to open a PR for that @remingu.
Description of Issue/Question
Hi,
while i'm playing with my driver fork ... it's probably nothing, but it doesn't make much sense to me. See the if ttl block.
regards
ios.py:L3196 [traceroute()]
Did you follow the steps from https://github.com/napalm-automation/napalm#faq
(Place an
x
between the square brackets where applicable)Setup
napalm version
(Paste verbatim output from
pip freeze | grep napalm
between quotes below)Network operating system version
(Paste verbatim output from
show version
- or equivalent - between quotes below)Steps to Reproduce the Issue
n/a
Error Traceback
(Paste the complete traceback of the exception between quotes below)
The text was updated successfully, but these errors were encountered: