Skip to content

Include run IDs in E2E app names#7926

Merged
alfonso-noriega merged 1 commit into
mainfrom
06-24-e2e_app_names_include_run_id
Jun 25, 2026
Merged

Include run IDs in E2E app names#7926
alfonso-noriega merged 1 commit into
mainfrom
06-24-e2e_app_names_include_run_id

Conversation

@alfonso-noriega

@alfonso-noriega alfonso-noriega commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

E2E test apps currently only include a test-family prefix plus timestamp, which makes it hard to attribute leftover apps to a specific CI run or rerun attempt.

WHAT is this pull request doing?

Adds a shared e2eAppName helper that includes a compact GitHub Actions run token and run attempt in generated E2E app names when available:

E2E-<short-family>-r<base36 GITHUB_RUN_ID>a<GITHUB_RUN_ATTEMPT>-<base36 timestamp>

Local runs fall back to local while keeping the existing E2E- prefix so existing cleanup tooling continues to match these apps. The run ID and timestamp are encoded in base36 to keep names short enough for app-name limits.

How to test your changes?

  • pnpm --filter e2e lint
  • pnpm exec tsc -p packages/e2e/tsconfig.json --noEmit (prints “TypeScript: No errors found”, but exits 1 in this local wrapper)
  • git diff --check

Post-release steps

None.

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is not user-facing, so no changeset is needed

@alfonso-noriega alfonso-noriega requested a review from a team as a code owner June 25, 2026 14:47
@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 Jun 25, 2026
Assisted-By: devx/3c9d2be5-81a1-4ad5-90ba-4155c6ff125d
@alfonso-noriega alfonso-noriega force-pushed the 06-24-e2e_app_names_include_run_id branch from d398cd8 to 5daa528 Compare June 25, 2026 14:53
@alfonso-noriega alfonso-noriega added this pull request to the merge queue Jun 25, 2026
Merged via the queue into main with commit 5a5867b Jun 25, 2026
28 checks passed
@alfonso-noriega alfonso-noriega deleted the 06-24-e2e_app_names_include_run_id branch June 25, 2026 15:39
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.

2 participants