Commit Graph
2 Commits
Author SHA1 Message Date
Aleksei Sviridkin 1d938a8598 fix(ci): exempt priority/important-longterm from issue staling
Address review feedback from coderabbitai on .github/workflows/stale.yaml:37:
add priority/important-longterm to exempt-issue-labels. The label is
documented in labels.yml as "Important over the long term, but may
not be staffed and/or may need multiple releases to complete" — such
issues sit idle for 60+ days by design and must not be auto-staled.

Assisted-By: Claude <noreply@anthropic.com>
Signed-off-by: Aleksei Sviridkin <f@lex.la>
2026-04-27 18:40:34 +03:00
Aleksei Sviridkin bc6d147b85 chore(ci): replace Dosu stale and size labeling with native workflows
Add two GitHub-native workflows ahead of disabling the Dosu app and
migrate size labels to canonical k8s namespacing.

- .github/workflows/stale.yaml uses actions/stale@v10 to manage the
  lifecycle/stale and lifecycle/rotten lifecycle. Daily cron, 60d
  before stale, 14d before close. Issue/PR exempt lists differ on
  purpose: an accepted issue (triage/accepted) is a known long-tail
  task, not stale; a held PR (do-not-merge/hold) is paused
  intentionally. Priority/security/regression labels protect issues
  someone explicitly decided to keep visible.

- .github/workflows/pr-size.yaml uses actions/github-script@v7 to
  apply size/* labels based on diff line count. Thresholds match the
  descriptions in .github/labels.yml. Inline ignore patterns skip
  vendored deps, kubebuilder/openapi inline generated files
  (zz_generated.{deepcopy,conversion,defaults,openapi}.go), whole
  code-generated trees (any path under generated/ or *.pb.go),
  vendored Helm charts, and lockfiles. Concurrency group keyed on PR
  number with cancel-in-progress: true. removeLabel is wrapped in
  try/catch tolerating 404 for races between event payload and
  execution.

- Migrate the six size labels from legacy 'size:X' (colon) to
  canonical 'size/X' (slash) via aliases in .github/labels.yml. The
  colon form was an artefact of the previous sizing bot's output;
  with the bot replaced and namespacing already used everywhere else
  (kind/, area/, priority/, triage/, lifecycle/, do-not-merge/,
  security/), the slash form aligns with the rest of the scheme. The
  alias migration preserves IDs and keeps every existing labelled PR
  pointing at the renamed entry.

  pr-size.yaml emits the new 'size/' form from day 1 and accepts
  either prefix when looking up the previous size label, to handle
  the brief window between PR merge and the labels-sync run.

Verification of zero hard dependencies on Dosu in workflows: only
backport.yaml (consumes backport*, applied by humans only) and
pull-requests*.yaml + tags.yaml (consume release, applied by tags
workflow itself) — none of those labels come from Dosu.

The lgtm label (33/33 in last 100 PRs from Dosu, none from humans)
is intentionally not replaced in this PR — observe reviewer
behaviour for 7 days after Dosu is disabled and decide whether a
pull_request_review-on-approve workflow is needed.

Assisted-By: Claude <noreply@anthropic.com>
Signed-off-by: Aleksei Sviridkin <f@lex.la>
2026-04-27 18:00:01 +03:00