Merge ~peter-sabaini/charm-hw-health:sas2ircu-allow-optimal-state into ~nagios-charmers/charm-hw-health:master

[Peter Sabaini]

Fix: sas2ircu "Optimal" state should be ok too

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

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

[Aurelien Lourot (aurelien-lourot)]

+1, looks good to me

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

Change cannot be self approved, setting status to needs review.

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

Change successfully merged at revision 4ec4549fd5cb205b9d31a81ca55d49203ac8de25

diff --git a/src/files/sas2ircu/check_sas2ircu.py b/src/files/sas2ircu/check_sas2ircu.py
index b22c380..7a4c861 100755
--- a/src/files/sas2ircu/check_sas2ircu.py
+++ b/src/files/sas2ircu/check_sas2ircu.py
@@ -34,7 +34,7 @@ def parse_output():
                 tmpdev = devices.get(device[2], [])
                 tmpdev.append('{}:{}'.format(device[0], device[1]))
                 devices[device[2]] = tmpdev
-                if 'Ready' not in device:
+                if not ('Ready' in device or 'Optimal' in device):
                     critical = True
                 device = []