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

[dev.icinga.com #12992] Add check_md_raid to ITL #4751

Closed
icinga-migration opened this issue Oct 27, 2016 · 8 comments
Closed

[dev.icinga.com #12992] Add check_md_raid to ITL #4751

icinga-migration opened this issue Oct 27, 2016 · 8 comments
Assignees
Labels
area/itl Template Library CheckCommands enhancement New feature or request

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/12992

Created by twidhalm on 2016-10-27 08:24:29 +00:00

Assignee: (none)
Status: New
Target Version: (none)
Last Update: 2016-12-08 00:27:56 +00:00 (in Redmine)

Backport?: Not yet backported
Include in Changelog: 1

Hi,

This patch adds check_md_raid to the ITL. This is a very minimalistic plugin and does not need any configuration. The main reason I want this in the ITL is that there is no mdadm check plugin so far and the documentation could use a reference to a working one. If you have a better alternative, I'll be glad to change the request to incorporate another mdadm check plugin.

Cheers,
Thomsa

Attachments

@icinga-migration
Copy link
Author

Updated by twidhalm on 2016-10-27 08:35:42 +00:00

  • File added 0001-Add-check_md_raid-to-ITL.patch

Sorry, I realized too late that I had the wrong plugindir in the command.

@icinga-migration
Copy link
Author

Updated by nlm on 2016-11-10 10:34:58 +00:00

Maybe adding check_raid instead would be a more standard option ?
It's included in Debian/Ubuntu nagios-plugins-contrib package, and handles md raid.
Didn't check for other distributions, though. What do you think ?

# grep 'Linux software RAID' /usr/lib/nagios/plugins/check_raid
# - Linux software RAID (md) via /proc/mdstat
# lsb_release -d
Description:    Ubuntu 16.04 LTS
# dpkg -L nagios-plugins-contrib | grep raid
/usr/lib/nagios/plugins/check_raid

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2016-12-08 00:27:56 +00:00

Problem is that this Debian specific package isn't available anywhere else so we have to stick with one plugin which a) works very well everywhere b) has proper documentation/url for us.

@icinga-migration icinga-migration added enhancement New feature or request area/itl Template Library CheckCommands labels Jan 17, 2017
@dnsmichi
Copy link
Contributor

@widhalmt can you please update the patch against current master and send a PR?

@widhalmt
Copy link
Member

Will do as soon as I find the time. Hopefully this or next week. Sorry for the delay.

@dnsmichi
Copy link
Contributor

Ping :)

@gunnarbeutner
Copy link
Contributor

Pong.

@dnsmichi
Copy link
Contributor

Closing as no updated patch was received.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/itl Template Library CheckCommands enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants