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

remove extra character from dell iDrac-SMIv1 MIB #1141

Merged
merged 1 commit into from
May 5, 2024

Conversation

fbegyn
Copy link
Contributor

@fbegyn fbegyn commented Mar 21, 2024

The dell idrac mib is currently failing on parsing with the following error. This is because after the MIB-END indicitor, there are some broken characters interrupting the parsing.

Bad operator (: At line 13878 in ./mibs//iDRAC-SMIv1.mib

This can easily be solved by just removing the last line of this MIB.

The dell idrac mib is currently failing on parsing with the following
error. This is because after the `MIB-END` indicitor, there are some
broken characters interrupting the parsing.

```
Bad operator (: At line 13878 in ./mibs//iDRAC-SMIv1.mib
```

This can easily be solved by just removing the last line of this MIB.

Signed-off-by: Francis Begyn <francis@begyn.be>
Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I just noticed this with older versions of net-snmp.

@SuperQ SuperQ merged commit a55c2b6 into prometheus:main May 5, 2024
6 checks passed
@SuperQ SuperQ mentioned this pull request May 10, 2024
SuperQ pushed a commit that referenced this pull request May 11, 2024
* [CHANGE] Improve generator parse error handling #1167
* [ENHANCEMENT] generator: Add generator HELP override #1106
* [ENHANCEMENT] Refactoring of Scrape process, fixing multiple module issues #1111
* [ENHANCEMENT] Skip using an interactive terminal in "make docker-generate". #1113
* [ENHANCEMENT] Add SNMPInflight metric #1119
* [FEATURE] Support for passing username, password & priv_password as env vars #1074
* [FEATURE] Add GoSNMP logger #1157
* [FEATURE] Add a "snmp_context" parameter to the URL #1163
* [BUGFIX] generator: curl failed #1094
* [BUGFIX] Fix SNMPv3 password configuration #1122
* [BUGFIX] generator: Update generator User-Agent #1133
* [BUGFIX] generator: fix mibs directory specification for parse_errors command #1135
* [BUGFIX] generator: remove extra character from dell iDrac-SMIv1 MIB #1141
* [BUGFIX] Fix do not expand envvars for empty config fields #1148

snmp.yml changes:
* Updated Cisco MIBs #1180
* Updated Cyberpower MIBs #1124
* Updated servertech_sentry3 #1090
* Added support for Dell iDrac  #1125
---------

Signed-off-by: Sebastian Schubert <basti@schubert.digital>
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.

None yet

2 participants