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

Adding check if Counter is actually registered before trying to add value #8

Merged
merged 1 commit into from
Jun 17, 2018
Merged

Adding check if Counter is actually registered before trying to add value #8

merged 1 commit into from
Jun 17, 2018

Conversation

kalebris
Copy link
Contributor

@kalebris kalebris commented Jan 2, 2018

Adding a check for badly written plugins where config returns less values then fetch as munin exporter crashes in that case.

@kalebris kalebris mentioned this pull request Jan 3, 2018
Open
@zstyblik
Copy link

This actually seems to help. @pvdh, are there any plans to merge this?

@pvdh pvdh merged commit bcfbd1d into pvdh:master Jun 17, 2018
@pvdh
Copy link
Owner

pvdh commented Jun 17, 2018

Thank you for your contribution, highly appreciated.
I'm not longer using munin_exporter myself and didn't pay attention, sorry about that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kalebris @zstyblik @pvdh