pyasic-umhost 0.3.0.dev5__tar.gz → 0.3.0.dev6__tar.gz

{pyasic_umhost-0.3.0.dev5 → pyasic_umhost-0.3.0.dev6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyasic-umhost
-Version: 0.3.0.dev5
+Version: 0.3.0.dev6
 Summary: Fork of pyasic — a simplified and standardized interface for Bitcoin ASICs.
 License: Apache 2.0
 License-File: LICENSE.txt
@@ -165,6 +165,30 @@ average = sum(data, start=MinerData("0.0.0.0")) / len(data)
 `get_hostname`, `get_model`, `reboot`, `restart_backend`, `stop_mining`, `resume_mining`,
 `is_mining`, `send_config`, `set_power_limit`.
 
+Для stock Antminer `stop_mining()` и `resume_mining()` проверяют результат
+повторным чтением `bitmain-work-mode`: ответ CGI `M000` сам по себе не
+подтверждает смену режима. Сначала используется поле записи `miner-mode`.
+Если прошивка его игнорирует или возвращает структурированный CGI-отказ,
+а чтение подтверждает прежний режим, библиотека пробует `bitmain-work-mode`.
+Поддерживаются числовое и строковое значения обоих полей; подтвержденный
+формат запоминается в экземпляре майнера. Наличие
+`bitmain-user-ip-cat` не определяет формат: это поле есть и у T21 с прошивкой
+26.01.2025, которая принимает только `miner-mode`.
+
+Пулы и остальные параметры сохраняются из текущей конфигурации. После
+потерянного HTTP-ответа (встречается на KS5 при перезапуске сервиса) режим
+также проверяется чтением: подтвержденная цель означает успех без повторной
+записи. При неизвестном или неподтвержденном режиме методы выбрасывают
+`APIError`, чтобы агент мог записать причину и повторить задачу. Ошибка HTTP
+сама по себе не разрешает подбор другого формата. Выход из сна подтверждает
+режим работы; запуск плат и восстановление хешрейта могут занять дополнительное время.
+
+Эта совместимость относится к stock-прошивкам с JSON CGI (`AntminerModern`).
+Она не добавляет сон прошивкам, в которых такой функции нет, и не заменяет
+отдельные протоколы старого form CGI, VNish, Braiins OS или LuxOS.
+Новая схема `bitmain-work-mode` и необходимость передавать текущие пулы
+описаны в [официальной документации Bitmain](https://docs.bitmain.com/en/antminer/CONTENTS.html#modify-miner-configuration).
+
 ```python
 import asyncio
 from pyasic import get_miner

{pyasic_umhost-0.3.0.dev5 → pyasic_umhost-0.3.0.dev6}/README.md

@@ -131,6 +131,30 @@ average = sum(data, start=MinerData("0.0.0.0")) / len(data)
 `get_hostname`, `get_model`, `reboot`, `restart_backend`, `stop_mining`, `resume_mining`,
 `is_mining`, `send_config`, `set_power_limit`.
 
+Для stock Antminer `stop_mining()` и `resume_mining()` проверяют результат
+повторным чтением `bitmain-work-mode`: ответ CGI `M000` сам по себе не
+подтверждает смену режима. Сначала используется поле записи `miner-mode`.
+Если прошивка его игнорирует или возвращает структурированный CGI-отказ,
+а чтение подтверждает прежний режим, библиотека пробует `bitmain-work-mode`.
+Поддерживаются числовое и строковое значения обоих полей; подтвержденный
+формат запоминается в экземпляре майнера. Наличие
+`bitmain-user-ip-cat` не определяет формат: это поле есть и у T21 с прошивкой
+26.01.2025, которая принимает только `miner-mode`.
+
+Пулы и остальные параметры сохраняются из текущей конфигурации. После
+потерянного HTTP-ответа (встречается на KS5 при перезапуске сервиса) режим
+также проверяется чтением: подтвержденная цель означает успех без повторной
+записи. При неизвестном или неподтвержденном режиме методы выбрасывают
+`APIError`, чтобы агент мог записать причину и повторить задачу. Ошибка HTTP
+сама по себе не разрешает подбор другого формата. Выход из сна подтверждает
+режим работы; запуск плат и восстановление хешрейта могут занять дополнительное время.
+
+Эта совместимость относится к stock-прошивкам с JSON CGI (`AntminerModern`).
+Она не добавляет сон прошивкам, в которых такой функции нет, и не заменяет
+отдельные протоколы старого form CGI, VNish, Braiins OS или LuxOS.
+Новая схема `bitmain-work-mode` и необходимость передавать текущие пулы
+описаны в [официальной документации Bitmain](https://docs.bitmain.com/en/antminer/CONTENTS.html#modify-miner-configuration).
+
 ```python
 import asyncio
 from pyasic import get_miner

{pyasic_umhost-0.3.0.dev5 → pyasic_umhost-0.3.0.dev6}/pyasic/miners/backends/antminer.py

@@ -14,10 +14,12 @@
 #  limitations under the License.                                              -
 # ------------------------------------------------------------------------------
 
+import asyncio
 import logging
 from pathlib import Path
 
-from pyasic.config import MinerConfig, MiningModeConfig, MiningModeSleep, PoolConfig
+from pyasic import settings
+from pyasic.config import MinerConfig, PoolConfig
 from pyasic.data import Fan, HashBoard
 from pyasic.data.error_codes import X19Error
 from pyasic.data.pools import PoolMetrics, PoolUrl
@@ -36,7 +38,7 @@ from pyasic.rpc.antminer import AntminerRPCAPI
 from pyasic.ssh.antminer import AntminerModernSSH
 from pyasic.web.antminer import AntminerModernWebAPI, AntminerOldWebAPI
 
-ANTMINER_NEW_FIRMWARE_CONFIG_MARKER = "bitmain-user-ip-cat"
+_POWER_MODE_VERIFY_DELAYS = (1, 2, 5)
 
 
 def _pick_stats_payload(
@@ -290,69 +292,133 @@ class AntminerModern(BMMiner):
         return False
 
     @staticmethod
-    def _uses_new_firmware_power_mode(config: dict) -> bool:
-        """Определяет новый CGI-формат конфигурации без привязки к модели."""
-        return ANTMINER_NEW_FIRMWARE_CONFIG_MARKER in config
-
-    async def _set_new_firmware_power_mode(
-        self, config: dict, work_mode: int
-    ) -> bool:
-        """Отправляет полный конфиг новой прошивке, меняя только work mode."""
-        try:
-            current_mode = int(config["bitmain-work-mode"])
-        except (KeyError, TypeError, ValueError) as error:
+    def _read_power_mode(config: dict) -> int:
+        value = config.get("bitmain-work-mode") if isinstance(config, dict) else None
+        if (
+            isinstance(value, bool)
+            or not isinstance(value, (str, int))
+            or not str(value).isdigit()
+        ):
             raise APIError(
-                "Invalid bitmain-work-mode in Antminer configuration: "
-                f"{config.get('bitmain-work-mode')!r}"
-            ) from error
+                f"Invalid bitmain-work-mode in Antminer configuration: {value!r}"
+            )
+        mode = int(value)
+        if mode not in (0, 1, 2, 3, 5):
+            raise APIError(
+                f"Unknown bitmain-work-mode in Antminer configuration: {value!r}"
+            )
+        return mode
 
-        if current_mode == work_mode:
+    async def _set_power_mode(self, work_mode: int) -> bool:
+        """Меняет только режим, подтверждая формат CGI чтением конфигурации.
+
+        bitmain-user-ip-cat встречается и у прошивок с miner-mode, поэтому
+        не определяет формат записи. Даже M000 может означать, что неизвестное
+        поле молча проигнорировано. Другой формат пробуем только после чтения
+        прежнего режима; при недоступном API повторную запись не делаем.
+        """
+        config = await self.web.get_miner_conf()
+        current_mode = self._read_power_mode(config)
+        if current_mode == work_mode or (work_mode == 0 and current_mode != 1):
             return True
+        initial_mode = current_mode
+
+        legacy_as_str = settings.get("antminer_mining_mode_as_str", False)
+        candidates = [
+            ("miner-mode", legacy_as_str),
+            ("bitmain-work-mode", True),
+            ("miner-mode", not legacy_as_str),
+            ("bitmain-work-mode", False),
+        ]
+        preferred = getattr(self, "_power_mode_format", None)
+        if preferred in candidates:
+            candidates.remove(preferred)
+            candidates.insert(0, preferred)
+        for index, (field, as_str) in enumerate(candidates):
+            # Сохраняем пулы, лимит хешрейта и специфичные поля прошивки:
+            # преобразование через MinerConfig теряет неизвестные ему опции.
+            payload = {key: value for key, value in config.items() if value is not None}
+            payload.pop("miner-mode", None)
+            payload.pop("bitmain-work-mode", None)
+            payload[field] = str(work_mode) if as_str else work_mode
+            if field == "miner-mode":
+                if "bitmain-freq-level" in payload:
+                    payload["freq-level"] = payload.pop("bitmain-freq-level")
+            response = await self.web.set_miner_conf(payload)
+            response_stats = (
+                str(response.get("stats", "")).lower()
+                if isinstance(response, dict)
+                else ""
+            )
+            response_code = (
+                str(response.get("code", "")) if isinstance(response, dict) else ""
+            )
+            accepted = response_stats == "success" and response_code == "M000"
+            # Некоторые CGI явно отвергают незнакомое поле. Только такой
+            # структурированный отказ допускает проверку альтернативы;
+            # HTTP/auth/сетевой сбой не является признаком формата прошивки.
+            rejected = response_stats in ("error", "fail") and response_code.startswith(
+                "M"
+            )
+            # Ответ POST может потеряться при перезапуске сервиса (KS5).
+            # Проверяем фактический режим даже после HTTP timeout, но такой
+            # сбой сам по себе не разрешает повторную запись другим форматом.
+            details = (
+                {key: response.get(key) for key in ("stats", "code", "msg", "message")}
+                if isinstance(response, dict)
+                else type(response).__name__
+            )
 
-        payload = dict(config)
-        payload["bitmain-work-mode"] = work_mode
-        response = await self.web.set_miner_conf(payload)
-        if not isinstance(response, dict) or not (
-            str(response.get("stats", "")).lower() == "success"
-            and response.get("code") == "M000"
-        ):
-            raise APIError(
-                f"Antminer rejected bitmain-work-mode={work_mode}: {response!r}"
+            last_error = None
+            for delay in _POWER_MODE_VERIFY_DELAYS:
+                await asyncio.sleep(delay)
+                try:
+                    observed = await self.web.get_miner_conf()
+                    current_mode = self._read_power_mode(observed)
+                except APIError as exc:
+                    last_error = exc
+                    continue
+                config = observed
+                last_error = None
+                if current_mode == work_mode:
+                    self._power_mode_format = (field, as_str)
+                    return True
+            if last_error is not None:
+                raise APIError(
+                    f"Antminer {self.ip}: cannot verify {field}={work_mode}: {last_error}; "
+                    f"command response={details}"
+                ) from last_error
+            if not accepted and not rejected:
+                raise APIError(
+                    f"Antminer {self.ip} rejected {field}={work_mode}: {details}; "
+                    f"observed={current_mode}"
+                )
+            if current_mode != initial_mode:
+                raise APIError(
+                    f"Antminer {self.ip}: unexpected power mode change "
+                    f"(initial={initial_mode}, expected={work_mode}, observed={current_mode})"
+                )
+            logging.warning(
+                "%s: %s=%r returned %s/%s but readback bitmain-work-mode=%s%s",
+                self.ip,
+                field,
+                payload[field],
+                response_stats,
+                response_code,
+                current_mode,
+                "; trying alternate CGI format" if index < len(candidates) - 1 else "",
             )
-        return True
+
+        raise APIError(
+            f"Antminer {self.ip}: power mode not applied after both CGI formats "
+            f"(expected={work_mode}, observed={current_mode})"
+        )
 
     async def stop_mining(self) -> bool:
-        raw_config = await self.web.get_miner_conf()
-        if not raw_config:
-            raise APIError("No data was returned from the API")
-        if self._uses_new_firmware_power_mode(raw_config):
-            return await self._set_new_firmware_power_mode(raw_config, work_mode=1)
-
-        # send_config -> set_miner_conf.cgi на стоковой прошивке Antminer
-        # применяется через системный reboot. Не пушим конфиг, если режим
-        # уже sleep — иначе получим лишнюю перезагрузку устройства.
-        self.config = MinerConfig.from_am_modern(raw_config)
-        cfg = self.config
-        if isinstance(cfg.mining_mode, MiningModeSleep):
-            return True
-        cfg.mining_mode = MiningModeConfig.sleep()
-        await self.send_config(cfg)
-        return True
+        return await self._set_power_mode(work_mode=1)
 
     async def resume_mining(self) -> bool:
-        raw_config = await self.web.get_miner_conf()
-        if not raw_config:
-            raise APIError("No data was returned from the API")
-        if self._uses_new_firmware_power_mode(raw_config):
-            return await self._set_new_firmware_power_mode(raw_config, work_mode=0)
-
-        self.config = MinerConfig.from_am_modern(raw_config)
-        cfg = self.config
-        if not isinstance(cfg.mining_mode, MiningModeSleep):
-            return True
-        cfg.mining_mode = MiningModeConfig.normal()
-        await self.send_config(cfg)
-        return True
+        return await self._set_power_mode(work_mode=0)
 
     async def _get_hostname(
         self, web_get_system_info: dict | None = None
@@ -632,18 +698,11 @@ class AntminerModern(BMMiner):
             try:
                 web_get_conf = await self.web.get_miner_conf()
             except APIError:
-                pass
-
-        if web_get_conf is not None:
-            try:
-                if str(web_get_conf["bitmain-work-mode"]).isdigit():
-                    return (
-                        False if int(web_get_conf["bitmain-work-mode"]) == 1 else True
-                    )
-                return False
-            except LookupError:
-                pass
-        return None
+                return None
+        try:
+            return self._read_power_mode(web_get_conf) != 1
+        except APIError:
+            return None
 
     async def _get_uptime(
         self,
@@ -773,17 +832,11 @@ class AntminerModern(BMMiner):
             try:
                 web_get_conf = await self.web.get_miner_conf()
             except APIError:
-                pass
-
-        if web_get_conf is not None:
-            try:
-                if str(web_get_conf["bitmain-work-mode"]).isdigit():
-                    return (
-                        True if int(web_get_conf["bitmain-work-mode"]) == 1 else False
-                    )
-                return False
-            except LookupError:
-                pass
+                return None
+        try:
+            return self._read_power_mode(web_get_conf) == 1
+        except APIError:
+            return None
 
     async def _get_kernel_log(self) -> str | None:
         return await self.web.get_kernel_log()
@@ -1255,27 +1308,23 @@ class AntminerOld(CGMiner):
                 pass
         return None
 
-    async def _is_sleep(self, web_get_conf: dict = None) -> bool:
-        """
-        Определяет, находится ли майнер в режиме "sleep" на основе параметра "bitmain-work-mode".
-
-        Если значение "bitmain-work-mode" равно 1, возвращается True, иначе False.
-        В случае ошибки получения конфигурации или отсутствия нужного ключа возвращается False.
-
-        :param web_get_conf: (необязательный) словарь с конфигурацией майнера.
-        :return: True, если режим равен 1, иначе False.
-        """
+    async def _is_sleep(self, web_get_conf: dict | None = None) -> bool | None:
+        """Возвращает состояние сна; неизвестный режим не считается пробуждением."""
         if web_get_conf is None:
             try:
                 web_get_conf = await self.web.get_miner_conf()
             except APIError:
-                return False
+                return None
 
+        if not isinstance(web_get_conf, dict):
+            return None
+        value = web_get_conf.get("bitmain-work-mode")
+        if isinstance(value, bool) or not isinstance(value, (str, int)):
+            return None
         try:
-            mode_str = str(web_get_conf["bitmain-work-mode"])
-            if mode_str.isdigit():
-                mode = int(mode_str)
-                return True if mode == "1" else False
-            return False
-        except (KeyError, LookupError, ValueError):
-            return False
+            mode = int(value)
+        except ValueError:
+            return None
+        if mode not in (0, 1, 2, 3, 5):
+            return None
+        return mode == 1

{pyasic_umhost-0.3.0.dev5 → pyasic_umhost-0.3.0.dev6}/pyasic/web/antminer.py

@@ -75,7 +75,10 @@ class AntminerModernWebAPI(BaseWebAPI):
                 else:
                     data = await client.get(url, auth=auth)
         except httpx.HTTPError as e:
-            return {"success": False, "message": f"HTTP error occurred: {str(e)}"}
+            return {
+                "success": False,
+                "message": f"HTTP {type(e).__name__} from Antminer CGI {command}: {e}",
+            }
         else:
             if data.status_code == 200:
                 try:
@@ -84,7 +87,10 @@ class AntminerModernWebAPI(BaseWebAPI):
                     return data.json()
                 except json.decoder.JSONDecodeError:
                     return {"success": False, "message": "Failed to decode JSON"}
-        return {"success": False, "message": "Unknown error occurred"}
+        return {
+            "success": False,
+            "message": f"HTTP {data.status_code} from Antminer CGI {command}",
+        }
 
     async def multicommand(
         self, *commands: str, ignore_errors: bool = False, allow_warning: bool = True

{pyasic_umhost-0.3.0.dev5 → pyasic_umhost-0.3.0.dev6}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pyasic-umhost"
-version = "0.3.0.dev5"
+version = "0.3.0.dev6"
 description = "Fork of pyasic — a simplified and standardized interface for Bitcoin ASICs."
 authors = [{name = "UpstreamData", email = "brett@upstreamdata.ca"}]
 repository = "https://github.com/UpstreamData/pyasic"