-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Nokia isam #3725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Nokia isam #3725
Conversation
|
@Noppes will try to look at this in the next few days. Feel free to ping me, if this slips through the cracks. |
netmiko/nokia/nokia_isam.py
Outdated
| force_regex: bool = False, | ||
| ) -> bool: | ||
| """Use equivalent enable method.""" | ||
| return super().check_config_mode(check_string=check_string) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this right? It seems inconsistent with set_base_prompt and what I am seeing on the CLI behavior (but really hard for me to find documentation on this platform so maybe I am wrong here).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
default promp is name># and when you go into config mode it becomes name>configure#
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And what happens when to the prompt when you traverse down the configuration tree?
I am having a lot of trouble finding documentation on this platform, but it looks to me like it does something like:
name>configure#
And
configure>system>security#
Note, I definitely could be wrong given the difficult obtaining documentation on this platform.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Noppes Any update on my question here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From the logging the customer send, I only saw name>configure#, but its an easy change to just check for >configure instead, see new commit
Driver for Nokia isam devices