Skip to content

feat(PLT-3267): skip redundant tests on deploy when verified on PR#247

Draft
skridlevsky wants to merge 1 commit into
mainfrom
PLT-3267-skip-verified-tests-on-deploy
Draft

feat(PLT-3267): skip redundant tests on deploy when verified on PR#247
skridlevsky wants to merge 1 commit into
mainfrom
PLT-3267-skip-verified-tests-on-deploy

Conversation

@skridlevsky

@skridlevsky skridlevsky commented Jun 30, 2026

Copy link
Copy Markdown

What

Skips redundant test suites on the main/deploy run when the same content already passed on the PR. Adds a skip-verified-tests input (default true).

Why

The deploy workflow re-runs the exact suites that already passed on the PR, gating production deploys on a second run that produces no new signal.

How

  • PR workflow (record-verification): after all enabled suites pass, pushes a marker ref refs/tests-passed/<tree> keyed on the git tree-hash of the merge ref.
  • Deploy workflow (check-verification): computes the merged commit's tree-hash, looks up the marker, and skips the redundant suites when it matches.
  • Works because repos are squash-merge + strict branch protection (up-to-date required), so the PR's merge-ref tree equals the squashed commit's tree on main.

Safety

  • Fail-safe, never fail-open: missing or mismatched marker (hotfix, direct push to main, stale PR, revert mode) → suites run as normal.
  • Escape hatch: set skip-verified-tests: false to always re-run on main.

@skridlevsky skridlevsky force-pushed the PLT-3267-skip-verified-tests-on-deploy branch from 435f6ae to da2abdc Compare June 30, 2026 13:49
@sonarqubecloud

Copy link
Copy Markdown

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