check for os update separated from main query,performed only once per…

… 12 hours

custom_components/truenas/coordinator.py

@@ -72,7 +72,7 @@ def __init__(self, hass: HomeAssistant, config_entry: ConfigEntry):
 
         self._systemstats_errored = []
         self.datasets_hass_device_id = None
-        self.last_hwinfo_update = datetime(1970, 1, 1)
+        self.last_updatecheck_update = datetime(1970, 1, 1)
 
         self._is_scale = False
         self._is_virtual = False
@@ -89,10 +89,6 @@ def connected(self) -> str:
     # ---------------------------
     async def _async_update_data(self):
         """Update TrueNAS data."""
-        # delta = datetime.now().replace(microsecond=0) - self.last_hwinfo_update
-        # if self.api.has_reconnected() or delta.total_seconds() > 60 * 60 * 4:
-        #     await self.hass.async_add_executor_job(self.get_access)
-
         await self.hass.async_add_executor_job(self.get_systeminfo)
         if self.api.connected():
             await self.hass.async_add_executor_job(self.get_systemstats)
@@ -117,6 +113,11 @@ async def _async_update_data(self):
         if self.api.connected():
             await self.hass.async_add_executor_job(self.get_app)
 
+        delta = datetime.now().replace(microsecond=0) - self.last_updatecheck_update
+        if self.api.connected() and delta.total_seconds() > 60 * 60 * 12:
+            await self.hass.async_add_executor_job(self.get_updatecheck)
+            self.last_updatecheck_update = datetime.now().replace(microsecond=0)
+
         if not self.api.connected():
             raise UpdateFailed("TrueNas Disconnected")
 
@@ -169,30 +170,6 @@ def get_systeminfo(self) -> None:
         if not self.api.connected():
             return
 
-        self.ds["system_info"] = parse_api(
-            data=self.ds["system_info"],
-            source=self.api.query("update/check_available", method="post"),
-            vals=[
-                {
-                    "name": "update_status",
-                    "source": "status",
-                    "default": "unknown",
-                },
-                {
-                    "name": "update_version",
-                    "source": "version",
-                    "default": "unknown",
-                },
-            ],
-        )
-
-        if not self.api.connected():
-            return
-
-        self.ds["system_info"]["update_available"] = (
-            self.ds["system_info"]["update_status"] == "AVAILABLE"
-        )
-
         if not self.ds["system_info"]["update_available"]:
             self.ds["system_info"]["update_version"] = self.ds["system_info"]["version"]
 
@@ -282,6 +259,34 @@ def get_systeminfo(self) -> None:
             ],
         )
 
+    # ---------------------------
+    #   get_updatecheck
+    # ---------------------------
+    def get_updatecheck(self) -> None:
+        self.ds["system_info"] = parse_api(
+            data=self.ds["system_info"],
+            source=self.api.query("update/check_available", method="post"),
+            vals=[
+                {
+                    "name": "update_status",
+                    "source": "status",
+                    "default": "unknown",
+                },
+                {
+                    "name": "update_version",
+                    "source": "version",
+                    "default": "unknown",
+                },
+            ],
+        )
+
+        if not self.api.connected():
+            return
+
+        self.ds["system_info"]["update_available"] = (
+            self.ds["system_info"]["update_status"] == "AVAILABLE"
+        )
+
     # ---------------------------
     #   get_systemstats
     # ---------------------------