End-to-end tests on a local model, and in CI on every PR
- Every e2e role runs on qwen3.8-27b on the RTX 4090 (e2e_roles(); NISSE_E2E_MODEL overrides, e.g. claude-haiku-4-5). The first full run passed 7 of 8 in 26 minutes for $0.0000; the eighth was a real finding, fixed with the new-project PR. - scripts/e2e.sh reports a run's spend on the test key and the day's total against its budget: above $0 on the local model means a paid fallback. - CI runs the suite as its own job on pull requests (not pushes to main), on a throwaway sshd (--local), cancelling a run a newer push supersedes, so the checks job still reports in minutes. - The plan says so: e2e on every PR; locally only -k / --lf. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPQ14L3LikyefCMFBqEbKE
This commit is contained in:
1 parent
b3bf6d5a6e
commit
6db80dfb4b
6 files changed
+91
-9
No files matched your search
@@ -54,3 +54,44 @@ jobs:
|
|||||||
# key) and deletes it afterwards.
|
# key) and deletes it afterwards.
|
||||||
NISSE_OPERATOR_TOKEN: ${{ secrets.NISSE_OPERATOR_TOKEN }}
|
NISSE_OPERATOR_TOKEN: ${{ secrets.NISSE_OPERATOR_TOKEN }}
|
||||||
run: uv run pytest -m "integration and not e2e" -rs
|
run: uv run pytest -m "integration and not e2e" -rs
|
||||||
|
|
||||||
|
e2e:
|
||||||
|
# The end-to-end suite (scripts/e2e.sh): the whole agent against real
|
||||||
|
# services, every role on a local model, so it spends nothing but keeps
|
||||||
|
# the RTX 4090 busy for ~25 minutes. On PRs only, as its own job so the
|
||||||
|
# checks above report first; a newer push to the same PR cancels the
|
||||||
|
# run it supersedes. Locally, run only what you're writing (-k) or
|
||||||
|
# diagnosing (--lf).
|
||||||
|
if: github.event_name != 'push'
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 90
|
||||||
|
concurrency:
|
||||||
|
group: e2e-${{ github.head_ref || github.ref }}
|
||||||
|
cancel-in-progress: true
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Install uv
|
||||||
|
run: |
|
||||||
|
curl -LsSf https://astral.sh/uv/install.sh | sh
|
||||||
|
echo "$HOME/.local/bin" >> "$GITHUB_PATH"
|
||||||
|
|
||||||
|
- name: Sync workspace
|
||||||
|
run: uv sync --all-packages
|
||||||
|
|
||||||
|
- name: Install sshd
|
||||||
|
# The build VM is a throwaway sshd here (scripts/e2e.sh --local).
|
||||||
|
run: |
|
||||||
|
SUDO=$(command -v sudo || true)
|
||||||
|
$SUDO apt-get update -qq
|
||||||
|
$SUDO apt-get install -y -qq openssh-server openssh-client > /dev/null
|
||||||
|
$SUDO mkdir -p /run/sshd
|
||||||
|
|
||||||
|
- name: End-to-end tests
|
||||||
|
env:
|
||||||
|
NISSE_LLM_TOKEN: ${{ secrets.NISSE_LLM_TOKEN }}
|
||||||
|
NISSE_MATRIX_TOKEN: ${{ secrets.NISSE_MATRIX_TOKEN }}
|
||||||
|
NISSE_GITEA_TOKEN: ${{ secrets.NISSE_GITEA_TOKEN }}
|
||||||
|
NISSE_OPERATOR_TOKEN: ${{ secrets.NISSE_OPERATOR_TOKEN }}
|
||||||
|
NISSE_E2E_RUNS: /tmp/nisse-e2e-runs
|
||||||
|
run: scripts/e2e.sh --local -v
|
||||||
@@ -73,9 +73,11 @@ Only `nisse-dev` has real code; `nisse-core` and `nisse-ops` are placeholders.
|
|||||||
`CLAUDE.md`. A milestone marked **needs discussion** starts with a design
|
`CLAUDE.md`. A milestone marked **needs discussion** starts with a design
|
||||||
discussion first.
|
discussion first.
|
||||||
- **Every change comes with tests,** through `scripts/check.sh`.
|
- **Every change comes with tests,** through `scripts/check.sh`.
|
||||||
- **The end-to-end test follows each milestone** (M1.6): the real agent, real models,
|
- **The end-to-end test runs on every PR** (M1.6): the real agent, every role on a
|
||||||
a repository of the run's own and a real build host, driven by a scripted operator. It runs
|
local model (so it costs nothing per call), a repository of the run's own and a
|
||||||
after each milestone, not on every push, since it spends model tokens. Manual
|
real build host, driven by a scripted operator. CI runs it as its own job on pull
|
||||||
|
requests, cancelling a run a newer push supersedes. Locally only the tests being
|
||||||
|
written (`-k`) or diagnosed (`--lf`); `scripts/e2e.sh` reports what a run spent. Manual
|
||||||
live runs are for the new loop's first runs and phase ends. Findings are filed as
|
live runs are for the new loop's first runs and phase ends. Findings are filed as
|
||||||
issues and either fixed before moving on or deliberately deferred.
|
issues and either fixed before moving on or deliberately deferred.
|
||||||
- **The Quint models move with the loop:** `orchestrator.qnt` while `control.py`
|
- **The Quint models move with the loop:** `orchestrator.qnt` while `control.py`
|
||||||
|
|||||||
+30
-4
@@ -1,13 +1,16 @@
|
|||||||
#!/usr/bin/env bash
|
#!/usr/bin/env bash
|
||||||
# The end-to-end test (Development Plan M1.6): the whole agent against real
|
# The end-to-end test (Development Plan M1.6): the whole agent against real
|
||||||
# services, driven by a scripted operator. Run after each milestone; it
|
# services, driven by a scripted operator. CI runs it on every PR (with
|
||||||
# spends model tokens and takes minutes, so it is not part of check.sh or CI.
|
# --local); locally, run only the tests you're writing (-k) or diagnosing
|
||||||
|
# (--lf).
|
||||||
#
|
#
|
||||||
# scripts/e2e.sh on the bring-up build VM (configured below)
|
# scripts/e2e.sh on the bring-up build VM (configured below)
|
||||||
# scripts/e2e.sh --local on a throwaway local sshd instead
|
# scripts/e2e.sh --local on a throwaway local sshd instead
|
||||||
#
|
#
|
||||||
# Uses tests/integration/nisse-test.yaml and its secrets (.env at the repo
|
# Uses tests/integration/nisse-test.yaml and its secrets (.env at the repo
|
||||||
# root), like the integration tests.
|
# root), like the integration tests. Every role runs on a local model
|
||||||
|
# (tests/integration/e2e_support.py; NISSE_E2E_MODEL overrides it), so tell
|
||||||
|
# the operator before a full run: it keeps the RTX 4090 busy for a while.
|
||||||
set -euo pipefail
|
set -euo pipefail
|
||||||
cd "$(dirname "$0")/.."
|
cd "$(dirname "$0")/.."
|
||||||
|
|
||||||
@@ -19,4 +22,27 @@ else
|
|||||||
fi
|
fi
|
||||||
# Each run's state store is kept here, for nisse_dev.dump after a failure.
|
# Each run's state store is kept here, for nisse_dev.dump after a failure.
|
||||||
export NISSE_E2E_RUNS=${NISSE_E2E_RUNS:-/root/nisse-e2e/test-runs}
|
export NISSE_E2E_RUNS=${NISSE_E2E_RUNS:-/root/nisse-e2e/test-runs}
|
||||||
exec uv run pytest -m e2e -rs tests/integration/test_end_to_end.py tests/integration/test_end_to_end_loop.py "$@"
|
|
||||||
|
# What the run spent on the test key, read from LiteLLM (spend only). The
|
||||||
|
# end-to-end tests run on a local model, so anything above $0 means a call
|
||||||
|
# went to a paid one: an override, or a proxy fallback.
|
||||||
|
spend() {
|
||||||
|
(set -a; [ -f .env ] && . ./.env; set +a
|
||||||
|
uv run python -c '
|
||||||
|
import os, httpx
|
||||||
|
r = httpx.get("https://litellm.dongar.net/key/info", timeout=30,
|
||||||
|
headers={"Authorization": "Bearer " + os.environ["NISSE_LLM_TOKEN"]})
|
||||||
|
i = r.json()["info"]
|
||||||
|
print(i["spend"], i.get("max_budget"))' 2>/dev/null) || echo "? ?"
|
||||||
|
}
|
||||||
|
read -r before budget < <(spend)
|
||||||
|
status=0
|
||||||
|
uv run pytest -m e2e -rs tests/integration/test_end_to_end.py tests/integration/test_end_to_end_loop.py "$@" || status=$?
|
||||||
|
read -r after _ < <(spend)
|
||||||
|
python3 -c "
|
||||||
|
b, a, cap = '$before', '$after', '$budget'
|
||||||
|
try:
|
||||||
|
print(f'e2e spend: \${float(a) - float(b):.4f} this run; \${float(a):.2f} of \${float(cap):.2f} today')
|
||||||
|
except ValueError:
|
||||||
|
print('e2e spend: unknown (could not read the key)')"
|
||||||
|
exit "$status"
|
||||||
@@ -11,6 +11,17 @@ from dataclasses import dataclass
|
|||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
|
|
||||||
from nisse_dev.gitea import GiteaClient, GiteaError
|
from nisse_dev.gitea import GiteaClient, GiteaError
|
||||||
|
from nisse_dev.llm import RoleMap
|
||||||
|
|
||||||
|
# The end-to-end tests run every role on a local model, which costs nothing
|
||||||
|
# per call; NISSE_E2E_MODEL overrides it (e.g. claude-haiku-4-5 for a quick,
|
||||||
|
# paid run).
|
||||||
|
E2E_MODEL = "qwen3.8-27b"
|
||||||
|
|
||||||
|
|
||||||
|
def e2e_roles() -> RoleMap:
|
||||||
|
model = os.environ.get("NISSE_E2E_MODEL", "").strip() or E2E_MODEL
|
||||||
|
return RoleMap(executor=model, planner=model, helper=model, expert=model)
|
||||||
|
|
||||||
|
|
||||||
def task(run: str) -> str:
|
def task(run: str) -> str:
|
||||||
|
|||||||
@@ -25,7 +25,7 @@ from pathlib import Path
|
|||||||
from typing import Any
|
from typing import Any
|
||||||
|
|
||||||
import pytest
|
import pytest
|
||||||
from e2e_support import fresh_repo, operator_gitea
|
from e2e_support import e2e_roles, fresh_repo, operator_gitea
|
||||||
from e2e_support import task as make_task
|
from e2e_support import task as make_task
|
||||||
|
|
||||||
from nisse_dev.agent import Agent, build_agent
|
from nisse_dev.agent import Agent, build_agent
|
||||||
@@ -84,6 +84,7 @@ def agent(
|
|||||||
workspace=Path(f"/tmp/nisse-e2e-{run}/sandbox"),
|
workspace=Path(f"/tmp/nisse-e2e-{run}/sandbox"),
|
||||||
build_host=build_host(ssh_host),
|
build_host=build_host(ssh_host),
|
||||||
),
|
),
|
||||||
|
roles=e2e_roles(),
|
||||||
state_db=tmp_path / "state.db",
|
state_db=tmp_path / "state.db",
|
||||||
)
|
)
|
||||||
transport = InMemoryTransport()
|
transport = InMemoryTransport()
|
||||||
|
|||||||
@@ -21,7 +21,7 @@ from pathlib import Path
|
|||||||
from typing import Any
|
from typing import Any
|
||||||
|
|
||||||
import pytest
|
import pytest
|
||||||
from e2e_support import dependent_task, fresh_repo, operator_gitea, task
|
from e2e_support import dependent_task, e2e_roles, fresh_repo, operator_gitea, task
|
||||||
|
|
||||||
from nisse_dev.config import AgentConfig, LoopTuning
|
from nisse_dev.config import AgentConfig, LoopTuning
|
||||||
from nisse_dev.execution import SSHHost
|
from nisse_dev.execution import SSHHost
|
||||||
@@ -74,6 +74,7 @@ def running(
|
|||||||
build_host=build_host(ssh_host),
|
build_host=build_host(ssh_host),
|
||||||
),
|
),
|
||||||
loop="new",
|
loop="new",
|
||||||
|
roles=e2e_roles(),
|
||||||
state_db=keep / "state.db",
|
state_db=keep / "state.db",
|
||||||
# Short, so a bug's grace period runs out within the test.
|
# Short, so a bug's grace period runs out within the test.
|
||||||
tuning=replace(live_config.tuning, loop=LoopTuning(bug_grace_s=BUG_GRACE_S)),
|
tuning=replace(live_config.tuning, loop=LoopTuning(bug_grace_s=BUG_GRACE_S)),
|
||||||
|
|||||||
Reference in new issue
Block a user