[plugin] report DOWN port status as warning (#183)

This switches port status to WARNING from CRITICAL if their instances were shutdown.

Closes: #179

Signed-off-by: Ponnuvel Palaniyappan <[email protected]>

src/files/plugins/check_resources.py

@@ -308,6 +308,23 @@ def mechanism_warning_ids(connection, resource_type) -> Dict[str, str]:
         ]
         for ip in not_assigned_ips:
             warn_ids[ip] = "unassigned"
+
+    if resource_type == "port":
+        # Move DOWN ports from CRITICAL to warning if the instance was shutoff
+        all_ports = RESOURCES[resource_type](connection)
+        for port in all_ports:
+            try:
+                if port.status == "DOWN":
+                    server = connection.compute.get_server(port.device_id)
+                    if server.power_state == 4:  # 4 is SHUTOFF state
+                        warn_ids[port.id] = "SHUTOFF"
+            except openstack.exceptions.ResourceNotFound:
+                # device_id won't be available for internal ports, and we don't
+                # expect them to be shutdown deliberately either. Basically,
+                # for any reason, if power_state can't be determined, we ignore
+                # them, and they'll be reported as CRITICAL as before.
+                pass
+
     return warn_ids
 
 

src/tests/unit/test_check_resources.py

@@ -21,6 +21,7 @@ def __init__(self, type_, id_, status=None, **kwargs):
         self._type = type_
         self._id = id_
         self.status = status
+        self.device_id = "fake_device_id"
         for key, value in kwargs.items():
             setattr(self, key, value)