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.

Sign up for GitHub

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

Jump to bottom

fix for panic with snmp exporter from the recent version upgrade #6921

Merged
merged 2 commits into from
May 21, 2024
Merged

fix for panic with snmp exporter from the recent version upgrade #6921

merged 2 commits into from
May 21, 2024

Conversation

erikbaranowski
Copy link
Contributor

@erikbaranowski erikbaranowski commented May 21, 2024 •
edited
Loading

PR Description

Which issue(s) this PR fixes

Fixes #6918

Notes to the Reviewer

This was validated as fixed by reproducing the error on main and then validating the fix on this branch with the following config (thanks @ptodev):

server:
  log_level: debug
integrations:
  snmp:
    enabled: true
    scrape_interval: 1m
    scrape_timeout: 30s
    snmp_targets:
      - name: testing
        address: demo.pysnmp.com
        module: if_mib
        auth: public_v2

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@erikbaranowski erikbaranowski enabled auto-merge (squash) May 21, 2024 19:35
@erikbaranowski erikbaranowski merged commit eb01156 into main May 21, 2024
10 checks passed
@erikbaranowski erikbaranowski deleted the snmp-exporter-bugfix branch May 21, 2024 19:37
@ptodev ptodev mentioned this pull request May 31, 2024
Closed
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Jun 21, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ptodev ptodev ptodev approved these changes

Assignees
No one assigned
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

snmp_exporter version update not working due to new snmp_exporter metric
2 participants
@erikbaranowski @ptodev