Repository navigation
Adopt cloudshell-snmp 6.0 / cloudshell-snmp-autoload 3.0 (pysnmp 7) — Python 3.12/3.13 - #11
Closed
nahumtimerman wants to merge 1 commit into
Closed
nahumtimerman wants to merge 1 commit into
nahumtimerman wants to merge 1 commit into
Conversation
- cloudshell-snmp >=6.0,<7.0 and cloudshell-snmp-autoload >=3.0,<4.0 - GenericSNMPAutoload new API: resource_model passed to the constructor, discover() takes only supported_os - port customization ported from the removed autoload-1.x SnmpIfTable to the 3.x PortsTable service: F5PortsTable keeps dots legal in port names and excludes mgmt/management/loopback/null - standard-telnetlib backport on Python >= 3.13 (telnetlib removed from stdlib, cloudshell-cli 4.x imports it); attrs >=21,<23 - tests: cloudshell-shell-networking-standard >=6.0 (shell-standards 2.x) - Python >=3.9, classifiers 3.9-3.13; tox py39; CI matrix py39 only; isort pre-commit hook 5.10.1 -> 5.12.0 (5.10.1 no longer installs); version 2.1.0 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
|
Superseded by #12, which is the same change pushed as an in-repo branch so the org reusable workflows can check it out (they can't check out forks). Closing this one. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adopts cloudshell-snmp 6.0.0 and cloudshell-snmp-autoload 3.0.0 (pysnmp 7), adding Python 3.12/3.13 support. Validated against the same snmpsim recording set used for the cloudshell-networking-cisco migration (143-device parity gate); the BIG-IP i2600 recording autoloads successfully on Python 3.9 and 3.13.
Changes
requirements.txt:cloudshell-snmp>=6.0,<7.0,cloudshell-snmp-autoload>=3.0,<4.0;attrs>=21,<23;standard-telnetlibon Python >= 3.13 (stdlibtelnetlibwas removed and cloudshell-cli 4.x imports it).BigIPAutoloadFlow: autoload-3.xGenericSNMPAutoloadAPI —resource_modelmoves to the constructor,discover()takes onlysupported_os.SnmpIfTableto the 3.xPortsTableservice: newF5PortsTablekeeps dots legal in F5 port names (e.g.1.1) and excludesmgmt/management/loopback/null, matching the oldF5SnmpIfTablebehavior.F5FirewallGenericSNMPAutoloadnow overridesport_table_serviceinstead ofif_table_service.cloudshell-shell-networking-standard>=6.0,<7.0(shell-standards 2.x).python_requires>=3.9, classifiers 3.9–3.13, version 2.1.0, tox envlistpy39,sdistwithout--format zip, CI matrix py39 only, isort pre-commit hook 5.10.1 -> 5.12.0 (5.10.1 no longer installs).Testing
F5FirewallGenericSNMPAutoload.discoveron both Pythons — 1 chassis, 7 ports (dotted names included, mgmt/loopback excluded), system attributes populated.🤖 Generated with Claude Code