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
ci: a per-suite edit under a directory with no chainsaw-test.yaml is swallowed the same way #3330 was #3710
hack/select-e2e.sh carries a second copy of the #3330 swallowing, in the branch that handles per-suite edits rather than the one that walks the dependency graph.
A path under hack/e2e-chainsaw/<dir>/ puts <dir> into selected_apps, taken from the directory name and without checking that the directory holds a chainsaw-test.yaml. A name that is not a suite is then dropped by the final intersection, so what makes such an edit escalate is the backstop at the bottom of the script, and that only fires when the whole selection came back empty. The escalation therefore survives exactly as long as nothing else in the diff contributes a suite name, which is the shape #3330 described on the graph side.
Reproduction, identical at the merge base and at current main:
Alone the path runs everything; add one covered package and it runs one suite.
The shape this bites is someone adding shared material next to _lib/, a fixtures directory or a helper the suites source, in the same pull request as an app change. Nothing covering that material runs, and the output reads as an ordinary narrow selection rather than as something that was dropped.
#3441 closes the graph side (coverage decided per changed path) and deliberately leaves this one, since it is pre-existing and outside that change. Its test an edit to a non-suite directory under e2e-chainsaw escalates pins the single-path case only, so the combined shape above is unpinned today.
One fix that fits the existing structure: check the directory against the discovered suite list at classification time and escalate there, the way the graph branch now escalates per path, instead of relying on the empty-selection backstop.
Activity
fix(ci): close e2e suite selection gaps in select-e2e and select-inst…