Skip to content
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

Some (old) netgear switches ignore "terminal length 0" on the show version command #3341

Closed
syn-bit opened this issue Dec 9, 2024 · 5 comments

Comments

@syn-bit
Copy link
Contributor

syn-bit commented Dec 9, 2024

As per #2394 I encountered the same problem. My Netgear GS108Tv2 switch kept timing out, looking at the pcap I can see it that is still uses the pager:

......
(Broadcom FASTPATH Switching) 
Applying Interface configuration, please wait ...
......admin

admin
Password:
<deleted>

***********
(Broadcom FASTPATH Switching) >
enable
.
enable
Password:

.


(Broadcom FASTPATH Switching) #
terminal length 0
.
terminal length 0

(Broadcom FASTPATH Switching) #
show version
.
show version

Switch: 1

System Description............................. GS108Tv2
Machine Type................................... GS108Tv2
Machine Model.................................. GS108T smartSwitch
Serial Number.................................. 29S4115R00309
FRU Number..................................... 
Part Number.................................... BCM53312
Maintenance Level.............................. A
Manufacturer................................... Netgear
Burned In MAC Address.......................... C4:3D:C7:88:17:8C
Software Version............................... 5.4.2.36
Operating System............................... ecos-2.0
Network Processing Device...................... BCM53312_B0

--More-- or (q)uit

The proposed fix in #2394 seems to never been merged, although it does fix this issue for my switch. Perhaps @candlerb never submitted a pull request. Can the fix be added to the netgear model? I can create a PR myself, but I think credits should go where credits are due :-)

@robertcheramy
Copy link
Collaborator

Please submit a PR. You can include @candlerb in the CHANGELOG.md and in the commit log. Submitting the PR is also some work, so it is okay if you get part of the credit :-).

Copy link

This issue is stale because it has been open 90 days with no activity.

@github-actions github-actions bot added the Stale label Mar 11, 2025
syn-bit added a commit to syn-bit/oxidized that referenced this issue Mar 11, 2025
On some Netgear models, disabling the pager does not apply to config
dumps. Add a pager handler to work around this issue.

The fix was proposed by @candlerb in ytti#2391, but never committed.

Fixes: ytti#2391 and ytti#3341
@candlerb
Copy link
Contributor

I didn't submit a PR because at the time telnet wasn't working with expect. I believe that #2668 fixed this.

@syn-bit
Copy link
Contributor Author

syn-bit commented Mar 11, 2025

I didn't submit a PR because at the time telnet wasn't working with expect. I believe that #2668 fixed this.

Then I'm glad #2668 enabled me to use your fix.

Thanks for your fix! And I hope my PR credits you enough :-)

@robertcheramy
Copy link
Collaborator

Fixed by PRs #2668 and #3452

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants