ellf-cli 10.0.2__tar.gz → 11.0.1__tar.gz

{ellf_cli-10.0.2/ellf_cli.egg-info → ellf_cli-11.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ellf-cli
-Version: 10.0.2
+Version: 11.0.1
 Summary: Ellf Command Line Interface
 Home-page: https://prodi.gy
 Author: ExplosionAI GmbH

{ellf_cli-10.0.2 → ellf_cli-11.0.1}/ellf_cli/about.json

@@ -1,7 +1,7 @@
 {
   "title": "Ellf CLI",
   "name": "ellf-cli",
-  "version": "10.0.2",
+  "version": "11.0.1",
   "summary": "Ellf Command Line Interface",
   "uri": "https://prodi.gy",
   "prog": "ellf",

{ellf_cli-10.0.2 → ellf_cli-11.0.1}/ellf_cli/ellf.json

@@ -1,7 +1,7 @@
 {
   "prog": "ellf",
   "help": "Ellf Command Line Interface.",
-  "version": "6.1.5",
+  "version": "11.0.0",
   "extra_key": "_extra",
   "commands": {
     "actions": {
@@ -356,11 +356,12 @@
             "api",
             "cluster",
             "id",
-            "ci"
+            "ci",
+            "ellf-core"
           ],
           "has_converter": false,
           "type": "str",
-          "orig_type": "Literal[api, cluster, id, ci]"
+          "orig_type": "Literal[api, cluster, id, ci, ellf-core]"
         }
       ],
       "description": "\nReturn an auth token for the current user\n",

{ellf_cli-10.0.2 → ellf_cli-11.0.1}/ellf_cli/query.py

@@ -826,7 +826,11 @@ def delete_job(job: _JobT, auth: AuthState, quiet: bool = False) -> _JobT:
 
 
 def check_job_started(
-    job: _JobT, auth: AuthState, *, wait_seconds: int = 60 * 2
+    job: _JobT,
+    auth: AuthState,
+    *,
+    wait_seconds: int = 60 * 2,
+    not_found_grace_seconds: int = 10,
 ) -> _JobT | None:
     job_type = _get_job_type(job)
     started_states = {
@@ -846,7 +850,7 @@ def check_job_started(
     from .ui import loading
 
     with loading(Messages.T034.format(noun=job_type)):
-        for _ in range(wait_seconds):
+        for attempt in range(wait_seconds):
             try:
                 status = auth.broker_client.jobs.get_status(job.id)
             except (httpx.HTTPError, BrokerError):
@@ -858,6 +862,16 @@ def check_job_started(
                 and getattr(status, "health", None) not in failed_healths
             ):
                 break
+            if (
+                status.state == broker_models.WorkloadState.not_found
+                and attempt < not_found_grace_seconds
+                and getattr(status, "health", None) not in failed_healths
+            ):
+                # A just-submitted job can report not_found before its
+                # resources appear in the cluster listing, so an early
+                # not_found is not a failure: keep polling.
+                time.sleep(1)
+                continue
             if status.state in failed_states or (
                 getattr(status, "health", None) in failed_healths
             ):

{ellf_cli-10.0.2 → ellf_cli-11.0.1/ellf_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ellf-cli
-Version: 10.0.2
+Version: 11.0.1
 Summary: Ellf Command Line Interface
 Home-page: https://prodi.gy
 Author: ExplosionAI GmbH

{ellf_cli-10.0.2 → ellf_cli-11.0.1}/tests/test_query.py

@@ -8,7 +8,9 @@ from uuid import UUID, uuid4
 
 import pytest
 
+from ellf_broker_sdk import models as broker_models
 from ellf_cli import query
+from ellf_cli.errors import CLIError
 from ellf_cli.query import _normalise_scope_params, get_not_found_error
 from ellf_pam_sdk.errors import ClusterNotFound, ProjectNotFound
 
@@ -119,3 +121,104 @@ class TestNormaliseScopeParams:
     def test_does_not_touch_unrelated_keys(self, fake_scope_resolution):
         out = _normalise_scope_params({"name": "foo", "after": "cursor-1"})
         assert out == {"name": "foo", "after": "cursor-1"}
+
+
+# ---------------------------------------------------------------------------
+# check_job_started: a just-submitted job can report ``not_found`` before its
+# resources appear in the cluster listing (the broker derives ``not_found``
+# from an empty label-selector list). The first polls must therefore tolerate
+# ``not_found`` instead of treating it as a terminal failure -- this race
+# caused freshly created actions to be reported as failed-to-start (and, before
+# the JobStartRejectedError split, rolled back and deleted) while the workload
+# was in fact running.
+
+
+class _FakeJobsEndpoint:
+    """Broker jobs stub returning a scripted sequence of statuses."""
+
+    def __init__(self, statuses: list[Any]) -> None:
+        self._statuses = statuses
+        self.calls = 0
+
+    def get_status(self, job_id: UUID) -> Any:
+        index = min(self.calls, len(self._statuses) - 1)
+        self.calls += 1
+        return self._statuses[index]
+
+
+class TestCheckJobStarted:
+    def _status(
+        self,
+        state: broker_models.WorkloadState,
+        health: broker_models.WorkloadHealth,
+        *,
+        is_ready: bool = False,
+    ) -> broker_models.JobStatusResponse:
+        return broker_models.JobStatusResponse(
+            job_id=uuid4(), state=state, health=health, is_ready=is_ready
+        )
+
+    def _run(
+        self,
+        statuses: list[broker_models.JobStatusResponse],
+        monkeypatch: pytest.MonkeyPatch,
+        **kwargs: Any,
+    ) -> tuple[Any, _FakeJobsEndpoint]:
+        monkeypatch.setattr(query.time, "sleep", lambda seconds: None)
+        jobs = _FakeJobsEndpoint(statuses)
+        auth = SimpleNamespace(broker_client=SimpleNamespace(jobs=jobs))
+        job = SimpleNamespace(id=uuid4(), url_logs="https://example.com/logs")
+        result = query.check_job_started(job, auth, **kwargs)  # type: ignore[arg-type]
+        return result, jobs
+
+    def test_not_found_within_grace_then_running(
+        self, monkeypatch: pytest.MonkeyPatch
+    ) -> None:
+        statuses = [
+            self._status(
+                broker_models.WorkloadState.not_found,
+                broker_models.WorkloadHealth.inactive,
+            )
+        ] * 3 + [
+            self._status(
+                broker_models.WorkloadState.running,
+                broker_models.WorkloadHealth.progressing,
+            )
+        ]
+        result, jobs = self._run(statuses, monkeypatch)
+        assert result is not None
+        assert jobs.calls == 4
+
+    def test_not_found_past_grace_raises(self, monkeypatch: pytest.MonkeyPatch) -> None:
+        statuses = [
+            self._status(
+                broker_models.WorkloadState.not_found,
+                broker_models.WorkloadHealth.inactive,
+            )
+        ]
+        with pytest.raises(CLIError):
+            self._run(statuses, monkeypatch, not_found_grace_seconds=5)
+        # Polls through the grace window, then fails on the next poll.
+
+    def test_failed_state_raises_without_grace(
+        self, monkeypatch: pytest.MonkeyPatch
+    ) -> None:
+        statuses = [
+            self._status(
+                broker_models.WorkloadState.failed,
+                broker_models.WorkloadHealth.terminal,
+            )
+        ]
+        with pytest.raises(CLIError):
+            self._run(statuses, monkeypatch)
+
+    def test_succeeded_returns_job(self, monkeypatch: pytest.MonkeyPatch) -> None:
+        statuses = [
+            self._status(
+                broker_models.WorkloadState.succeeded,
+                broker_models.WorkloadHealth.inactive,
+            )
+        ]
+        result, jobs = self._run(statuses, monkeypatch)
+        assert result is not None
+        assert jobs.calls == 1