Skip to content

Prevent and sweep leaked E2E resources - #8331

Draft
isaacroldan wants to merge 1 commit into
mainfrom
isaac/e2e-leaked-resource-sweep
Draft

Prevent and sweep leaked E2E resources#8331
isaacroldan wants to merge 1 commit into
mainfrom
isaac/e2e-leaked-resource-sweep

Conversation

@isaacroldan

Copy link
Copy Markdown
Contributor

Why

Failed E2E runs can leave apps and development stores behind. The PR cleanup job did not fail when resources remained, and there was no scheduled cleanup for older leaks.

What

  • Restrict cleanup to verified current or legacy E2E resource names.
  • Add age-bounded cleanup and verification modes to the app and store scripts.
  • Fail PR cleanup when current-run resources remain.
  • Add a daily sweep for E2E resources older than 24 hours.

Testing

  • pnpm --filter e2e lint
  • pnpm --filter e2e type-check
  • pnpm --filter e2e exec playwright test tests/resource-ownership.spec.ts --project local
  • pnpm --filter e2e exec playwright test --project local
  • Parsed both changed workflow files as YAML.

Assisted-By: devx/215513a9-13fa-4e74-bb8f-79faee6e4f09
@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant