You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Unit tests red on main: history-jsonpath pin rejects the monitoring suite's bounded read #4750
go test ./internal/fluxcontract/ fails on main, so Unit & controller tests is red and E2E (in-tree) is skipped on every PR based on main since f209c47. TestChainsawSuitesReadHistoryThroughTheSharedName rejects hack/e2e-chainsaw/monitoring/chainsaw-test.yaml:
snapshot_shape_test.go:632: ../../hack/e2e-chainsaw/monitoring/chainsaw-test.yaml fills statuses, the local it hands the guard, in a shape that does not take its expression from HELMRELEASE_HISTORY_JSONPATH.
The monitoring suite does read through the shared name. Its read is statuses=$(timeout 30 kubectl ... -o jsonpath="$HELMRELEASE_HISTORY_JSONPATH") (line 240), and the pin only accepts statuses=$(kubectl, so the timeout 30 prefix is enough to fail it. The two changes merged two minutes apart on 2026-10-05: #4678 added the suite, then #4689 added the pin. Neither PR's CI ran against the other.
Fix: let the pin's pattern accept an optional timeout <n> before kubectl, which keeps the bounded read, or drop the prefix in the monitoring suite. The first one looks better to me, a ceiling on a status read is something other suites should be allowed to add.
Activity
fix(ci): reconcile three unit-test contracts broken by adjacent merges (