feat: support of python 3.13 and replaced pysnmplib to lextudio's pysnmp

.github/workflows/ci-docker-deployment.yaml

@@ -14,7 +14,7 @@ jobs:
     strategy:
       matrix:
         python-version:
-          - 3.10
+          - 3.13
     steps:
       - name: Check out code
         uses: actions/checkout@v4

.github/workflows/ci-main.yaml

@@ -70,7 +70,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with:
-          python-version: "3.10"
+          python-version: "3.13"
       - uses: pre-commit/[email protected]
 
   trivy-scan:
@@ -102,7 +102,7 @@ jobs:
     strategy:
       matrix:
         python-version:
-          - "3.10"
+          - "3.13"
     steps:
       - uses: actions/checkout@v4
       - name: Setup python
@@ -197,7 +197,7 @@ jobs:
       - name: Setup python
         uses: actions/setup-python@v5
         with:
-          python-version: "3.10"
+          python-version: "3.13"
       - name: Install docker compose
         run: |
           # Add Docker's official GPG key:

.github/workflows/ci-ui-tests.yaml

@@ -64,7 +64,7 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v5
         with:
-          python-version: "3.10"
+          python-version: "3.13"
 
       - name: remove not used docker images
         run: |  

.github/workflows/mike.yaml

@@ -40,7 +40,7 @@ jobs:
           token: "${{ secrets.PAT_CLATOOL }}"
       - uses: actions/setup-python@v5
         with:
-          python-version: "3.10"
+          python-version: "3.13"
       - name: Upload Docs
         run: |
           sudo apt update

.github/workflows/offline-installation.yaml

@@ -14,7 +14,7 @@ jobs:
     strategy:
       matrix:
         python-version:
-          - "3.10"
+          - "3.13"
     steps:
       - name: Check out code
         uses: actions/checkout@v4

CHANGELOG.md

@@ -9,6 +9,7 @@
   - Implemented automatic data migration from Bitnami deployments (PVC reuse)
   - Enabled AOF persistence by default for data durability
 - add CounterBasedGauge64 and ZeroBasedCounter64 as metrics types
+- add SNMP-enabled device discovery feature
 
 ### Fixes
 - fix problem with service rendering when `traps.service.usemetallb` is set to false

Dockerfile

@@ -1,10 +1,11 @@
-FROM python:3.10-alpine AS base
+FROM python:3.13-alpine AS base
 
 ENV PYTHONFAULTHANDLER=1 \
     PYTHONHASHSEED=random \
     PYTHONUNBUFFERED=1
 RUN apk add -U git sqlite-dev
 RUN pip install --upgrade setuptools pip
+RUN apk add --no-cache nmap
 RUN mkdir /app
 WORKDIR /app
 

charts/splunk-connect-for-snmp/Chart.lock

@@ -5,5 +5,5 @@ dependencies:
 - name: mibserver
   repository: https://pysnmp.github.io/mibs/charts/
   version: 1.15.25
-digest: sha256:747fcedec83bf0d80600166a021b35436d8d2ea877b60e9a43044ed2140cf1c5
-generated: "2025-10-13T12:15:04.255986+02:00"
+digest: sha256:204f5bac63adfed27167a2d1f76682400e98ff9dd39bb319357633de25f91525
+generated: "2025-11-06T09:07:30.615857862Z"

charts/splunk-connect-for-snmp/templates/NOTES.txt

@@ -1,2 +1,3 @@
 Default walk no longer calls full oid tree, instead it is collecting only 'SNMPv2-MIB'.
-If you want to call full oid for the devices, you have to set enableFullWalk flag to true.
+If you want to call full oid for the devices, you have to set enableFullWalk flag to true.
+If the discovery feature is enabled, the time required may increase depending on the number of devices in the specified subnet.

charts/splunk-connect-for-snmp/templates/_helpers.tpl

@@ -96,6 +96,17 @@ Whether enable polling
 {{- end -}}
 {{- end }}
 
+{{/*
+Whether enable discovery
+*/}}
+{{- define "splunk-connect-for-snmp.discovery.enable" -}}
+{{- if .Values.discovery.enabled }}
+{{- printf "true" }}
+{{- else }}
+{{- printf "false" }}
+{{- end -}}
+{{- end }}
+
 {{- /*
 Generate Redis environment variables for application pods
 */ -}}

charts/splunk-connect-for-snmp/templates/common/discovery-config.yaml

@@ -0,0 +1,9 @@
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: {{ include "splunk-connect-for-snmp.name" . }}-discovery-config
+data:
+  discovery-config.yaml: |
+{{- if .Values.discovery }}
+{{ .Values.discovery | toYaml | indent 4 }}
+{{- end }}

charts/splunk-connect-for-snmp/templates/discovery/_helpers.tpl

@@ -0,0 +1,50 @@
+{{/*
+Expand the name of the chart.
+*/}}
+{{- define "splunk-connect-for-snmp.discovery.name" -}}
+{{- default (printf "%s-%s" .Chart.Name "discovery")  .Values.discovery.nameOverride | trunc 63 | trimSuffix "-" }}
+{{- end }}
+
+{{/*
+Create a default fully qualified app name.
+We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
+If release name contains chart name it will be used as a full name.
+*/}}
+{{- define "splunk-connect-for-snmp.discovery.fullname" -}}
+{{- if .Values.discovery.fullnameOverride }}
+{{- .Values.discovery.fullnameOverride | trunc 63 | trimSuffix "-" }}
+{{- else }}
+{{- $name := default (printf "%s-%s" .Chart.Name "discovery") .Values.discovery.nameOverride }}
+{{- if contains $name .Release.Name }}
+{{- .Release.Name | trunc 63 | trimSuffix "-" }}
+{{- else }}
+{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" }}
+{{- end }}
+{{- end }}
+{{- end }}
+
+{{/*
+Not using it anywhere
+*/}}
+{{/*
+Create chart name and version as used by the chart label.
+*/}}
+{{- define "splunk-connect-for-snmp.discovery.chart" -}}
+{{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" }}
+{{- end }}
+
+{{/*
+Selector labels
+*/}}
+{{- define "splunk-connect-for-snmp.discovery.selectorLabels" -}}
+app.kubernetes.io/name: {{ include "splunk-connect-for-snmp.discovery.name" . }}
+app.kubernetes.io/instance: {{ .Release.Name }}
+{{- end }}
+
+{{/*
+Common labels
+*/}}
+{{- define "splunk-connect-for-snmp.discovery.labels" -}}
+{{ include "splunk-connect-for-snmp.discovery.selectorLabels" . }}
+{{ include "splunk-connect-for-snmp.labels" . }}
+{{- end }}

charts/splunk-connect-for-snmp/templates/discovery/job.yaml

@@ -0,0 +1,78 @@
+{{- if eq (include "splunk-connect-for-snmp.discovery.enable" .) "true" }}
+apiVersion: batch/v1
+kind: Job
+metadata:
+  name: {{ include "splunk-connect-for-snmp.discovery.fullname" . }}
+  labels:
+    {{- include "splunk-connect-for-snmp.discovery.labels" . | nindent 4 }}
+spec:
+  ttlSecondsAfterFinished: 300
+  template:
+    metadata:
+      # {{- with .Values.inventory.podAnnotations }}
+      # annotations:      
+      #   {{- toYaml . | nindent 8 }}
+      # {{- end }}
+
+      labels:
+        {{- include "splunk-connect-for-snmp.discovery.selectorLabels" . | nindent 8 }}
+    spec:
+      {{- with .Values.imagePullSecrets }}
+      imagePullSecrets:
+        {{- toYaml . | nindent 8 }}
+      {{- end }}
+      containers:
+        - name: {{ .Chart.Name }}-discovery
+          image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
+          imagePullPolicy: {{ .Values.image.pullPolicy }}
+          args:
+              ["discovery"]
+          env:
+        {{- if .Values.redis.auth.enabled }}
+          - name: REDIS_PASSWORD
+            valueFrom:
+              secretKeyRef:
+                {{- if .Values.redis.auth.existingSecret }}
+                name: {{ .Values.redis.auth.existingSecret }}
+                key: {{ .Values.redis.auth.existingSecretPasswordKey | default "password" }}
+                {{- else }}
+                name: {{ .Release.Name }}-redis-secret
+                key: password
+                {{- end }}
+          {{- end }}
+          - name: REDIS_HOST
+            value: {{ .Release.Name }}-redis
+          - name: REDIS_PORT
+            value: "6379"
+          - name: REDIS_DB
+            value: "1"
+          - name: CELERY_DB
+            value: "0"
+          - name: DISCOVERY_CONFIG_PATH
+            value: /app/discovery/discovery-config.yaml
+          - name: LOG_LEVEL
+            value: {{ .Values.discovery.logLevel | default "INFO" }}
+          - name: CHAIN_OF_TASKS_EXPIRY_TIME
+            value: {{ .Values.scheduler.tasksExpiryTime | quote }}
+          - name: CELERY_TASK_TIMEOUT
+            value: {{ .Values.worker.taskTimeout | quote}}
+          volumeMounts:
+            - name: discovery-config
+              mountPath: "/app/discovery"
+              readOnly: true
+            - name: tmp
+              mountPath: "/tmp/"
+              readOnly: false
+
+      volumes:
+        # # You set volumes at the Pod level, then mount them into containers inside that Pod
+        - name: discovery-config
+          configMap:
+            name: {{ include "splunk-connect-for-snmp.name" . }}-discovery-config
+            items:
+              - key: "discovery-config.yaml"
+                path: "discovery-config.yaml"
+        - name: tmp
+          emptyDir: {}
+      restartPolicy: OnFailure
+{{- end -}}

charts/splunk-connect-for-snmp/templates/scheduler/deployment.yaml

@@ -44,6 +44,8 @@ spec:
               "celery", "beat",
             ]          
           env:
+            - name: USER
+              value: {{ .Values.deploymentUser | default "sc4snmp" | quote }}
             - name: CONFIG_PATH
               value: /app/config/config.yaml
             {{- include "splunk-connect-for-snmp.redis-env" . | nindent 12 }}

charts/splunk-connect-for-snmp/templates/sim/deployment.yaml

@@ -45,6 +45,8 @@ spec:
           imagePullPolicy: {{ .Values.sim.pullPolicy | default "IfNotPresent" }}
           args: ["--config=/config/otel-collector-config.yaml"]
           env:
+            - name: USER
+              value: {{ .Values.deploymentUser | default "sc4snmp" | quote }}
             {{- include "splunk-connect-for-snmp.redis-env" . | nindent 12 }}
             - name: signalfxToken
               valueFrom:

charts/splunk-connect-for-snmp/templates/tests/test-connection.yaml

@@ -27,5 +27,9 @@ spec:
           memory: 128Mi
         requests:
           cpu: 100m
-          memory: 128Mi            
+          memory: 128Mi
+      env:
+        - name: USER
+          value: {{ .Values.deploymentUser | default "sc4snmp" | quote }}          
+
   restartPolicy: Never

charts/splunk-connect-for-snmp/templates/traps/deployment.yaml

@@ -46,6 +46,8 @@ spec:
               "trap"
             ] 
           env:
+            - name: USER
+              value: {{ .Values.deploymentUser | default "sc4snmp" | quote }}
             - name: CONFIG_PATH
               value: /app/config/config.yaml
             - name: MONGO_URI

charts/splunk-connect-for-snmp/templates/worker/_helpers.tpl

@@ -59,6 +59,10 @@ app.kubernetes.io/name: {{ include "splunk-connect-for-snmp.worker.name" . }}-fl
 app.kubernetes.io/instance: {{ .Release.Name }}
 {{- end }}
 
+{{- define "splunk-connect-for-snmp.worker.discovery.selectorLabels" -}}
+app.kubernetes.io/name: {{ include "splunk-connect-for-snmp.worker.name" . }}-discovery
+app.kubernetes.io/instance: {{ .Release.Name }}
+{{- end }}
 
 {{/*
 Common labels
@@ -88,6 +92,11 @@ Common labels
 {{ include "splunk-connect-for-snmp.labels" . }}
 {{- end }}
 
+{{- define "splunk-connect-for-snmp.worker.discovery.labels" -}}
+{{ include "splunk-connect-for-snmp.worker.discovery.selectorLabels" . }}
+{{ include "splunk-connect-for-snmp.labels" . }}
+{{- end }}
+
 {{- define "environmental-variables" -}}
 - name: CONFIG_PATH
   value: /app/config/config.yaml
@@ -118,6 +127,8 @@ Common labels
   value: {{ .Values.worker.disableMongoDebugLogging | quote }}
 - name: UDP_CONNECTION_TIMEOUT
   value: {{ .Values.worker.udpConnectionTimeout | default "3" | quote }}
+- name: UDP_CONNECTION_RETRIES
+  value: {{ .Values.worker.udpConnectionRetries | default "5" | quote }}
 - name: MAX_OID_TO_PROCESS
   value: {{ .Values.poller.maxOidToProcess | default "70" | quote }}
 - name: MAX_REPETITIONS
@@ -210,4 +221,11 @@ Common labels
   {{ else }}
   value: "false"
   {{- end }}
+{{- end }}
+
+{{- define "environmental-variables-discovery" -}}
+- name: WORKER_CONCURRENCY
+  value: {{ .Values.worker.discovery.concurrency | default "4" | quote }}
+- name: PREFETCH_COUNT
+  value: {{ .Values.worker.discovery.prefetch | default "30" | quote }}
 {{- end }}