hw-health-charm

~dmzoneill/charm-hw-health:stable/20.08

  1. Git
  2. lp:~dmzoneill/charm-hw-health
  3. stable/20.08
Last commit made on 2020-08-13
Get this branch:
git clone -b stable/20.08 https://git.launchpad.net/~dmzoneill/charm-hw-health
Only David O Neill can upload to this branch. If you are David O Neill please log in for upload directions.

Branch merges

Branch information

Name:
stable/20.08
Repository:
lp:~dmzoneill/charm-hw-health

Recent commits

27fac68... by Xav Paice

Merge branch 'remove-check-ilorest.py'

cc1f497... by Drew Freiberger

Remove extraneous debug print from test_cron_ilorest.py unit test.

fbeb27e... by Drew Freiberger

Extra Linting completed for 20.08 charm release

55756c1... by Drew Freiberger

Check issubclass() instead of isinstance() in cls.is_series_supported()

9e38254... by Xav Paice

Updated ilorest json parsing code to find bad status in more places.

Add unit tests for cron_ilorest to ensure detection of errors

5bab06c... by Paul Goins

Fixed PATH-related issue which broke cron_ilorest.py

Since /usr/sbin/ is not on the PATH via cron, call ilorest as
/usr/sbin/ilorest.

(Thanks Drew for the idea!)

f7d0294... by Paul Goins

Removing now-unused check file to avoid confusion

bd8b1c2... by Xav Paice

Change the ilorest output file name

To make the output of cron_ilorest.py consistent with other scripts, and
compatible with the nrpe check which templates the filename as
toolname.out, this changes the ilorest output name from ilorest.nagios
to ilorest.out.

2602e4f... by Paul Goins

Merge branch 'wait-for-vendor-apt-fix' into merge-test

f948c8a... by Paul Goins

Fixed a bug in wait_for_vendor_apt().

There's a clause which tries to do an isinstance check, but does not
instantiate an object from its original class.

This triggered on tools.ILOrest because, while it is a subclass of
tools.AptVendorTool, it is not an instance of it.

>>> isinstance(tools.ILOrest, tools.AptVendorTool)
False

This is fixed by instantiating an ILOrest object and using that instead.