Merge ~thogarre/charm-hw-health:remove-sudo-ipmi-logging into charm-hw-health:master

[Garrett Neugent]

Prevent logging sudo ipmi commands during checks

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack)]

A CI job is currently in progress. A follow up comment will be added when it completes.

[🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack)]

PASSED: Continuous integration, rev:4d051d5fd1382eac709ace0c884612211d6bd6e0
https://jenkins.canonical.com/bootstack/job/lp-charm-hw-health-ci/24/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/bootstack/job/lp-charm-test-unit/165/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/487/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-charm-hw-health-ci/24//rebuild

review: Approve (continuous-integration)

[Zachary Zehring (zzehring)]

LGTM +1

review: Approve

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 6ac3819b3e7f9522d75f790229cb61f38a95996e

diff --git a/src/files/ipmi/99-check_ipmi_sensor b/src/files/ipmi/99-check_ipmi_sensor
index ead55da..85e3886 100644
--- a/src/files/ipmi/99-check_ipmi_sensor
+++ b/src/files/ipmi/99-check_ipmi_sensor
@@ -1 +1,3 @@
-nagios ALL=(root) NOPASSWD: /usr/sbin/ipmimonitoring, /usr/sbin/ipmi-dcmi, /usr/sbin/ipmi-fru, /usr/sbin/ipmi-sensors, /usr/sbin/ipmi-sel
+Cmnd_Alias IPMI = /usr/sbin/ipmi-dcmi, /usr/sbin/ipmi-fru, /usr/sbin/ipmimonitoring, /usr/sbin/ipmi-sel, /usr/sbin/ipmi-sensors
+nagios ALL = (root) NOPASSWD: IPMI
+Defaults!IPMI !logfile, !syslog, !pam_session