Skip to content

Conversation

gmaiyani-crest
Copy link

@gmaiyani-crest gmaiyani-crest commented Sep 25, 2025

Description

  • Replace the pysnmplib with lextudio's pysmp
  • make sc4snmp compatible with python v3.13
  • changed the implementation of walk operation (bulkCmd to bulk_walk_cmd) and add the concurrency support on bulk_walk_cmd SNMP request.

Type of change

Please delete options that are not relevant.

  • Dependency update
  • New feature
  • Refactor/improvement
  • This change requires a documentation update

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

  • My commit message is conventional
  • I have run pre-commit on all files before creating the PR
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes

cwadhwani-splunk and others added 7 commits September 17, 2025 15:24
* feat: mTLS for Splunk 10 (#1197)

* feat: mTLS support for Splunk 10

* docs: fix typo

---------

Co-authored-by: Olga <[email protected]>
Co-authored-by: ajasnosz <[email protected]>

* chore(release): 1.13.0-beta.1

# [1.13.0-beta.1](v1.12.3...v1.13.0-beta.1) (2025-06-10)

### Features

* mTLS for Splunk 10 ([#1197](#1197)) ([e8cb2a4](e8cb2a4))

* chore(deps): update docker/login-action action to v3.4.0 (#1166)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: python and dependency update (#1200)

* chore(release): 1.13.0-beta.2

# [1.13.0-beta.2](v1.13.0-beta.1...v1.13.0-beta.2) (2025-06-10)

### Bug Fixes

* python and dependency update ([#1200](#1200)) ([dd39338](dd39338))

* chore: add missing sudo in integration test script (#1207)

* chore: update microk8s (#1206)

* chore: update microk8s
* chore: add changelog

* chore(deps): update dependency urllib3 to v2 [security] (#1205)

* chore(deps): update dependency urllib3 to v2 [security]

* test: test [run-int-tests]

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: omrozowicz-splunk <[email protected]>

* fix: replace - with _ for metrics (#1213)

* fix: add option to change "-" to an "_" to be compatible with metric schema in Splunk

* fix: doc fix and docker-compose change

* fix: add docs

* chore: pre-commit run

* doc: refine

* doc: address PR comments, update CHANGELOG

* chore(release): 1.13.0-beta.3

# [1.13.0-beta.3](v1.13.0-beta.2...v1.13.0-beta.3) (2025-07-09)

### Bug Fixes

* replace - with _ for metrics ([#1213](#1213)) ([cd44b56](cd44b56))

* feat: add partial walk (#1215)

* chore(release): 1.13.0-beta.4

# [1.13.0-beta.4](v1.13.0-beta.3...v1.13.0-beta.4) (2025-07-22)

### Features

* add partial walk ([#1215](#1215)) ([0e1f32f](0e1f32f))

* test: parallelised integration tests (#1220)

* test: add one more worker

* chore: add parallelism to docker-compose tests

* chore: move some polling tests to worker

* fix: problem with casting strings of scientific notation (#1219)

* fix: problems with interpreting scientific value as float

* fix: unnecessary import

* fix: run pre-commit

* fix: check regex only if value is an instance of str

* fix: wrong function

* test: delete unnecessary test case

* test: rename test file

* chore: run pre-commit

* chore: update CHANGELOG

* chore(release): 1.13.0-beta.5

# [1.13.0-beta.5](v1.13.0-beta.4...v1.13.0-beta.5) (2025-07-30)

### Bug Fixes

* problem with casting strings of scientific notation ([#1219](#1219)) ([80818aa](80818aa))

* fix: add ipv6 flag for worker poller (#1218)

* chore(release): 1.13.0-beta.6

# [1.13.0-beta.6](v1.13.0-beta.5...v1.13.0-beta.6) (2025-07-31)

### Bug Fixes

* add ipv6 flag for worker poller ([#1218](#1218)) ([57d44ae](57d44ae))

* chore(deps): update dependency mkdocs-material to v9.6.16 (#1211)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: fix license accept issue in integration tests (#1227)

* fix: logging (#1223)

* fix: disable mongo logs, fix traps logging

* fix: run pre-commit, fix conflicts

* fix: run pre-commit

* fix: make mongo logging configurable

* fix: make mongo logging configurable

* fix: add docs

* fix: fix traps helm template 

* fix: add CHANGELOG

* chore(release): 1.13.0-beta.7

# [1.13.0-beta.7](v1.13.0-beta.6...v1.13.0-beta.7) (2025-08-13)

### Bug Fixes

* logging ([#1223](#1223)) ([a98987c](a98987c))

* fix: update dependencies (#1229)

* chore(release): 1.13.0-beta.8

# [1.13.0-beta.8](v1.13.0-beta.7...v1.13.0-beta.8) (2025-08-13)

### Bug Fixes

* update dependencies ([#1229](#1229)) ([fcf8805](fcf8805))

* fix: traps add engine contextId (#1230)

* fix: add decode method security engine ID

* chore: add missing import

* fix: unit tests

* fix: run pre-commit

* fix: update docker env file, docs and changelog

* fix: update method description

---------

Co-authored-by: ajasnosz <[email protected]>

* chore(release): 1.13.0-beta.9

# [1.13.0-beta.9](v1.13.0-beta.8...v1.13.0-beta.9) (2025-08-14)

### Bug Fixes

* traps add engine contextId ([#1230](#1230)) ([3dff9cd](3dff9cd))

* feat: merged the improved polling changes to main build

* chore: added existing variables

* chore: removing unnecessary fields

---------

Co-authored-by: Ilya Kheifets <[email protected]>
Co-authored-by: Olga <[email protected]>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: srv-rr-github-token <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: omrozowicz-splunk <[email protected]>
# [1.14.0-beta.1](v1.13.0...v1.14.0-beta.1) (2025-09-17)

### Features

* merged the improved polling changes to main build ([#1239](#1239)) ([6f6b4d0](6f6b4d0)), closes [#1197](#1197)
* fix: migrating bitnami to bitnamilegacy for redis and mongodb

* chore: executed make render

* chore: empty commit to run integration tests [run-int-tests]
# [1.14.0-beta.2](v1.14.0-beta.1...v1.14.0-beta.2) (2025-09-24)

### Bug Fixes

* migrating bitnami to bitnamilegacy for redis and mongodb ([#1245](#1245)) ([0de679b](0de679b))
* chore: update develop

---------

Co-authored-by: Olga <[email protected]>
Co-authored-by: ajasnosz <[email protected]>

* chore(release): 1.13.0-beta.1

# [1.13.0-beta.1](v1.12.3...v1.13.0-beta.1) (2025-06-10)

### Features

* mTLS for Splunk 10 ([#1197](#1197)) ([e8cb2a4](e8cb2a4))

* chore(deps): update docker/login-action action to v3.4.0 (#1166)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: python and dependency update (#1200)

* chore(release): 1.13.0-beta.2

# [1.13.0-beta.2](v1.13.0-beta.1...v1.13.0-beta.2) (2025-06-10)

### Bug Fixes

* python and dependency update ([#1200](#1200)) ([dd39338](dd39338))

* chore: add missing sudo in integration test script (#1207)

* chore: update microk8s (#1206)

* chore: update microk8s
* chore: add changelog

* chore(deps): update dependency urllib3 to v2 [security] (#1205)

* chore(deps): update dependency urllib3 to v2 [security]

* test: test [run-int-tests]

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: omrozowicz-splunk <[email protected]>

* fix: replace - with _ for metrics (#1213)

* fix: add option to change "-" to an "_" to be compatible with metric schema in Splunk

* fix: doc fix and docker-compose change

* fix: add docs

* chore: pre-commit run

* doc: refine

* doc: address PR comments, update CHANGELOG

* chore(release): 1.13.0-beta.3

# [1.13.0-beta.3](v1.13.0-beta.2...v1.13.0-beta.3) (2025-07-09)

### Bug Fixes

* replace - with _ for metrics ([#1213](#1213)) ([cd44b56](cd44b56))

* feat: add partial walk (#1215)

* chore(release): 1.13.0-beta.4

# [1.13.0-beta.4](v1.13.0-beta.3...v1.13.0-beta.4) (2025-07-22)

### Features

* add partial walk ([#1215](#1215)) ([0e1f32f](0e1f32f))

* test: parallelised integration tests (#1220)

* test: add one more worker

* chore: add parallelism to docker-compose tests

* chore: move some polling tests to worker

* fix: problem with casting strings of scientific notation (#1219)

* fix: problems with interpreting scientific value as float

* fix: unnecessary import

* fix: run pre-commit

* fix: check regex only if value is an instance of str

* fix: wrong function

* test: delete unnecessary test case

* test: rename test file

* chore: run pre-commit

* chore: update CHANGELOG

* chore(release): 1.13.0-beta.5

# [1.13.0-beta.5](v1.13.0-beta.4...v1.13.0-beta.5) (2025-07-30)

### Bug Fixes

* problem with casting strings of scientific notation ([#1219](#1219)) ([80818aa](80818aa))

* fix: add ipv6 flag for worker poller (#1218)

* chore(release): 1.13.0-beta.6

# [1.13.0-beta.6](v1.13.0-beta.5...v1.13.0-beta.6) (2025-07-31)

### Bug Fixes

* add ipv6 flag for worker poller ([#1218](#1218)) ([57d44ae](57d44ae))

* chore(deps): update dependency mkdocs-material to v9.6.16 (#1211)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: fix license accept issue in integration tests (#1227)

* fix: logging (#1223)

* fix: disable mongo logs, fix traps logging

* fix: run pre-commit, fix conflicts

* fix: run pre-commit

* fix: make mongo logging configurable

* fix: make mongo logging configurable

* fix: add docs

* fix: fix traps helm template 

* fix: add CHANGELOG

* chore(release): 1.13.0-beta.7

# [1.13.0-beta.7](v1.13.0-beta.6...v1.13.0-beta.7) (2025-08-13)

### Bug Fixes

* logging ([#1223](#1223)) ([a98987c](a98987c))

* fix: update dependencies (#1229)

* chore(release): 1.13.0-beta.8

# [1.13.0-beta.8](v1.13.0-beta.7...v1.13.0-beta.8) (2025-08-13)

### Bug Fixes

* update dependencies ([#1229](#1229)) ([fcf8805](fcf8805))

* fix: traps add engine contextId (#1230)

* fix: add decode method security engine ID

* chore: add missing import

* fix: unit tests

* fix: run pre-commit

* fix: update docker env file, docs and changelog

* fix: update method description

---------

Co-authored-by: ajasnosz <[email protected]>

* chore(release): 1.13.0-beta.9

# [1.13.0-beta.9](v1.13.0-beta.8...v1.13.0-beta.9) (2025-08-14)

### Bug Fixes

* traps add engine contextId ([#1230](#1230)) ([3dff9cd](3dff9cd))

* feat: merged the improved polling changes to main build

* chore: added existing variables

* chore: removing unnecessary fields

---------

Co-authored-by: Ilya Kheifets <[email protected]>
Co-authored-by: Olga <[email protected]>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: srv-rr-github-token <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: omrozowicz-splunk <[email protected]>

* chore(release): 1.14.0-beta.1

# [1.14.0-beta.1](v1.13.0...v1.14.0-beta.1) (2025-09-17)

### Features

* merged the improved polling changes to main build ([#1239](#1239)) ([6f6b4d0](6f6b4d0)), closes [#1197](#1197)

* fix: migrating bitnami to bitnamilegacy for redis and mongodb (#1245)

* fix: migrating bitnami to bitnamilegacy for redis and mongodb

* chore: executed make render

* chore: empty commit to run integration tests [run-int-tests]

* chore: update CHANGELOG (#1246)

* chore(release): 1.14.0-beta.2

# [1.14.0-beta.2](v1.14.0-beta.1...v1.14.0-beta.2) (2025-09-24)

### Bug Fixes

* migrating bitnami to bitnamilegacy for redis and mongodb ([#1245](#1245)) ([0de679b](0de679b))

---------

Co-authored-by: cwadhwani-splunk <[email protected]>
Co-authored-by: Ilya Kheifets <[email protected]>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: srv-rr-github-token <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ajasnosz <[email protected]>

* Revert "Release 1.14.0 (#1247)" (#1249)

This reverts commit 1a0c9ff.

* chore(release): 1.13.1

## [1.13.1](v1.13.0...v1.13.1) (2025-09-25)

### Reverts

* Revert "Release 1.14.0 (#1247)" (#1249) ([a55ce6f](a55ce6f)), closes [#1247](#1247) [#1249](#1249)

* feat: merged the improved polling changes to main build (#1239)

* feat: mTLS for Splunk 10 (#1197)

* feat: mTLS support for Splunk 10

* docs: fix typo

---------

Co-authored-by: Olga <[email protected]>
Co-authored-by: ajasnosz <[email protected]>

* chore(release): 1.13.0-beta.1

# [1.13.0-beta.1](v1.12.3...v1.13.0-beta.1) (2025-06-10)

### Features

* mTLS for Splunk 10 ([#1197](#1197)) ([e8cb2a4](e8cb2a4))

* chore(deps): update docker/login-action action to v3.4.0 (#1166)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: python and dependency update (#1200)

* chore(release): 1.13.0-beta.2

# [1.13.0-beta.2](v1.13.0-beta.1...v1.13.0-beta.2) (2025-06-10)

### Bug Fixes

* python and dependency update ([#1200](#1200)) ([dd39338](dd39338))

* chore: add missing sudo in integration test script (#1207)

* chore: update microk8s (#1206)

* chore: update microk8s
* chore: add changelog

* chore(deps): update dependency urllib3 to v2 [security] (#1205)

* chore(deps): update dependency urllib3 to v2 [security]

* test: test [run-int-tests]

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: omrozowicz-splunk <[email protected]>

* fix: replace - with _ for metrics (#1213)

* fix: add option to change "-" to an "_" to be compatible with metric schema in Splunk

* fix: doc fix and docker-compose change

* fix: add docs

* chore: pre-commit run

* doc: refine

* doc: address PR comments, update CHANGELOG

* chore(release): 1.13.0-beta.3

# [1.13.0-beta.3](v1.13.0-beta.2...v1.13.0-beta.3) (2025-07-09)

### Bug Fixes

* replace - with _ for metrics ([#1213](#1213)) ([cd44b56](cd44b56))

* feat: add partial walk (#1215)

* chore(release): 1.13.0-beta.4

# [1.13.0-beta.4](v1.13.0-beta.3...v1.13.0-beta.4) (2025-07-22)

### Features

* add partial walk ([#1215](#1215)) ([0e1f32f](0e1f32f))

* test: parallelised integration tests (#1220)

* test: add one more worker

* chore: add parallelism to docker-compose tests

* chore: move some polling tests to worker

* fix: problem with casting strings of scientific notation (#1219)

* fix: problems with interpreting scientific value as float

* fix: unnecessary import

* fix: run pre-commit

* fix: check regex only if value is an instance of str

* fix: wrong function

* test: delete unnecessary test case

* test: rename test file

* chore: run pre-commit

* chore: update CHANGELOG

* chore(release): 1.13.0-beta.5

# [1.13.0-beta.5](v1.13.0-beta.4...v1.13.0-beta.5) (2025-07-30)

### Bug Fixes

* problem with casting strings of scientific notation ([#1219](#1219)) ([80818aa](80818aa))

* fix: add ipv6 flag for worker poller (#1218)

* chore(release): 1.13.0-beta.6

# [1.13.0-beta.6](v1.13.0-beta.5...v1.13.0-beta.6) (2025-07-31)

### Bug Fixes

* add ipv6 flag for worker poller ([#1218](#1218)) ([57d44ae](57d44ae))

* chore(deps): update dependency mkdocs-material to v9.6.16 (#1211)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: fix license accept issue in integration tests (#1227)

* fix: logging (#1223)

* fix: disable mongo logs, fix traps logging

* fix: run pre-commit, fix conflicts

* fix: run pre-commit

* fix: make mongo logging configurable

* fix: make mongo logging configurable

* fix: add docs

* fix: fix traps helm template 

* fix: add CHANGELOG

* chore(release): 1.13.0-beta.7

# [1.13.0-beta.7](v1.13.0-beta.6...v1.13.0-beta.7) (2025-08-13)

### Bug Fixes

* logging ([#1223](#1223)) ([a98987c](a98987c))

* fix: update dependencies (#1229)

* chore(release): 1.13.0-beta.8

# [1.13.0-beta.8](v1.13.0-beta.7...v1.13.0-beta.8) (2025-08-13)

### Bug Fixes

* update dependencies ([#1229](#1229)) ([fcf8805](fcf8805))

* fix: traps add engine contextId (#1230)

* fix: add decode method security engine ID

* chore: add missing import

* fix: unit tests

* fix: run pre-commit

* fix: update docker env file, docs and changelog

* fix: update method description

---------

Co-authored-by: ajasnosz <[email protected]>

* chore(release): 1.13.0-beta.9

# [1.13.0-beta.9](v1.13.0-beta.8...v1.13.0-beta.9) (2025-08-14)

### Bug Fixes

* traps add engine contextId ([#1230](#1230)) ([3dff9cd](3dff9cd))

* feat: merged the improved polling changes to main build

* chore: added existing variables

* chore: removing unnecessary fields

---------

Co-authored-by: Ilya Kheifets <[email protected]>
Co-authored-by: Olga <[email protected]>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: srv-rr-github-token <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: omrozowicz-splunk <[email protected]>

* chore(release): 1.14.0-beta.1

# [1.14.0-beta.1](v1.13.0...v1.14.0-beta.1) (2025-09-17)

### Features

* merged the improved polling changes to main build ([#1239](#1239)) ([6f6b4d0](6f6b4d0)), closes [#1197](#1197)

* fix: migrating bitnami to bitnamilegacy for redis and mongodb (#1245)

* fix: migrating bitnami to bitnamilegacy for redis and mongodb

* chore: executed make render

* chore: empty commit to run integration tests [run-int-tests]

* chore: update CHANGELOG (#1246)

* chore(release): 1.14.0-beta.2

# [1.14.0-beta.2](v1.14.0-beta.1...v1.14.0-beta.2) (2025-09-24)

### Bug Fixes

* migrating bitnami to bitnamilegacy for redis and mongodb ([#1245](#1245)) ([0de679b](0de679b))

---------

Co-authored-by: cwadhwani-splunk <[email protected]>
Co-authored-by: Ilya Kheifets <[email protected]>
Co-authored-by: ajasnosz <[email protected]>
Co-authored-by: srv-rr-github-token <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ajasnosz <[email protected]>
make sc4snmp compatible with python v3.13
changed the implementation of walk operation
@gmaiyani-crest gmaiyani-crest changed the title Feat/replace the pysnmplib to lextudio pysnmp feat: replace the pysnmplib to lextudio's pysnmp Sep 25, 2025
@gmaiyani-crest gmaiyani-crest self-assigned this Sep 25, 2025
Base automatically changed from develop to main September 25, 2025 12:41
@omrozowicz-splunk omrozowicz-splunk changed the base branch from main to develop September 25, 2025 13:12
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 55d7ed6 to 354c35c Compare September 25, 2025 13:38
…nto feat/replace-the-pysnmplib-to-lextudio-pysnmp
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 43deb4f to ef70416 Compare September 25, 2025 14:31
@gmaiyani-crest gmaiyani-crest marked this pull request as ready for review September 25, 2025 14:55
@gmaiyani-crest gmaiyani-crest marked this pull request as draft September 26, 2025 07:42
@gmaiyani-crest gmaiyani-crest marked this pull request as ready for review September 28, 2025 11:41
@omrozowicz-splunk
Copy link
Contributor

Can you run integration tests? Push empty commit with [run-int-tests] tag

@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 358ce0e to ac1ab92 Compare September 29, 2025 08:01
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch 3 times, most recently from f055468 to 52feaf0 Compare September 29, 2025 13:38
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch 2 times, most recently from cd1a594 to 20681ce Compare October 1, 2025 09:53
@gmaiyani-crest gmaiyani-crest changed the base branch from main to develop October 7, 2025 13:38
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch 6 times, most recently from fb54085 to fc9d1c7 Compare October 8, 2025 06:43
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from fc9d1c7 to b07aaad Compare October 8, 2025 09:25
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch 2 times, most recently from d091afd to 4cd7c51 Compare October 9, 2025 06:28
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 4cd7c51 to b1aff79 Compare October 9, 2025 08:43
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch 2 times, most recently from 79af6ca to 89bce2b Compare October 9, 2025 19:04
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 89bce2b to 0ba6233 Compare October 10, 2025 05:51
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 08a4b01 to 53e70ec Compare October 11, 2025 10:33
@gmaiyani-crest gmaiyani-crest force-pushed the feat/replace-the-pysnmplib-to-lextudio-pysnmp branch from 0da52bf to 1d983d2 Compare October 13, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants