Skip to content

(MOT-4367) feat(ci): expose manual test pipelines - #725

Merged
ytallo merged 1 commit into
mainfrom
feat/manual-test-pipelines
Aug 6, 2026
Merged

(MOT-4367) feat(ci): expose manual test pipelines#725
ytallo merged 1 commit into
mainfrom
feat/manual-test-pipelines

Conversation

@ytallo

@ytallo ytallo commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What changed

  • add deterministic run titles and Release Control identity inputs to approved E2E workflows
  • allow manual runs to check out an immutable source commit
  • isolate concurrency by operation or source identity
  • extend Harness source and quickstart workflows with operation and step correlation

Why

Release Control needs a safe, allowlisted way to dispatch tests without exposing release or promotion workflows and without correlating runs by timing.

Impact

Existing pull request, scheduled, and reusable workflow behavior remains unchanged. Manual executions can now be tracked deterministically by Release Control.

Validation

  • parsed all six changed workflow files as YAML
  • actionlint on all six changed workflows
  • git diff --check

Refs MOT-4367

Summary by CodeRabbit

  • Enhancements
    • Added optional source revision, operation, and step inputs to supported manually triggered test workflows.
    • Workflow runs now display clearer operation-based names.
    • Improved concurrency handling to isolate runs by operation or source revision.
    • Workflows can now test a specified source revision when provided.

@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
workers Ready Ready Preview Aug 6, 2026 1:03pm
workers-tech-spec Ready Ready Preview Aug 6, 2026 1:03pm

Request Review

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7cee03b5-a6e0-45e8-8325-b65b88593216

📥 Commits

Reviewing files that changed from the base of the PR and between b62a368 and 0958bed.

📒 Files selected for processing (6)
  • .github/workflows/database-e2e.yml
  • .github/workflows/harness-e2e-daily.yml
  • .github/workflows/harness-quickstart.yml
  • .github/workflows/rbac-proxy-e2e.yml
  • .github/workflows/shell-e2e.yml
  • .github/workflows/storage-e2e.yml

📝 Walkthrough

Walkthrough

Six E2E workflows now accept execution metadata, use it for run names and concurrency groups, and optionally check out a specified source SHA.

Changes

E2E workflow identity

Layer / File(s) Summary
Workflow inputs and run names
.github/workflows/database-e2e.yml, .github/workflows/harness-e2e-daily.yml, .github/workflows/harness-quickstart.yml, .github/workflows/rbac-proxy-e2e.yml, .github/workflows/shell-e2e.yml, .github/workflows/storage-e2e.yml
The workflows accept source SHA, operation ID, and step ID inputs where supported. Run names include the operation ID or event name.
Operation-aware concurrency
.github/workflows/database-e2e.yml, .github/workflows/harness-e2e-daily.yml, .github/workflows/harness-quickstart.yml, .github/workflows/rbac-proxy-e2e.yml, .github/workflows/shell-e2e.yml, .github/workflows/storage-e2e.yml
Concurrency groups prefer the operation ID, then source SHA or workflow-specific fallback values, before the Git ref or event identity.
Source SHA checkout
.github/workflows/database-e2e.yml, .github/workflows/harness-quickstart.yml, .github/workflows/rbac-proxy-e2e.yml, .github/workflows/shell-e2e.yml, .github/workflows/storage-e2e.yml
Checkout uses the supplied source SHA when available. It otherwise uses the triggering commit SHA.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested reviewers: andersonleal

Poem

A rabbit checks the SHA with care,
Names each run in open air.
Operation paths align,
Concurrency groups now bind.
“Hop,” says Bun, “the workflows flow!”

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/manual-test-pipelines

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 54 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@ytallo
ytallo marked this pull request as ready for review August 6, 2026 13:09
@ytallo
ytallo merged commit b35ee10 into main Aug 6, 2026
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant