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.

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

fix: snmp marshal error #10322

Merged
merged 2 commits into from
Dec 22, 2021
Merged

fix: snmp marshal error #10322

merged 2 commits into from
Dec 22, 2021

Conversation

MyaLongmire
Copy link
Contributor

@MyaLongmire MyaLongmire commented Dec 21, 2021

Required for all PRs:

resolves #10305, resolves #10304

Thank you @Doridian for the debugging and the solution!!
Store modules in an array of strings and then load them in as this ensures modules are loaded in the correct order. Also added better error messaging instead of the marshal: marshalPDU error.

internal/snmp/translate.go Outdated Show resolved Hide resolved
@telegraf-tiger
Copy link
Contributor

@MyaLongmire MyaLongmire merged commit 8a9fbfd into master Dec 22, 2021
@MyaLongmire MyaLongmire deleted the snmp-marshal-error branch December 22, 2021 20:41
powersj pushed a commit that referenced this pull request Jan 5, 2022
(cherry picked from commit 8a9fbfd)
@Derek-K
Copy link

Derek-K commented Jan 11, 2022

Just updated to 1.21.2-1, my APC UPS readings are back (except model and device name, but it could be my bad when I was trying to debug the issues earlier) Thanks!

nward added a commit to nward/telegraf that referenced this pull request Jan 12, 2022
powersj pushed a commit to powersj/telegraf that referenced this pull request Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/snmp fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Marshal error with SNMP plugin 1.21.0 Telegraf 1.21.1: Marshal error with SNMP PowerNet MIB
4 participants