ibee-cli 0.5.0__tar.gz → 0.5.2__tar.gz

{ibee_cli-0.5.0/src/ibee_cli.egg-info → ibee_cli-0.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ibee-cli
-Version: 0.5.0
+Version: 0.5.2
 Summary: Official command-line interface for the IBEE Solutions cloud platform
 Author-email: IBEE Solutions <support@ibee.ai>
 License-Expression: MIT
@@ -136,6 +136,13 @@ the command can be repeated without doing the write twice. `--wait` polls the
 operation every `--poll-interval` seconds (default 5) for up to `--timeout` seconds
 (default 1200).
 
+## Development
+
+`pip install -e ".[dev]" && pytest -q`. CI also runs `python scripts/check_sdk_calls.py`
+against `ibee` from PyPI and from ibee-python `main`: every SDK method the CLI calls
+(including the names the VM commands build from templates) must exist in the installed
+SDK, and the CLI must make no raw HTTP calls.
+
 ## Related
 
 - [Python SDK](https://github.com/devs-ibee/ibee-python): `pip install ibee`

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/README.md

@@ -111,6 +111,13 @@ the command can be repeated without doing the write twice. `--wait` polls the
 operation every `--poll-interval` seconds (default 5) for up to `--timeout` seconds
 (default 1200).
 
+## Development
+
+`pip install -e ".[dev]" && pytest -q`. CI also runs `python scripts/check_sdk_calls.py`
+against `ibee` from PyPI and from ibee-python `main`: every SDK method the CLI calls
+(including the names the VM commands build from templates) must exist in the installed
+SDK, and the CLI must make no raw HTTP calls.
+
 ## Related
 
 - [Python SDK](https://github.com/devs-ibee/ibee-python): `pip install ibee`

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ibee-cli"
-version = "0.5.0"
+version = "0.5.2"
 description = "Official command-line interface for the IBEE Solutions cloud platform"
 readme = "README.md"
 license = "MIT"

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/src/ibee_cli/__init__.py

@@ -1,3 +1,3 @@
 """IBEE Solutions command-line interface."""
 
-__version__ = "0.5.0"
+__version__ = "0.5.2"

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/src/ibee_cli/commands/vm_lifecycle.py

@@ -177,7 +177,7 @@ def register_vm_commands(app: typer.Typer, spec: VmCommandSpec) -> None:
         def create_vm(
             ctx: typer.Context,
             name: str = typer.Argument(..., help="VM name (with --count, the base name)"),
-            site_id: Optional[str] = typer.Option(None, "--site-id", help="Site. See `ibee compute sites`."),
+            site_id: str = typer.Option(..., "--site-id", help="Site (required). See `ibee compute sites`."),
             plan_id: str = typer.Option(..., "--plan-id", help="Plan. See `ibee compute plans --vm-type cloud`."),
             template_id: str = typer.Option(..., "--template-id", help="Image. See `ibee compute images --vm-type cloud`."),
             os_type: Optional[str] = typer.Option(None, "--os-type", help="Default: the image's"),
@@ -228,7 +228,7 @@ def register_vm_commands(app: typer.Typer, spec: VmCommandSpec) -> None:
         def create_gpu(
             ctx: typer.Context,
             name: str = typer.Argument(..., help="VM name (with --count, the base name)"),
-            site_id: Optional[str] = typer.Option(None, "--site-id", help="Site. See `ibee compute sites`."),
+            site_id: str = typer.Option(..., "--site-id", help="Site (required). See `ibee compute sites`."),
             plan_id: str = typer.Option(..., "--plan-id", help="Plan. See `ibee compute plans --vm-type gpu`."),
             template_id: str = typer.Option(..., "--template-id", help="Image. See `ibee compute images --vm-type gpu`."),
             os_type: Optional[str] = typer.Option(None, "--os-type", help="Default: the image's"),

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/src/ibee_cli/render.py

@@ -216,10 +216,14 @@ def to_yaml(payload: Any) -> str:
 # ---------------------------------------------------------------------------
 
 
-def _items(data: Any) -> list[Any]:
+def _items(data: Any, id_field: Optional[str] = None) -> list[Any]:
     if isinstance(data, list):
         return data
     if isinstance(data, dict):
+        if id_field and data.get(id_field) not in (None, ""):
+            # One object that carries the command's id (a firewall group, a load
+            # balancer): its nested ``rules`` or ``subnets`` are not the result.
+            return [data]
         for key in COLLECTION_KEYS:
             if isinstance(data.get(key), list):
                 return data[key]
@@ -241,7 +245,7 @@ def _identifier(item: Any, id_field: Optional[str]) -> Optional[str]:
 def identifiers(payload: Any, id_field: Optional[str] = None) -> list[str]:
     """Identifiers for ``-o id``: one per item of a collection, or one for an object."""
     ids = []
-    for item in _items(to_data(payload)):
+    for item in _items(to_data(payload), id_field):
         value = _identifier(item, id_field)
         if value is not None:
             ids.append(value)

{ibee_cli-0.5.0 → ibee_cli-0.5.2/src/ibee_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ibee-cli
-Version: 0.5.0
+Version: 0.5.2
 Summary: Official command-line interface for the IBEE Solutions cloud platform
 Author-email: IBEE Solutions <support@ibee.ai>
 License-Expression: MIT
@@ -136,6 +136,13 @@ the command can be repeated without doing the write twice. `--wait` polls the
 operation every `--poll-interval` seconds (default 5) for up to `--timeout` seconds
 (default 1200).
 
+## Development
+
+`pip install -e ".[dev]" && pytest -q`. CI also runs `python scripts/check_sdk_calls.py`
+against `ibee` from PyPI and from ibee-python `main`: every SDK method the CLI calls
+(including the names the VM commands build from templates) must exist in the installed
+SDK, and the CLI must make no raw HTTP calls.
+
 ## Related
 
 - [Python SDK](https://github.com/devs-ibee/ibee-python): `pip install ibee`

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/tests/test_cli_core.py

@@ -29,7 +29,7 @@ def test_help_and_version() -> None:
     for group in ("billing", "block-storage", "buckets", "cdn", "secrets", "vms", "gpus", "console", "vpcs",
                   "reserved-ips", "firewalls", "load-balancers", "compute", "ops"):
         assert group in result.output
-    assert runner.invoke(app, ["--version"]).output.strip() == f"ibee-cli {__version__}" == "ibee-cli 0.5.0"
+    assert runner.invoke(app, ["--version"]).output.strip() == f"ibee-cli {__version__}" == "ibee-cli 0.5.2"
 
 
 def test_missing_token_and_workspace_are_usage_errors(monkeypatch: pytest.MonkeyPatch) -> None:

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/tests/test_networking_commands.py

@@ -10,9 +10,11 @@ import json
 
 import pytest
 from click import unstyle
-from _net_fixtures import FW, LB, NAT_CATALOG, RIP, RIP_CATALOG, V, WS, gateway_record, group, lb, pf_rule, plain, rip, run, vpc
+from _net_fixtures import (
+    FW, LB, NAT_CATALOG, RIP, RIP_CATALOG, V, WS, gateway_record, group, lb, pf_rule, plain, rip, run, subnet, vpc,
+)
 
-from ibee_cli.render import EXIT_CODES
+from ibee_cli.render import EXIT_CODES, identifiers
 
 BACKENDS = [{"type": "ip", "target": "10.0.0.5", "port": 80, "weight": 5}, {"target": "web.internal", "port": 8080}]
 
@@ -153,6 +155,34 @@ def test_load_balancer_create_sends_json_backends_unchanged(gw):
     assert gw.last("POST", f"{LB}/l7").json == {"name": "web", "protocol": "https", "backends": BACKENDS}
 
 
+L7_RULES = [{"priority": 1, "path_prefix": "/api", "backends": BACKENDS[:1]}]
+
+
+@pytest.mark.parametrize(
+    "args, method, path, record, expected",
+    [
+        (["firewalls", "create", "web"], "POST", FW, group(), "fw-1"),
+        (["firewalls", "get", "fw-1"], "GET", f"{FW}/fw-1", group(), "fw-1"),
+        (["load-balancers", "create-l7", "web", "--backends", json.dumps(BACKENDS)], "POST", f"{LB}/l7",
+         lb(backends=BACKENDS, rules=L7_RULES), "lb-1"),
+        (["load-balancers", "create-l7", "web", "--backends", "[]"], "POST", f"{LB}/l7", lb(rules=[]), "lb-1"),
+        (["vpcs", "get", "vpc-1"], "GET", V, vpc(subnets=[subnet(), subnet(subnet_id="sub-2")]), "vpc-1"),
+    ],
+)
+def test_id_output_prints_the_object_id_not_its_nested_list(gw, args, method, path, record, expected):
+    gw.on(method, path, (200, record))
+
+    result = run(["-o", "id", *args])
+    assert result.exit_code == 0, result.output
+    assert result.stdout.splitlines() == [expected]
+
+
+def test_id_output_still_prints_one_id_per_collection_item():
+    groups = {"groups": [group(), group(firewall_group_id="fw-2")]}
+    assert identifiers(groups, "firewall_group_id") == ["fw-1", "fw-2"]
+    assert identifiers({"rules": group()["rules"]}) == ["sys-1", "r-1"]
+
+
 def test_load_balancer_repeatable_backend_and_routing(gw):
     gw.on("POST", f"{LB}/l4", (201, lb(layer="l4", protocol="tcp")))
 

{ibee_cli-0.5.0 → ibee_cli-0.5.2}/tests/test_vm_commands.py

@@ -57,16 +57,25 @@ def test_create_name_and_shape_are_sent_as_typed(gw) -> None:
     """No client name rule, no plan selectable check: the server answers."""
     catalog(gw, selectable=False, pricing_status="unpriced")
     gw.on("POST", "compute/cloud-vms", (422, {"detail": [{"loc": ["body", "name"], "msg": "invalid hostname"}]}))
-    result = run(["vms", "create", "Web_01!", "--plan-id", "p1", "--template-id", "t1"])
+    result = run(["vms", "create", "Web_01!", "--site-id", "s1", "--plan-id", "p1", "--template-id", "t1"])
     assert result.exit_code == 16
     assert gw.last("POST", "compute/cloud-vms").json["name"] == "Web_01!"
     assert "Error: HTTP 422" in plain(result)
 
 
+def test_create_requires_a_site(gw) -> None:
+    """Contract 0.4.1: site_id is required on VM create; nothing is sent without it."""
+    for group in ("vms", "gpus"):
+        result = run([group, "create", "web", "--plan-id", "p1", "--template-id", "t1"])
+        assert result.exit_code == 2
+        assert "--site-id" in plain(result)
+    assert gw.requests("POST") == []
+
+
 def test_count_creates_numbered_vms_with_numbered_keys(gw) -> None:
     catalog(gw)
     gw.on("POST", "compute/cloud-vms", (202, ACCEPTED))
-    result = run(["vms", "create", "web", "--plan-id", "p1", "--template-id", "t1", "--count", "3", "--idempotency-key", "k"])
+    result = run(["vms", "create", "web", "--site-id", "s1", "--plan-id", "p1", "--template-id", "t1", "--count", "3", "--idempotency-key", "k"])
     assert result.exit_code == 0, result.output
     creates = gw.requests("POST")
     assert [c.json["name"] for c in creates] == ["web-1", "web-2", "web-3"]
@@ -76,7 +85,7 @@ def test_count_creates_numbered_vms_with_numbered_keys(gw) -> None:
 def test_gpu_create_takes_the_gpu_shape_from_the_plan(gw) -> None:
     catalog(gw, vm_type="gpu", gpu_count=1, gpu_model="L4")
     gw.on("POST", "compute/gpu-vms", (202, ACCEPTED))
-    assert run(["gpus", "create", "g", "--plan-id", "p1", "--template-id", "t1"]).exit_code == 0
+    assert run(["gpus", "create", "g", "--site-id", "s1", "--plan-id", "p1", "--template-id", "t1"]).exit_code == 0
     body = gw.last("POST", "compute/gpu-vms").json
     assert (body["gpu_count"], body["gpu_model"]) == (1, "L4")
 
@@ -85,7 +94,7 @@ def test_create_with_wait_polls_the_operation(gw) -> None:
     catalog(gw)
     gw.on("POST", "compute/cloud-vms", (202, ACCEPTED))
     gw.on("GET", "compute/operations/op-1", (200, operation("running")), (200, operation("succeeded")))
-    result = run(["vms", "create", "web", "--plan-id", "p1", "--template-id", "t1", "--wait"])
+    result = run(["vms", "create", "web", "--site-id", "s1", "--plan-id", "p1", "--template-id", "t1", "--wait"])
     assert result.exit_code == 0, result.output
     assert "Create succeeded for web" in plain(result)