testing: 89 of 165 packages have no helm test target, including eight user-facing apps #3631

Activity

1. Aleksei Sviridkin (lexfrei)

   added

   good first issueGood for newcomersGood for newcomers

   communityCommunity contributions are welcome in this issueCommunity contributions are welcome in this issue

   kind/cleanupCategorizes issue or PR as related to cleanup of code, process, or technical debtCategorizes issue or PR as related to cleanup of code, process, or technical debt

   area/testingIssues or PRs related to testing (e2e, bats, unit tests)Issues or PRs related to testing (e2e, bats, unit tests)

   on Aug 7, 2026
2. github-actions

   added

   triage/needs-triageIndicates an issue needs triage by a maintainerIndicates an issue needs triage by a maintainer

   on Aug 12, 2026
3. lexfrei commented on Sep 10, 2026

   

   Aleksei Sviridkin (lexfrei)

   on Sep 10, 2026

   ContributorAuthor

   More actions

   Three of the 89 carry an extra hazard worth using to order the work. Their update recipe clears their own templates/ before refetching, so a first-party file added there is deleted by the next make update, which exits 0. With no suite in the package, nothing downstream reddens and the loss reaches a cluster.

   Measured at origin/main, f7cb9e785:

   REF=origin/main
   for mk in $(git ls-tree -r --name-only "$REF" -- packages | grep -E '^packages/[^/]+/[^/]+/Makefile$'); do
     body=$(git show "$REF:$mk")
     w=$(printf '%s\n' "$body" | awk '/^update:/ {r=1; next}
          r && /^\t[[:space:]]*@?#/ {next}
          r && /^\t/ {print; next}
          r && /^[[:space:]]*$/ {next}
          r && /^[[:space:]]*#/ {next}
          r {exit}' | grep -cE 'rm[[:space:]]+-(rf|fr|r)[[:space:]]+(\./)?templates(/\*?)?([[:space:]]|;|$)')
     [ "$w" -gt 0 ] || continue
     printf '%s\n' "$body" | grep -qE '^test:' || echo "$mk"
   done

   Nine packages clear their own templates/ at that ref. Three of them have no test target: kubevirt-cdi-operator, kubevirt-instancetypes, objectstorage-controller.

   The ref is in the command because the count moves with the branch you run it on. kubevirt-cdi is the ninth wiper at origin/main and stops being one when #4097 lands, so a run inside a checkout of that branch gives eight.

   Not a separate issue, and not the same as #4206 or #4119: those are about a guard that cannot see a loss, this is about a package that has no suite to report one. It just says which three of the 89 to do first.
4. IvanHunters

   added a commit that references this issue on Sep 22, 2026

   test(vpn): add default render helm unit tests (#4347)

   ...

   ff0b851
5. Aleksei Sviridkin (lexfrei)

   added

   triage/acceptedIndicates an issue is ready to be actively worked onIndicates an issue is ready to be actively worked on

   priority/backlogGeneral backlog priority. Lower than priority/important-longtermGeneral backlog priority. Lower than priority/important-longterm

   and removed

   triage/needs-triageIndicates an issue needs triage by a maintainerIndicates an issue needs triage by a maintainer

   on Sep 23, 2026