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

Add nagios_send_cmd interface to give access to the cmd fifo #72

Open
wants to merge 1 commit into
base: rawhide
Choose a base branch
from

Conversation

mscherer
Copy link
Contributor

The cmd fifo is used by software such as nsca to trigger alert
in nagios.

The cmd fifo is used by software such as nsca to trigger alert
in nagios.
@wrabcak
Copy link
Member

wrabcak commented Feb 21, 2019

@mscherer,

Why you created new Label for it? I whink we can use nagios_spool_t for it.

@mscherer
Copy link
Contributor Author

mscherer commented Mar 6, 2019

It did seems cleaner to have a specific label for that access, rather than mixing that with "fiddling with the spool directory" access. But I see why we also do not want to have a explosion of labels, so if you prefer that we use the existing one, I guess that's fine for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants