[NDINT-387] Get health score from healthInst if fvOverallHealth isn't present (DataDog#21988)

* [NDINT-387] Get health score from healthInst if fvOverallHealth isn't present.

* Changelog entry.

Author: dplepage-dd

cisco_aci/changelog.d/21988.added

@@ -0,0 +1 @@
+Get health score from healthInst if fvOverallHealth isn't enabled.

cisco_aci/datadog_checks/cisco_aci/aci_metrics.py

@@ -63,6 +63,7 @@
 
 def make_tenant_metrics():
     metrics = {
+        "healthInst": {"cur": "{}.health"},
         "fvOverallHealth": {"healthAvg": "{}.overall_health", "healthLast": "{}.health"},
         "fvFltCounter": {"warncountAvg": "{}.fault_counter"},
     }

cisco_aci/datadog_checks/cisco_aci/api.py

@@ -219,7 +219,7 @@ def get_eth_list_for_epg(self, tenant, app, epg):
         return self._parse_response(response)
 
     def get_tenant_stats(self, tenant):
-        path = "/api/mo/uni/tn-{}.json?rsp-subtree-include=stats,no-scoped".format(tenant)
+        path = "/api/mo/uni/tn-{}.json?rsp-subtree-include=stats,health,no-scoped".format(tenant)
         response = self.make_request(path)
         # return only the list of stats
         return self._parse_response(response)

cisco_aci/datadog_checks/cisco_aci/tenant.py

@@ -85,6 +85,7 @@ def _submit_ten_data(self, tenant):
             pass
 
     def submit_raw_obj(self, raw_stats, tags, obj_type):
+        got_health = False
         for s in raw_stats:
             name = list(s.keys())[0]
             # we only want to collect the 15 minutes metrics.
@@ -110,5 +111,25 @@ def submit_raw_obj(self, raw_stats, tags, obj_type):
                     json_attrs = s.get(name, {}).get("attributes", {})
                     if mval and helpers.check_metric_can_be_zero(cisco_metric, mval, json_attrs):
                         metrics[dd_metric] = mval
-
+            if 'fvOverallHealth' in name:
+                got_health = True
             self.submit_metrics(metrics, tags, instance=self.instance)
+
+        if got_health:
+            return
+        self.log.debug("No fvOverallHealth reported, looking for healthInst instead")
+        health_insts = [s for s in raw_stats if list(s.keys())[0] == "healthInst"]
+        if not health_insts:
+            self.log.debug("No healthInst reported")
+            return
+        s = health_insts[0]
+        self.log.debug("submitting metrics for: %s", 'healthInst')
+        metrics = {}
+
+        ms = self.tenant_metrics.get(obj_type, {}).get('healthInst', {})
+        for cisco_metric, dd_metric in ms.items():
+            mval = s.get('healthInst', {}).get("attributes", {}).get(cisco_metric)
+            json_attrs = s.get('healthInst', {}).get("attributes", {})
+            if mval and helpers.check_metric_can_be_zero(cisco_metric, mval, json_attrs):
+                metrics[dd_metric] = mval
+        self.submit_metrics(metrics, tags, instance=self.instance)

cisco_aci/tests/common.py

@@ -172,8 +172,10 @@
     # 643d217904f09445fbc9f7b43cd131f0 - Api.get_fabric_pods
     '_api_mo_uni_tn_DataDog_json_query_target_subtree_target_subtree_class_fvAp',
     # 4efe80304d50330f5ed0f79252ef0a84 - Api.get_apps
-    '_api_mo_uni_tn_DataDog_json_rsp_subtree_include_stats_no_scoped',
-    # c8e9a0dbceac67fb1149684f7fc7772c - Api.get_tenant_stats
+    '_api_mo_uni_tn_DataDog_json_rsp_subtree_include_stats_health_no_scoped',
+    # 07f9ef7474c39eef33e9ddfd269d54fb - Api.get_tenant_stats
+    '_api_mo_uni_tn_DataDogAlt_json_rsp_subtree_include_stats_health_no_scoped',
+    # cd84d80314a78b27c72f287322f30b68 - Api.get_tenant_stats
     '_api_node_class_lldpAdjEp_json',
     # f3713df3a586908a3a11f4c356153519 - Api.get_lldp_adj_eps
     '_api_node_class_cdpAdjEp_json',
@@ -228,6 +230,9 @@ def make_request(self, path):
         mock_path = mock_path.replace('[', '_')
         mock_path = mock_path.replace(']', '_')
         mock_path = mock_path.replace('|', '_')
+        if mock_path not in FIXTURE_LIST_FILE_MAP:
+            log.debug("Skipping %s - no fixture file found", path)
+            return {"imdata": []}
         mock_path = FIXTURE_LIST_FILE_MAP[mock_path]
         for p in self.fixture_dirs:
             path = os.path.join(p, mock_path)