golem-vm-provider 0.1.59__tar.gz → 0.1.61__tar.gz

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: golem-vm-provider
-Version: 0.1.59
+Version: 0.1.61
 Summary: VM on Golem Provider Node - Run your own provider node to offer VMs on the Golem Network
 Keywords: golem,vm,provider,cloud,decentralized
 Author: Phillip Jensen
@@ -456,7 +456,7 @@ Response (per stream):
 
 Notes:
 - Endpoints return 400 when streaming is disabled (zero `STREAM_PAYMENT_ADDRESS`).
-- In development mode (`GOLEM_PROVIDER_ENVIRONMENT=development`) additional debug logs are emitted around stream verification and monitor ticks.
+- In development mode (`GOLEM_ENVIRONMENT=development`) additional debug logs are emitted around stream verification and monitor ticks.
 
  ## Operations
  
@@ -467,7 +467,7 @@ Notes:
 golem-provider start
 
 # Development mode with extra logs and reload
-GOLEM_PROVIDER_ENVIRONMENT=development golem-provider start --network testnet
+GOLEM_ENVIRONMENT=development golem-provider start --network testnet
 ```
 
 Run as a background service (no terminal):
@@ -485,7 +485,7 @@ golem-provider status [--json]
 
 ### Mode vs. Network
 
-- Development Mode (`GOLEM_PROVIDER_ENVIRONMENT=development`)
+- Development Mode (`GOLEM_ENVIRONMENT=development`)
   - Optimizes for local iteration: enables reload + debug logging and uses local defaults (e.g., local port check servers). May derive a local/LAN IP automatically and prefix the provider name with `DEVMODE-`.
   - Does not decide which chain you target.
 
@@ -498,7 +498,7 @@ golem-provider status [--json]
   - Selects the payments chain profile (e.g., `l2.holesky`, `mainnet`). Determines default payments RPC, faucet enablement, and symbols.
 
 Common setups:
-- Local dev on testnet: `GOLEM_PROVIDER_ENVIRONMENT=development` plus `--network testnet`.
+- Local dev on testnet: `GOLEM_ENVIRONMENT=development` plus `--network testnet`.
 - Staging on testnet: keep `ENVIRONMENT=production`, set `--network testnet` and testnet RPCs.
 - Production on mainnet: `ENVIRONMENT=production` with `--network mainnet` and mainnet RPCs.
 

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/README.md

@@ -411,7 +411,7 @@ Response (per stream):
 
 Notes:
 - Endpoints return 400 when streaming is disabled (zero `STREAM_PAYMENT_ADDRESS`).
-- In development mode (`GOLEM_PROVIDER_ENVIRONMENT=development`) additional debug logs are emitted around stream verification and monitor ticks.
+- In development mode (`GOLEM_ENVIRONMENT=development`) additional debug logs are emitted around stream verification and monitor ticks.
 
  ## Operations
  
@@ -422,7 +422,7 @@ Notes:
 golem-provider start
 
 # Development mode with extra logs and reload
-GOLEM_PROVIDER_ENVIRONMENT=development golem-provider start --network testnet
+GOLEM_ENVIRONMENT=development golem-provider start --network testnet
 ```
 
 Run as a background service (no terminal):
@@ -440,7 +440,7 @@ golem-provider status [--json]
 
 ### Mode vs. Network
 
-- Development Mode (`GOLEM_PROVIDER_ENVIRONMENT=development`)
+- Development Mode (`GOLEM_ENVIRONMENT=development`)
   - Optimizes for local iteration: enables reload + debug logging and uses local defaults (e.g., local port check servers). May derive a local/LAN IP automatically and prefix the provider name with `DEVMODE-`.
   - Does not decide which chain you target.
 
@@ -453,7 +453,7 @@ golem-provider status [--json]
   - Selects the payments chain profile (e.g., `l2.holesky`, `mainnet`). Determines default payments RPC, faucet enablement, and symbols.
 
 Common setups:
-- Local dev on testnet: `GOLEM_PROVIDER_ENVIRONMENT=development` plus `--network testnet`.
+- Local dev on testnet: `GOLEM_ENVIRONMENT=development` plus `--network testnet`.
 - Staging on testnet: keep `ENVIRONMENT=production`, set `--network testnet` and testnet RPCs.
 - Production on mainnet: `ENVIRONMENT=production` with `--network mainnet` and mainnet RPCs.
 

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/api/routes.py

@@ -31,6 +31,12 @@ from ..vm.multipass_adapter import MultipassError
 logger = setup_logger(__name__)
 router = APIRouter()
 
+# Expose Settings class at module scope for tests to monkeypatch default deployment lookup
+try:
+    from ..config import Settings as _Cfg  # type: ignore
+except Exception:  # noqa: BLE001
+    _Cfg = None  # type: ignore
+
 # Job status persisted in SQLite via JobStore (see Container.job_store)
 
 
@@ -56,8 +62,10 @@ async def create_vm(
         if spa and spa != "0x0000000000000000000000000000000000000000":
             if os.environ.get("PYTEST_CURRENT_TEST"):
                 try:
-                    from ..config import Settings as _Cfg  # type: ignore
-                    default_spa, _ = _Cfg._load_l2_deployment()  # type: ignore[attr-defined]
+                    if _Cfg is not None:
+                        default_spa, _ = _Cfg._load_l2_deployment()  # type: ignore[attr-defined]
+                    else:
+                        default_spa = None
                 except Exception:
                     default_spa = None
                 if not default_spa or spa.lower() != default_spa.lower():
@@ -302,6 +310,8 @@ async def provider_info(settings: Any = Depends(Provide[Container.config])) -> P
         provider_id=settings["PROVIDER_ID"],
         stream_payment_address=settings["STREAM_PAYMENT_ADDRESS"],
         glm_token_address=settings["GLM_TOKEN_ADDRESS"],
+        # Provide ETH-focused alias for clients; keep legacy field too
+        eth_token_address=settings["GLM_TOKEN_ADDRESS"],
         ip_address=ip_addr,
         country=(settings.get("PROVIDER_COUNTRY") if isinstance(settings, dict) else getattr(settings, "PROVIDER_COUNTRY", None)),
         platform=platform_str,

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/config.py

@@ -27,7 +27,7 @@ def ensure_config() -> None:
             created = True
 
     if not env_file.exists():
-        env_file.write_text("GOLEM_PROVIDER_ENVIRONMENT=production\n")
+        env_file.write_text("GOLEM_ENVIRONMENT=production\n")
         created = True
 
     from .security.ethereum import EthereumIdentity
@@ -55,8 +55,12 @@ class Settings(BaseSettings):
     PORT: int = 7466
     SKIP_PORT_VERIFICATION: bool = False
     ENVIRONMENT: str = "production"
-    # Logical network selector for annotation and client defaults
-    NETWORK: str = "mainnet"  # one of: "testnet", "mainnet"
+    # Logical network selector for advertisement scope and client defaults
+    # If not explicitly provided, computed by validator below (dev -> testnet, else -> mainnet)
+    NETWORK: str = Field(
+        default="",
+        description="Logical Golem network: 'testnet' or 'mainnet'"
+    )
 
     # Payments chain selection (modular network profiles). Keep default on l2.holesky
     PAYMENTS_NETWORK: str = Field(
@@ -73,6 +77,37 @@ class Settings(BaseSettings):
     def DEV_MODE(self) -> bool:
         return self.ENVIRONMENT == "development"
 
+    @field_validator("ENVIRONMENT", mode='before')
+    @classmethod
+    def prefer_global_env(cls, v: str) -> str:
+        """Prefer unified GOLEM_ENVIRONMENT when provided; fallback to service-specific env."""
+        ge = os.environ.get("GOLEM_ENVIRONMENT")
+        if ge:
+            return ge
+        return v
+
+    @field_validator("NETWORK", mode='before')
+    @classmethod
+    def resolve_network(cls, v: str, values: dict) -> str:
+        """Resolve logical network with sensible defaults.
+
+        Priority:
+        1) Explicit override via GOLEM_PROVIDER_NETWORK env or provided value
+        2) If ENVIRONMENT == development -> 'testnet'
+        3) Otherwise -> 'mainnet'
+        """
+        # Prefer explicit provider-scoped env override
+        env_override = os.environ.get("GOLEM_PROVIDER_NETWORK")
+        if env_override:
+            return env_override
+        # If value provided (via settings or direct assignment), keep it
+        val = (v or "").strip()
+        if val:
+            return val
+        # Default based on environment
+        env = (values.data.get("ENVIRONMENT") or "").lower()
+        return "testnet" if env == "development" else "mainnet"
+
     @field_validator("SKIP_PORT_VERIFICATION", mode='before')
     def set_skip_verification(cls, v: bool, values: dict) -> bool:
         """Set skip verification based on debug mode."""
@@ -186,6 +221,16 @@ class Settings(BaseSettings):
         description="Min withdrawable amount (wei) before triggering withdraw"
     )
 
+    # Behavior on exhausted runway
+    STREAM_REMOVE_MAPPING_ON_EXHAUSTED: bool = Field(
+        default=True,
+        description="When true, remove the VM->stream mapping after a successful stop on exhausted runway to prevent repeated stop attempts."
+    )
+    STREAM_DELETE_ON_EXHAUSTED: bool = Field(
+        default=False,
+        description="When true, delete the VM entirely once runway is exhausted and the VM has been stopped."
+    )
+
     # Shutdown behavior
     STOP_VMS_ON_EXIT: bool = Field(
         default=False,

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/discovery/golem_base_advertiser.py

@@ -1,7 +1,14 @@
 import asyncio
 from typing import Optional
 
-from golem_base_sdk import GolemBaseClient, GolemBaseCreate, GolemBaseUpdate, GolemBaseDelete, Annotation
+from golem_base_sdk import (
+    GolemBaseClient,
+    GolemBaseCreate,
+    GolemBaseUpdate,
+    GolemBaseDelete,
+    Annotation,
+    GolemBaseExtend,
+)
 from .advertiser import Advertiser
 from .golem_base_utils import get_provider_entity_keys
 from ..config import settings
@@ -36,7 +43,9 @@ class GolemBaseAdvertiser(Advertiser):
         try:
             while not self._stop_event.is_set():
                 await self.post_advertisement()
-                await asyncio.sleep(settings.GOLEM_BASE_ADVERTISEMENT_INTERVAL)
+                # Check more frequently than full TTL to ensure seamless renewal
+                interval = max(30, int(settings.GOLEM_BASE_ADVERTISEMENT_INTERVAL // 3))
+                await asyncio.sleep(interval)
         finally:
             await self.stop()
 
@@ -122,14 +131,36 @@ class GolemBaseAdvertiser(Advertiser):
                 logger.info(f"Current on-chain annotations: {current_annotations}")
                 logger.info(f"Expected annotations based on current config: {expected_annotations}")
 
+                desired_btl_blocks = int(settings.GOLEM_BASE_ADVERTISEMENT_INTERVAL) * 2  # ~2 blocks/sec
+
                 if sorted(current_annotations.items()) == sorted(expected_annotations.items()):
-                    logger.info("Advertisement is up-to-date. Waiting for expiration.")
+                    # Refresh TTL proactively if nearing expiry
+                    try:
+                        current_block = await self.client.http_client().eth.get_block_number()
+                        remaining_blocks = int(metadata.expires_at_block) - int(current_block)
+                    except Exception:
+                        # If we cannot determine remaining blocks, extend defensively
+                        remaining_blocks = 0
+
+                    # Refresh when below 20% of desired TTL (or if unknown/negative)
+                    refresh_threshold = max(10, desired_btl_blocks // 5)
+                    if remaining_blocks <= refresh_threshold:
+                        logger.info(
+                            f"Extending advertisement TTL (remaining_blocks={remaining_blocks}, threshold={refresh_threshold})."
+                        )
+                        ext = GolemBaseExtend(entity_key=entity_key, number_of_blocks=desired_btl_blocks)
+                        await self.client.extend_entities([ext])
+                        logger.info(f"Extended advertisement. Entity key: {entity_key}")
+                    else:
+                        logger.info(
+                            f"Advertisement up-to-date; TTL sufficient (remaining_blocks={remaining_blocks})."
+                        )
                 else:
                     logger.info("Advertisement is outdated. Updating.")
                     update = GolemBaseUpdate(
                         entity_key=entity_key,
                         data=b"",
-                        btl=settings.ADVERTISEMENT_INTERVAL * 2,
+                        btl=desired_btl_blocks,
                         string_annotations=string_annotations,
                         numeric_annotations=numeric_annotations,
                     )
@@ -144,9 +175,10 @@ class GolemBaseAdvertiser(Advertiser):
 
     async def _create_advertisement(self, string_annotations, numeric_annotations):
         """Helper to create a new advertisement."""
+        desired_btl_blocks = int(settings.GOLEM_BASE_ADVERTISEMENT_INTERVAL) * 2  # ~2 blocks/sec
         entity = GolemBaseCreate(
             data=b"",
-            btl=settings.ADVERTISEMENT_INTERVAL * 2,
+            btl=desired_btl_blocks,
             string_annotations=string_annotations,
             numeric_annotations=numeric_annotations,
         )

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/main.py

@@ -15,6 +15,7 @@ if "--json" in _sys.argv:
 
 # Defer heavy local imports (may import config) until after we decide on silence
 from .container import Container
+from .config import settings  # used by pricing CLI and server commands
 from .service import ProviderService
 
 logger = setup_logger(__name__)
@@ -324,8 +325,8 @@ def status(json_out: bool = typer.Option(False, "--json", help="Output machine-r
     latest = _get_latest_version_from_pypi(pkg)
     update_available = bool(latest and current != latest)
 
-    # Environment
-    env = os.environ.get("GOLEM_PROVIDER_ENVIRONMENT", _settings.ENVIRONMENT)
+    # Environment (use unified GOLEM_ENVIRONMENT only)
+    env = os.environ.get("GOLEM_ENVIRONMENT", _settings.ENVIRONMENT)
     net = getattr(_settings, "NETWORK", None)
     dev_mode = env == "development" or bool(getattr(_settings, "DEV_MODE", False))
 
@@ -1323,7 +1324,7 @@ def stop(timeout: int = typer.Option(15, "--timeout", help="Seconds to wait for
     _remove_pid_file()
     print("Provider stopped")
 
-# Removed separate 'dev' command; use environment GOLEM_PROVIDER_ENVIRONMENT=development instead.
+# Removed separate 'dev' command; use environment GOLEM_ENVIRONMENT=development instead.
 
 def _env_path_for(dev_mode: Optional[bool]) -> str:
     from pathlib import Path
@@ -1534,9 +1535,10 @@ def run_server(
     from pathlib import Path
     from dotenv import load_dotenv
     import uvicorn
-    # Decide dev mode from explicit arg or environment
+    # Decide dev mode from explicit arg or environment (unified only)
     if dev_mode is None:
-        dev_mode = os.environ.get("GOLEM_PROVIDER_ENVIRONMENT", "").lower() == "development"
+        env_val = os.environ.get("GOLEM_ENVIRONMENT", "")
+        dev_mode = env_val.lower() == "development"
 
     # Load appropriate .env file based on mode
     env_file = ".env.dev" if dev_mode else ".env"

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/payments/monitor.py

@@ -54,7 +54,18 @@ class StreamMonitor:
                     try:
                         s = self.reader.get_stream(stream_id)
                     except Exception as e:
-                        logger.warning(f"stream {stream_id} lookup failed: {e}")
+                        # No payment info available; delete the VM and remove mapping per unified policy
+                        logger.info(
+                            f"Deleting VM {vm_id} due to missing/unavailable payment stream (id={stream_id}): {e}"
+                        )
+                        try:
+                            await self.vm_service.delete_vm(vm_id)
+                        except Exception as del_err:
+                            logger.warning(f"delete_vm failed for {vm_id} after stream lookup failure: {del_err}")
+                        try:
+                            await self.stream_map.remove(vm_id)
+                        except Exception as rem_err:
+                            logger.debug(f"failed to remove vm {vm_id} from stream map: {rem_err}")
                         continue
                     # Stop VM if remaining runway < threshold
                     remaining = max(int(s["stopTime"]) - int(now), 0)
@@ -69,33 +80,72 @@ class StreamMonitor:
                         )
                         try:
                             await self.vm_service.delete_vm(vm_id)
+                            # Best-effort verification of deletion for investigation
+                            try:
+                                _ = await self.vm_service.get_vm_status(vm_id)
+                                logger.info(
+                                    f"Post-delete status check: VM {vm_id} still present after delete request"
+                                )
+                            except VMNotFoundError:
+                                logger.info(
+                                    f"Post-delete status check: VM {vm_id} not found (expected)"
+                                )
+                            except Exception as chk_err:
+                                logger.debug(
+                                    f"Post-delete status check failed for {vm_id}: {chk_err}"
+                                )
                         except Exception as e:
                             logger.warning(f"delete_vm failed for {vm_id}: {e}")
                         try:
                             await self.stream_map.remove(vm_id)
+                            logger.debug(f"Removed {vm_id} from stream map after delete")
                         except Exception as e:
                             logger.debug(f"failed to remove vm {vm_id} from stream map: {e}")
                         continue
 
-                    # Only stop a VM when runway is completely empty
+                    # If runway is exhausted, delete the VM and remove mapping
                     if remaining == 0:
                         logger.info(
-                            f"Stopping VM {vm_id} as stream runway is exhausted (id={stream_id}, now={now}, stop={s.get('stopTime')})"
+                            f"Deleting VM {vm_id} as stream runway is exhausted (id={stream_id}, now={now}, stop={s.get('stopTime')})"
                         )
+                        # Capture pre-delete status for context
+                        try:
+                            pre = await self.vm_service.get_vm_status(vm_id)
+                            logger.info(
+                                f"Pre-delete status for {vm_id}: status={getattr(pre, 'status', '?')} ip={getattr(pre, 'ip_address', '?')}"
+                            )
+                        except VMNotFoundError:
+                            logger.info(
+                                f"Pre-delete status for {vm_id}: not found (will remove mapping)"
+                            )
+                        except Exception as pre_err:
+                            logger.debug(f"Pre-delete status check failed for {vm_id}: {pre_err}")
+
                         try:
-                            await self.vm_service.stop_vm(vm_id)
-                        except VMNotFoundError as e:
-                            # If the VM cannot be found, remove it from the stream map
-                            # to avoid repeated stop attempts and log spam.
-                            logger.warning(f"stop_vm failed for {vm_id}: {e}")
+                            await self.vm_service.delete_vm(vm_id)
+                            # Verify deletion
                             try:
-                                await self.stream_map.remove(vm_id)
-                            except Exception as rem_err:
+                                _ = await self.vm_service.get_vm_status(vm_id)
+                                logger.info(
+                                    f"Post-delete status check: VM {vm_id} still present after delete request"
+                                )
+                            except VMNotFoundError:
+                                logger.info(
+                                    f"Post-delete status check: VM {vm_id} not found (expected)"
+                                )
+                            except Exception as chk_err:
                                 logger.debug(
-                                    f"failed to remove vm {vm_id} from stream map after not-found: {rem_err}"
+                                    f"Post-delete status check failed for {vm_id}: {chk_err}"
                                 )
                         except Exception as e:
-                            logger.warning(f"stop_vm failed for {vm_id}: {e}")
+                            logger.warning(f"delete_vm failed for {vm_id}: {e}")
+                        try:
+                            await self.stream_map.remove(vm_id)
+                            logger.info(f"Removed mapping for {vm_id} after delete on exhausted runway")
+                        except Exception as rem_err:
+                            logger.debug(
+                                f"failed to remove vm {vm_id} from stream map after delete: {rem_err}"
+                            )
                         continue
 
                     # Otherwise, do not stop; just log health and consider withdrawals

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/vm/multipass_adapter.py

@@ -32,6 +32,26 @@ class MultipassAdapter(VMProvider):
         self.proxy_manager = proxy_manager
         self.name_mapper = name_mapper
 
+    @staticmethod
+    def _safe_int(value, default: int = 0) -> int:
+        """Best-effort int conversion that treats missing/blank values as default.
+
+        Multipass may return empty strings for numeric fields (e.g., when a VM is
+        stopped). This helper prevents ValueError by mapping '', None, or
+        unparsable values to a sensible default.
+        """
+        try:
+            if value is None:
+                return default
+            if isinstance(value, str):
+                v = value.strip()
+                if v == "":
+                    return default
+                return int(v)
+            return int(value)
+        except (ValueError, TypeError):
+            return default
+
     async def _run_multipass(self, args: List[str], check: bool = True) -> subprocess.CompletedProcess:
         """Run a multipass command."""
         # Commands that produce JSON or version info that we need to parse.
@@ -67,7 +87,8 @@ class MultipassAdapter(VMProvider):
         """Get detailed information about a VM."""
         try:
             result = await self._run_multipass(["info", vm_id, "--format", "json"])
-            logger.info(f"Raw multipass info for {vm_id}: {result.stdout}")
+            # Only log raw multipass output in debug mode to avoid noisy logs
+            logger.debug(f"Raw multipass info for {vm_id}: {result.stdout}")
             info = json.loads(result.stdout)
             vm_info = info["info"][vm_id]
             essential_fields = ["state", "ipv4", "cpu_count", "memory", "disks"]
@@ -160,8 +181,8 @@ class MultipassAdapter(VMProvider):
         vms: List[VMInfo] = []
         for requestor_name, multipass_name in list(all_mappings.items()):
             try:
-                # get_vm_status expects multipass_name
-                vm_info = await self.get_vm_status(multipass_name)
+                # Pass requestor id; get_vm_status accepts either id
+                vm_info = await self.get_vm_status(requestor_name)
                 vms.append(vm_info)
             except VMNotFoundError:
                 logger.warning(
@@ -188,30 +209,40 @@ class MultipassAdapter(VMProvider):
         await self._run_multipass(["stop", multipass_name])
         return await self.get_vm_status(multipass_name)
 
-    async def get_vm_status(self, multipass_name: str) -> VMInfo:
-        """Get the status of a VM."""
+    async def get_vm_status(self, name_or_id: str) -> VMInfo:
+        """Get VM status by multipass name or requestor id."""
+        # Resolve identifiers flexibly
+        requestor_name = await self.name_mapper.get_requestor_name(name_or_id)
+        if requestor_name:
+            multipass_name = name_or_id
+        else:
+            multipass_name = await self.name_mapper.get_multipass_name(name_or_id)
+            if not multipass_name:
+                raise VMNotFoundError(f"VM {name_or_id} mapping not found")
+            requestor_name = name_or_id
         try:
             info = await self._get_vm_info(multipass_name)
         except MultipassError:
             raise VMNotFoundError(f"VM {multipass_name} not found in multipass")
 
-        requestor_name = await self.name_mapper.get_requestor_name(multipass_name)
-        if not requestor_name:
-            raise VMNotFoundError(f"Mapping for VM {multipass_name} not found")
-
         ipv4 = info.get("ipv4")
         ip_address = ipv4[0] if ipv4 else None
         logger.debug(f"Parsed VM info for {requestor_name}: {info}")
         
         disks_info = info.get("disks", {})
-        total_storage = sum(int(disk.get("total", 0)) for disk in disks_info.values())
+        total_storage = 0
+        for disk in disks_info.values():
+            total_storage += self._safe_int(disk.get("total"), 0)
+
+        # Memory reported by multipass is in bytes; default to 1 GiB if missing/blank
+        mem_total_bytes = self._safe_int(info.get("memory", {}).get("total"), 1024**3)
         vm_info_obj = VMInfo(
             id=requestor_name,
             name=requestor_name,
             status=VMStatus(info["state"].lower()),
             resources=VMResources(
-                cpu=int(info.get("cpu_count", "1")),
-                memory=round(info.get("memory", {}).get("total", 1024**3) / (1024**3)),
+                cpu=self._safe_int(info.get("cpu_count"), 1),
+                memory=round(mem_total_bytes / (1024**3)),
                 storage=round(total_storage / (1024**3)) if total_storage > 0 else 10
             ),
             ip_address=ip_address,
@@ -228,10 +259,13 @@ class MultipassAdapter(VMProvider):
             try:
                 info = await self._get_vm_info(multipass_name)
                 disks_info = info.get("disks", {})
-                total_storage = sum(int(disk.get("total", 0)) for disk in disks_info.values())
+                total_storage = 0
+                for disk in disks_info.values():
+                    total_storage += self._safe_int(disk.get("total"), 0)
+                mem_total_bytes = self._safe_int(info.get("memory", {}).get("total"), 1024**3)
                 vm_resources[requestor_name] = VMResources(
-                    cpu=int(info.get("cpu_count", "1")),
-                    memory=round(info.get("memory", {}).get("total", 1024**3) / (1024**3)),
+                    cpu=self._safe_int(info.get("cpu_count"), 1),
+                    memory=round(mem_total_bytes / (1024**3)),
                     storage=round(total_storage / (1024**3)) if total_storage > 0 else 10
                 )
             except (MultipassError, VMNotFoundError):

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/provider/vm/service.py

@@ -59,6 +59,7 @@ class VMService:
 
         try:
             vm_info = await self.provider.get_vm_status(multipass_name)
+            logger.info(f"Deleting VM {vm_id} (multipass={multipass_name}) with status={vm_info.status}")
             await self.provider.delete_vm(multipass_name)
             await self.resource_tracker.deallocate(vm_info.resources, vm_id)
             # Optional: best-effort on-chain termination if we have a mapping
@@ -83,7 +84,9 @@ class VMService:
         multipass_name = await self.name_mapper.get_multipass_name(vm_id)
         if not multipass_name:
             raise VMNotFoundError(f"VM {vm_id} not found")
+        logger.info(f"Stopping VM {vm_id} (multipass={multipass_name})")
         vm = await self.provider.stop_vm(multipass_name)
+        logger.info(f"Stopped VM {vm_id} result status={getattr(vm, 'status', '?')}")
         # Optional: best-effort withdraw for active stream
         try:
             if self.blockchain_client:

{golem_vm_provider-0.1.59 → golem_vm_provider-0.1.61}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "golem-vm-provider"
-version = "0.1.59"
+version = "0.1.61"
 description = "VM on Golem Provider Node - Run your own provider node to offer VMs on the Golem Network"
 authors = ["Phillip Jensen <phillip+vm-on-golem@golemgrid.com>"]
 readme = "README.md"