fix: add is_remote field, fix workflow URL check, remove dead code

- Add `is_remote: bool = False` to IrisIntegrationCluster dataclass
- Use local `$URL` variable instead of `$IRIS_CONTROLLER_URL` in same-step check
- Remove unused `validate_job_context` from jobs.py

Author: rjpower

.github/workflows/iris-integration.yaml

@@ -59,7 +59,7 @@ jobs:
             sleep 1
           done
 
-          if [ -z "${IRIS_CONTROLLER_URL:-}" ]; then
+          if [ -z "${URL:-}" ]; then
             echo "Cluster failed to start within timeout"
             cat /tmp/iris-cluster.log
             exit 1

tests/integration/iris/cluster.py

@@ -33,6 +33,7 @@ class IrisIntegrationCluster:
     url: str
     client: IrisClient
     controller_client: ControllerServiceClientSync
+    is_remote: bool = False
     job_timeout: float = 60.0
 
     def submit(

tests/integration/iris/jobs.py

@@ -97,17 +97,3 @@ def validate_ports():
         raise ValueError(f"Ports not set: {info.ports}")
     assert info.ports["http"] > 0
     assert info.ports["grpc"] > 0
-
-
-def validate_job_context():
-    """Validate job context via get_job_info() in a coscheduled job."""
-    import logging
-
-    from iris.cluster.client import get_job_info
-
-    logger = logging.getLogger("iris.test.context")
-    info = get_job_info()
-    if info is None:
-        raise RuntimeError("Not running in an Iris job context")
-    logger.info(f"Task {info.task_index} of {info.num_tasks} on worker {info.worker_id}")
-    return f"Task {info.task_index} done"