Skip to content

Fix CI release pipeline stability#233

Merged
s-b-e-n-s-o-n merged 1 commit intomainfrom
fix/ci-release-stability
Mar 30, 2026
Merged

Fix CI release pipeline stability#233
s-b-e-n-s-o-n merged 1 commit intomainfrom
fix/ci-release-stability

Conversation

@s-b-e-n-s-o-n
Copy link
Copy Markdown
Contributor

Summary

  • Lower load test request_rate floor from 10 to 5 req/s (CI runners hit 7-8)
  • Remove run_attempt suffix from QA image artifact names (re-runs broke Playwright)

Context

RC2 release workflow failed because CI Verify never passed — load test absolute floor too high for shared runners, and Playwright couldn't find artifacts after re-running failed jobs.

- Lower load test request_rate floor from 10 to 5 req/s — CI shared
  runners consistently hit 7-8, failing the absolute floor despite
  passing relative regression thresholds
- Remove run_attempt suffix from QA image artifact names — re-running
  failed jobs incremented the attempt counter, causing Playwright to
  look for an artifact name that doesn't exist
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

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

Project Deployment Actions Updated (UTC)
drydock-demo Ready Ready Preview, Comment Mar 30, 2026 2:28am
drydock-website Ready Ready Preview, Comment Mar 30, 2026 2:28am

@s-b-e-n-s-o-n s-b-e-n-s-o-n merged commit 81cdcbb into main Mar 30, 2026
10 of 13 checks passed
@s-b-e-n-s-o-n s-b-e-n-s-o-n deleted the fix/ci-release-stability branch March 30, 2026 02:25
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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