From c12d29e7a8d9da9a82d8cec664e7681f660c256d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20H=C3=B8ydahl?= Date: Tue, 10 Nov 2026 12:41:38 +0100 Subject: [PATCH] Correct the e2e CI note in the dependency-upgrade skill The e2e workflow is no longer disabled: since #883 it runs the "pr" labelled specs on every PR whose paths match, and a dependency sweep's paths do match. A green job is partial evidence rather than none. --- .agents/skills/dependency-upgrade/SKILL.md | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.agents/skills/dependency-upgrade/SKILL.md b/.agents/skills/dependency-upgrade/SKILL.md index 73ccee6d..b855dab9 100644 --- a/.agents/skills/dependency-upgrade/SKILL.md +++ b/.agents/skills/dependency-upgrade/SKILL.md @@ -253,9 +253,12 @@ Treat any failure as unfinished work. Add `make smoke-test` if packaging could b Docker, kind and network, unpacks the source release and runs `make check` inside it. CI coverage: build-and-check runs on every PR; smoke-test skips doc-only paths but runs for -`go.mod`, `go.sum`, `Makefile`, the license CSV and `helm/`. **The e2e workflow is permanently -disabled** and reports as skipped, so it is evidence of nothing — if runtime behaviour could be -affected, run `make e2e-tests` locally and say so in the PR. +`go.mod`, `go.sum`, `Makefile`, the license CSV and `helm/`. The e2e workflow runs on a sweep's PR +too — its path filter includes `go.mod`, `go.sum`, `Makefile`, `config/crd/`, +`config/dependencies/` and `helm/solr-operator/` — but only the `Label("pr")` specs, on one +Kubernetes and Solr version. That makes a green e2e job partial evidence: after a Kubernetes-set +bump, run the full suite with `make e2e-tests` locally, or dispatch `e2e-tests.yaml` with an empty +label filter, and say which one in the PR. ## Commit, then ask before pushing