-
Notifications
You must be signed in to change notification settings - Fork 3.8k
prometheus-node-exporter-lua: check if status is nil before processing wifi device metrics interfaces #27564
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
base: master
Are you sure you want to change the base?
Conversation
|
@oowl properly name your commit subject after squashing the two commits. |
|
@systemcrash Done, Sir. PTAL |
|
@oowl I've edited the PR title, please edit your commit title to start with |
|
@champtar done |
|
I thought you already did it but please fix your commit SOB. |
|
@systemcrash Sorry, I did not understand your requirement. I have been squashing my commits into one commit and also changed my PR title with proper words.
SOB seems offensive words? mean Son of Bitch? |
…g wifi device metrics interfaces
Add a check for status existence before processing.
In my OpenWRT device, it did not have a Wi-Fi device, so if I use

prometheus-node-exporter-lua, I found below error in my syslog.📦 Package Details
Maintainer: @
(You can find this by checking the history of the package
Makefile.)Description:
🧪 Run Testing Details
✅ Formalities
If your PR contains a patch:
git am(e.g., subject line, commit description, etc.)
We must try to upstream patches to reduce maintenance burden.