~jplettman/charm-hw-health:bug/1894763

Last commit made on 2020-11-20
Get this branch:
git clone -b bug/1894763 https://git.launchpad.net/~jplettman/charm-hw-health
Only John Lettman can upload to this branch. If you are John Lettman please log in for upload directions.

Branch merges

Branch information

Name:
bug/1894763
Repository:
lp:~jplettman/charm-hw-health

Recent commits

974271c... by Xav Paice

Provide more detail on yaml exception.

Exception in IPMI tool now has details provided.

Additionlly some lint cleanup.

4413444... by Xav Paice

Change method name, fix lint

get_exclusions is renamed to set_exclusions as requested in code review.
Fixed lint issues with the files changed for branch bug/1894763.

fb8fb18... by Adam Dyess

Created unit test to ensure exclusion creation is applied correctly

6ae94d1... by Adam Dyess

Progress towards fixing bug 1894763

882fb89... by Paul Goins

Lint changes. (Sorry; didn't get these in previously.)

3440d39... by Paul Goins

LP#1904032: Fix "lshw -json" parsing on Focal

In Bionic, "lshw -json" output one dictionary per item it was
displaying. In the case of multiple items (e.g. "-class network" with
multiple adapters), it would output multiple dictionaries in sequence.

In Focal, "lshw -json" outputs records in a list, which is more
correct for JSON parsers, however it breaks our current parsing.

The fix is simply to grab the first item in the list and act upon it.
Since the "lshw -json" query outputs the system record by default,
there will only be one item for our case.

278e94a... by Alvaro Uria

Merge branch 'bug/1904045'

Reviewed-on: https://code.launchpad.net/~canonical-is-bootstack/charm-hw-health/+git/charm-hw-health/+merge/393712
Reviewed-by: James Troup <email address hidden>

ee9ba76... by Alvaro Uria

LP#1904045: Fix ipmi cronjob runas permissions

When the ipmi cronjob is configured, the nagios user will run the croned
script. However, the configuration of the nrpe check reconfigured the
cronjob to be run as root. This fix makes the cronjob be always run by
the nagios user.

1ca277c... by Alvaro Uria

Merge branch 'aluria/bug/1893461'

Reviewed-on: https://code.launchpad.net/~canonical-is-bootstack/charm-hw-health/+git/charm-hw-health/+merge/393454
Reviewed-by: Paul Goins <email address hidden>
Reviewed-by: Alvaro Uria <email address hidden>

2e7f1c2... by Facundo Ciccioli

Add support for a whole family of controllers (MegaRAID SAS ones)

This is instead of listing every product that can be controlled by
megacli, encompass them by the driver.