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

Switching from declaring the class to including the class fixes an is… #174

Open
wants to merge 96 commits into
base: master
Choose a base branch
from

Conversation

hundredacres
Copy link

Switching from declaring the class to including the class fixes an issue where both nagios_pci_megaraid_sas is true and the host has nvme drives present

…sue where both nagios_pci_megaraid_sas is true and the host has nvme drives present
Matthew Schmitt and others added 28 commits April 7, 2020 11:37
purge nagios service dependencies also
require contactgroups listed in service
support sudo for service checks
support notes_url paramenter
Robert Brooks and others added 30 commits September 20, 2021 10:52
remove nagios_alias from here
add foo command as it is used in place of a real check
use a directory per resource type
…38fbd80659a37b22d3021a2b628dfd4c1c72

Picked current commit and this PR
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.

2 participants