-
Notifications
You must be signed in to change notification settings - Fork 22
feat: replaced pysnmplib with lextudio's pysnmp v7.1.8 #1276
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
base: feat/support-of-python-3.13-and-replaced-pysnmplib-to-lextudio-pysnmp
Are you sure you want to change the base?
Conversation
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
92dcd10 to
c94195a
Compare
…lextudio's pysnmp v7.1.0 - Replaced pysnmplib with lextudio's pysmp v7.1.0 - Moved from pysnmp-pyasn1 to lextudio's pyasn1 - Moved from pysnmp-pysmi to lextudio's pysmi - Included pysnmpcrypto (maintained by lextudio's) for stronger crypto algorithms to used with pysnmp v6+ - Updated SNMP walk implementation from bulkCmd to bulkWalkCmd - Added concurrency support for bulkWalkCmd-based SNMP requests - Made SC4SNMP compatible with Python 3.13 - Updated SNMP walk implementation from bulkCmd to bulkWalkCmd - Added concurrency support for bulkWalkCmd-based SNMP requests - Introduced deploymentUser in values.yaml - Added DEPLOYMENT_USER env variable for Docker Compose deployment [run-int-tests]
c94195a to
cde346d
Compare
omrozowicz-splunk
left a comment
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.
Looks good, please check my comments
Yes, the coverage is the same as before. |
…nt-tests] Signed-off-by: gmaiyani-crest <[email protected]>
d2f4532 to
4f14498
Compare
Signed-off-by: gmaiyani-crest <[email protected]>
dc112b6 to
eaaa62e
Compare
62784f2 to
dcd181c
Compare
Description
Adding support of python v3.13 and migrating from pysnmplib to lextudio's pysnmp v7.1.8
bulkCmdto custommulti_bulk_walk_cmddeploymentUserin values.yaml to make the sc4snmp entirely compatible with lextudio's pysnmp.DEPLOYMENT_USERvariable for Docker Compose deployment.Notes
Added custom multi_bulk_walk_cmd function for Multi-Varbind SNMP Walking.
Why?
However, to achieve the same behavior for multiple varbinds (from different OID subtrees), each varbind must be walked separately in sequence.
bulkCmd.Key Changes
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist