-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Fortinet does not work when config global not working #2833
Comments
Let me look at it. It has been a long time since that code was written. |
@FloLaco On fortinet, what are the mechanisms for disabling output paging? Can you disable paging if you cant go into |
I have a new Fortinet driver I've been using extensively, should be compatible with VDOM and non-VDOM devices. I'm working with Ktbyers to get it tested and approved. |
Related PR from @Gatorjosh14 |
Hi @ktbyers My fortinet is the property of my WAN carrier, so it's a managed device. |
@FloLaco Are you testing with the current I think it already does this. Though Netmiko probably needs to go into Can you post what error you are receiving? |
I've just tested the
The log :
|
@FloLaco It looks like Do you have |
I probably don't want to support So I don't see a good solution here. You can't determine the output mode without I guess we could try something like a Does that work for you? |
One thing that I don't understand. On my side, I don't have permission to type the config global but the show full-config yes and the output os not paged. |
@FloLaco This is Fortinet V6 only: self._send_command_str(
"config system console", expect_string=self.prompt_pattern
)
output = self._send_command_str(
"show full-configuration", expect_string=self.prompt_pattern
) Doesn't that only give you the configuration of the |
@ktbyers I've access to a fortinet with admin privilege. I'll try to add the fallback code for 'check if a big output if paginated or not' |
I found another issue. When user have read-only permission (cannot type
I'll fix this issue too |
@ktbyers I've fixed the issue like you proposed. You can check the PR I try to make a
I've fix another issue : |
@ktbyers any news about the PR I submited ? |
I got tied up in some other things. I should be able to look at this next
week.
Kirk
On Mon, Mar 6, 2023 at 12:21 AM FLaco ***@***.***> wrote:
@ktbyers <https://github.com/ktbyers> any news about the PR I submited ?
—
Reply to this email directly, view it on GitHub
<#2833 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAG3VBFQRCDWDULHAALVXJDW2WNAXANCNFSM5ZYCJGDA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Kirk Byers
***@***.***
Simplify through Automation
* <https://pynet.twb-tech.com>*
|
@ktbyers One of my colleagues faced a similar issue and I suggested to use |
PR #3248 has now been merged. In general, Netmiko requires output paging to be disabled or it won't work / is not supported (consequently, I am pretty reluctant to add things that require manual handling of output paging). Though I am more open to it, if it is caused by things directly baked into the driver that are hard to work around. I am going to close this issue and the earlier referenced PR. If things are still broken here, just re-open this issue. |
Hello @ktbyers
Just would like to know why you close the session and reopen it if
config global
does not work ?netmiko/netmiko/fortinet/fortinet_ssh.py
Line 54 in 5bbbb73
In my case, this command is not allowed but we are still in vdom mode.
The reconnection does not work because you don't type the 'a' the second time :
The text was updated successfully, but these errors were encountered: