quapp-hpc 0.0.1.dev11__tar.gz → 0.0.1.dev12__tar.gz

{quapp_hpc-0.0.1.dev11/quapp_hpc.egg-info → quapp_hpc-0.0.1.dev12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quapp-hpc
-Version: 0.0.1.dev11
+Version: 0.0.1.dev12
 Summary: Quapp HPC library — Slurm integration for Quapp Platform
 Author-email: "CITYNOW Co. Ltd." <corp@citynow.vn>
 License: The MIT License (MIT)

{quapp_hpc-0.0.1.dev11 → quapp_hpc-0.0.1.dev12}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "quapp-hpc"
-version = "0.0.1.dev11"
+version = "0.0.1.dev12"
 description = "Quapp HPC library — Slurm integration for Quapp Platform"
 readme = "README.md"
 authors = [{ name = "CITYNOW Co. Ltd.", email = "corp@citynow.vn" }]

{quapp_hpc-0.0.1.dev11 → quapp_hpc-0.0.1.dev12/quapp_hpc.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quapp-hpc
-Version: 0.0.1.dev11
+Version: 0.0.1.dev12
 Summary: Quapp HPC library — Slurm integration for Quapp Platform
 Author-email: "CITYNOW Co. Ltd." <corp@citynow.vn>
 License: The MIT License (MIT)

{quapp_hpc-0.0.1.dev11 → quapp_hpc-0.0.1.dev12}/quapp_slurm_hpc/log_writer.py

@@ -190,7 +190,18 @@ class JobLogWriter:
                 self._stopped = True
                 return None, None
 
-            payload = (response.json() or {}).get("data") or {}
+            try:
+                payload = (response.json() or {}).get("data") or {}
+            except ValueError as exc:
+                # A 200 with a non-JSON/empty body (proxy hiccup, truncated response) must not
+                # crash the caller — json.JSONDecodeError (modern requests) and the plain
+                # ValueError older requests/simplejson raise (e.g. requests 2.18 on the DGX
+                # station's Python 3.6) are both ValueError subclasses, so this catches either.
+                print(f"[log_writer] stopping log stream, could not parse response: {exc}",
+                      file=sys.stderr, flush=True)
+                self._stopped = True
+                return None, None
+
             outcome = payload.get("outcome")
             if outcome in _STOP_OUTCOMES:
                 self._stopped = True

{quapp_hpc-0.0.1.dev11 → quapp_hpc-0.0.1.dev12}/tests/test_log_writer.py

@@ -181,6 +181,22 @@ def test_401_stops_immediately_without_retrying():
     assert writer.stopped
 
 
+def test_non_json_200_response_stops_without_crashing():
+    # Regression: a 200 whose body isn't valid JSON (proxy hiccup, truncated response, or the
+    # ValueError requests 2.18/simplejson raises on Python 3.6 — the DGX station's interpreter)
+    # must be swallowed like any other failure here, not propagate out of append()/open()/close().
+    writer = JobLogWriter("http://backend/jobs/j1/logs", "tok", 1)
+    with patch.object(log_writer.requests, "post") as mock_post:
+        resp = MagicMock()
+        resp.status_code = 200
+        resp.raise_for_status.side_effect = None
+        resp.json.side_effect = ValueError("not JSON")
+        mock_post.return_value = resp
+        writer.append(b"hello\n")  # must not raise
+    assert mock_post.call_count == 1
+    assert writer.stopped
+
+
 @pytest.mark.parametrize("outcome", ["SEALED", "TOO_LATE", "TRUNCATED"])
 def test_stop_outcomes_halt_further_sends(outcome):
     writer = JobLogWriter("http://backend/jobs/j1/logs", "tok", 1)