refactoring

kvmd/apps/kvmd/api/export.py

@@ -0,0 +1,72 @@
+# ========================================================================== #
+#                                                                            #
+#    KVMD - The main Pi-KVM daemon.                                          #
+#                                                                            #
+#    Copyright (C) 2018  Maxim Devaev <mdevaev@gmail.com>                    #
+#                                                                            #
+#    This program is free software: you can redistribute it and/or modify    #
+#    it under the terms of the GNU General Public License as published by    #
+#    the Free Software Foundation, either version 3 of the License, or       #
+#    (at your option) any later version.                                     #
+#                                                                            #
+#    This program is distributed in the hope that it will be useful,         #
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of          #
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the           #
+#    GNU General Public License for more details.                            #
+#                                                                            #
+#    You should have received a copy of the GNU General Public License       #
+#    along with this program.  If not, see <https://www.gnu.org/licenses/>.  #
+#                                                                            #
+# ========================================================================== #
+
+
+import asyncio
+import operator
+
+from typing import Any
+from typing import List
+
+from aiohttp.web import Request
+from aiohttp.web import Response
+
+from ....plugins.atx import BaseAtx
+
+from ..info import InfoManager
+
+from ..http import exposed_http
+
+
+# =====
+class ExportApi:
+    def __init__(self, info_manager: InfoManager, atx: BaseAtx) -> None:
+        self.__info_manager = info_manager
+        self.__atx = atx
+
+    # =====
+
+    @exposed_http("GET", "/export/prometheus/metrics")
+    async def __prometheus_metrics_handler(self, _: Request) -> Response:
+        (atx_state, hw_state) = await asyncio.gather(*[
+            self.__atx.get_state(),
+            self.__info_manager.get_submanager("hw").get_state(),
+        ])
+        rows: List[str] = []
+        self.__append_prometheus_rows(rows, atx_state["enabled"], "pikvm_atx_enabled")
+        self.__append_prometheus_rows(rows, atx_state["leds"]["power"], "pikvm_atx_power")
+        if hw_state is not None:
+            self.__append_prometheus_rows(rows, hw_state["health"], "pikvm_hw")
+        return Response(text="\n".join(rows))
+
+    def __append_prometheus_rows(self, rows: List[str], value: Any, path: str) -> None:
+        if isinstance(value, bool):
+            value = int(value)
+        if isinstance(value, (int, float)):
+            rows.extend([
+                f"# TYPE {path} gauge",
+                f"{path} {value}",
+                "",
+            ])
+        elif isinstance(value, dict):
+            for (sub_key, sub_value) in sorted(value.items(), key=operator.itemgetter(0)):
+                sub_path = (f"{path}_{sub_key}" if sub_key != "parsed_flags" else path)
+                self.__append_prometheus_rows(rows, sub_value, sub_path)

kvmd/apps/kvmd/api/info.py

@@ -22,8 +22,6 @@
 
 import asyncio
 
-from typing import Any
-from typing import Dict
 from typing import List
 
 from aiohttp.web import Request
@@ -36,22 +34,6 @@ from ..info import InfoManager
 
 from ..http import exposed_http
 from ..http import make_json_response
-from ..http import make_text_response
-
-
-# ====
-def _build_metrics(metrics: List[str], name: str, value: Any) -> None:
-    if isinstance(value, bool):
-        value = 1 if value else 0
-    if isinstance(value, (int, float)):
-        metrics.append(f"# TYPE {name} gauge")
-        metrics.append(f"{name} {value}")
-    elif isinstance(value, dict):
-        for key, val in value.items():
-            if key == "parsed_flags":
-                _build_metrics(metrics, name, val)
-            else:
-                _build_metrics(metrics, f"{name}_{key}", val)
 
 
 # =====
@@ -77,17 +59,3 @@ class InfoApi:
             subval=(lambda field: check_string_in_list(field, "info field", subs)),
             name="info fields list",
         ))) or subs)
-
-    @exposed_http("GET", "/export/prometheus/metrics", False)
-    async def __metrics_handler(self, _: Request) -> Response:
-        data = await asyncio.gather(self.__info_manager.get_submanager("hw").get_state())
-        if data is None:
-            return make_text_response("error", 500)
-        else:
-            data_exists: Dict[Any, Any] = data
-            health = data_exists[0]["health"]
-
-            metrics: List[str] = []
-            _build_metrics(metrics, "pikvm", health)
-
-            return make_text_response("\n".join(metrics))