proxcli 0.1.1__tar.gz → 0.2.1__tar.gz

proxcli-0.2.1/.env.example

@@ -0,0 +1,3 @@
+# PyPI publish token
+# Get yours at: https://pypi.org/manage/account/token/
+PYPI_TOKEN=pypi-xxxxxxxxxxxxxxxx

{proxcli-0.1.1 → proxcli-0.2.1}/.gitignore

@@ -9,6 +9,9 @@ wheels/
 # Virtual environments
 .venv
 
+# Environment / secrets
+.env
+
 # IDE / Editor
 *.swp
 *.swo

{proxcli-0.1.1 → proxcli-0.2.1}/CHANGELOG.md

@@ -7,7 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [0.1.1] - 2026-06-20
+## [0.2.1] - 2026-06-20
+
+### Fixed
+- `--version` now reads from installed package metadata (`importlib.metadata`) instead of a hardcoded string.
+
+## [0.2.0] - 2026-06-20
+
+### Added
+- `proxmox storage upload` command for uploading ISO, vztmpl, and import files to storage via multipart/form-data.
+- `ProxmoxClient.upload()` method supporting file uploads with content type selection.
 
 ### Changed
 - `proxmox vm create --vmid` and `proxmox container create --vmid` are now optional.
@@ -32,5 +41,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - CSRF ticket auto-refresh on 401.
 - AI-agent-friendly: default JSON output, strict exit codes, `--dry-run` mode.
 
+[0.2.1]: https://github.com/xezpeleta/proxcli/releases/tag/v0.2.1
+[0.2.0]: https://github.com/xezpeleta/proxcli/releases/tag/v0.2.0
 [0.1.1]: https://github.com/xezpeleta/proxcli/releases/tag/v0.1.1
 [0.1.0]: https://github.com/xezpeleta/proxcli/releases/tag/v0.1.0

{proxcli-0.1.1 → proxcli-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: proxcli
-Version: 0.1.1
+Version: 0.2.1
 Summary: A CLI tool to interact with Proxmox VE nodes and clusters via the REST API
 Author-email: Xabi Ezpeleta <xezpeleta@gmail.com>
 License: MIT

{proxcli-0.1.1 → proxcli-0.2.1}/proxmox/cli/main.py

@@ -5,6 +5,7 @@ from __future__ import annotations
 import argparse
 import os
 import sys
+from importlib.metadata import version
 from typing import Any
 
 from proxmox.client.auth import AuthManager
@@ -48,7 +49,7 @@ def build_root_parser() -> argparse.ArgumentParser:
     )
     parser.add_argument("--verbose", action="store_true", help="Enable debug output to stderr")
     parser.add_argument(
-        "--version", action="version", version="proxmox 0.1.0"
+        "--version", action="version", version=f"proxmox {version('proxcli')}"
     )
 
     subparsers = parser.add_subparsers(dest="resource", title="resources", required=False)

{proxcli-0.1.1 → proxcli-0.2.1}/proxmox/cli/storage.py

@@ -28,6 +28,19 @@ def register_storage_parser(subparsers: argparse._SubParsersAction) -> None:
     st_content.add_argument("--node", help="Node name (auto-detected if omitted)")
     st_content.set_defaults(func=_st_content)
 
+    # --- storage upload ---
+    st_upload = st_sub.add_parser("upload", help="Upload a file to storage")
+    st_upload.add_argument("--node", required=True, help="Target node")
+    st_upload.add_argument("--storage", required=True, help="Storage ID (e.g. 'local')")
+    st_upload.add_argument("--file", required=True, help="Path to the local file")
+    st_upload.add_argument(
+        "--content-type",
+        default="iso",
+        choices=["iso", "vztmpl", "import"],
+        help="Content type (default: iso)",
+    )
+    st_upload.set_defaults(func=_st_upload)
+
 
 def _st_list(args: argparse.Namespace, client: ProxmoxClient) -> dict | list:
     if args.node:
@@ -61,3 +74,12 @@ def _st_content(args: argparse.Namespace, client: ProxmoxClient) -> dict | list:
     if not node:
         return {"error": f"Could not determine node for storage '{args.storage_name}'"}
     return client.get(f"/nodes/{node}/storage/{args.storage_name}/content")
+
+
+def _st_upload(args: argparse.Namespace, client: ProxmoxClient) -> dict:
+    return client.upload(
+        node=args.node,
+        storage=args.storage,
+        file_path=args.file,
+        content_type=args.content_type,
+    )

{proxcli-0.1.1 → proxcli-0.2.1}/proxmox/client/client.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import os
 import time
 from typing import Any
 
@@ -93,6 +94,70 @@ class ProxmoxClient:
     ) -> dict[str, Any] | list[Any]:
         return self.request("DELETE", path, params=params)
 
+    def upload(
+        self,
+        node: str,
+        storage: str,
+        file_path: str,
+        *,
+        content_type: str = "iso",
+    ) -> dict[str, Any]:
+        """Upload a file to a storage via multipart/form-data.
+
+        Args:
+            node: Target node name.
+            storage: Storage ID (e.g. 'local').
+            file_path: Path to the local file to upload.
+            content_type: Proxmox content type ('iso', 'vztmpl', 'import').
+        """
+        path = f"/nodes/{node}/storage/{storage}/upload"
+        full_url = f"{self._base_url}/api2/json{path}"
+
+        if self._dry_run:
+            print(f"POST {full_url}")
+            print(f"Headers: {self._auth.get_headers()}")
+            print(f"File: {file_path} (content={content_type})")
+            return {}
+
+        if not os.path.isfile(file_path):
+            raise FileNotFoundError(f"File not found: {file_path}")
+
+        filename = os.path.basename(file_path)
+        file_size = os.path.getsize(file_path)
+
+        self._debug(f"POST {full_url}")
+        self._debug(f"  file: {file_path} ({file_size} bytes)")
+
+        headers = self._auth.get_headers()
+
+        with open(file_path, "rb") as f:
+            try:
+                resp = httpx.post(
+                    full_url,
+                    files={"filename": (filename, f, "application/octet-stream")},
+                    data={"content": content_type},
+                    headers=headers,
+                    timeout=self._timeout,
+                    verify=self._verify_tls,
+                )
+            except httpx.RequestError as exc:
+                msg = str(exc)
+                if "SSL" in msg or "certificate" in msg.lower():
+                    msg += "\nHint: use --insecure to skip TLS verification"
+                raise ProxmoxAPIError(0, {"message": msg}, full_url) from exc
+
+        self._debug(f"  ← {resp.status_code}")
+
+        if not (200 <= resp.status_code < 300):
+            try:
+                body = resp.json()
+            except Exception:
+                body = {"message": resp.text}
+            raise ProxmoxAPIError(resp.status_code, body, full_url)
+
+        envelope = resp.json()
+        return envelope.get("data", envelope)
+
     def set_credentials(self, username: str, password: str) -> None:
         """Store credentials for lazy / auto-refresh authentication."""
         self._username = username

{proxcli-0.1.1 → proxcli-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "proxcli"
-version = "0.1.1"
+version = "0.2.1"
 description = "A CLI tool to interact with Proxmox VE nodes and clusters via the REST API"
 readme = "README.md"
 authors = [

{proxcli-0.1.1 → proxcli-0.2.1}/tests/test_client.py

@@ -160,3 +160,55 @@ class TestResolveVmid:
         client = ProxmoxClient("https://pve:8006", AuthManager())
         result = resolve_vmid(client, None)
         assert result == 108
+
+
+class TestStorageUpload:
+    def test_upload_success(self, mock_httpx_client, tmp_path):
+        """Upload sends multipart request and returns data."""
+        iso_file = tmp_path / "test.iso"
+        iso_file.write_bytes(b"fake iso content")
+
+        mock_httpx_client.add_response(
+            method="POST",
+            url="https://pve:8006/api2/json/nodes/pve01/storage/local/upload",
+            json={"data": "file test.iso uploaded"},
+        )
+        client = ProxmoxClient("https://pve:8006", AuthManager())
+        result = client.upload("pve01", "local", str(iso_file))
+        assert result == "file test.iso uploaded"
+
+        # Verify the request was multipart
+        requests = mock_httpx_client.get_requests()
+        assert len(requests) == 1
+        assert "multipart/form-data" in requests[0].headers.get("content-type", "")
+
+    def test_upload_dry_run(self, capsys):
+        """Upload in dry-run mode prints request without executing."""
+        client = ProxmoxClient("https://pve:8006", AuthManager(), dry_run=True)
+        result = client.upload("pve01", "local", "/fake/path.iso")
+        assert result == {}
+        captured = capsys.readouterr()
+        assert "POST https://pve:8006/api2/json/nodes/pve01/storage/local/upload" in captured.out
+        assert "/fake/path.iso" in captured.out
+
+    def test_upload_file_not_found(self):
+        """Upload raises FileNotFoundError for missing files."""
+        client = ProxmoxClient("https://pve:8006", AuthManager())
+        with pytest.raises(FileNotFoundError, match="File not found"):
+            client.upload("pve01", "local", "/nonexistent/file.iso")
+
+    def test_upload_error_response(self, mock_httpx_client, tmp_path):
+        """Upload raises ProxmoxAPIError on non-2xx response."""
+        iso_file = tmp_path / "test.iso"
+        iso_file.write_bytes(b"data")
+
+        mock_httpx_client.add_response(
+            method="POST",
+            url="https://pve:8006/api2/json/nodes/pve01/storage/local/upload",
+            status_code=403,
+            json={"message": "permission denied"},
+        )
+        client = ProxmoxClient("https://pve:8006", AuthManager())
+        with pytest.raises(ProxmoxAPIError) as exc_info:
+            client.upload("pve01", "local", str(iso_file))
+        assert exc_info.value.status_code == 403

{proxcli-0.1.1 → proxcli-0.2.1}/uv.lock

@@ -254,7 +254,7 @@ wheels = [
 
 [[package]]
 name = "proxcli"
-version = "0.1.0"
+version = "0.2.0"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },