Skip to content

Updates to check-megaraid-sas-status.rb#7

Closed
jdito wants to merge 7 commits intosensu-plugins:masterfrom
jdito:master
Closed

Updates to check-megaraid-sas-status.rb#7
jdito wants to merge 7 commits intosensu-plugins:masterfrom
jdito:master

Conversation

@jdito
Copy link

@jdito jdito commented Aug 17, 2016

Pull Request Checklist

Is this in reference to an existing issue?
no

General

Purpose

Needed to be able to run with an unprivileged user in my environment. Flag defaults to false to prevent breaking existing configs. Please let me know if there's any other information needed.

@eheydrick
Copy link
Contributor

I think a better option would be to run the whole check under sudo. That would offer more flexibility on sudo options, the path to sudo itself, etc. Good catch on the typo, I fixed it in master.

@eheydrick eheydrick closed this Feb 2, 2017
eheydrick added a commit that referenced this pull request Feb 2, 2017
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