Skip to content

ci: disable SDK preview tests that break Railway environment#4080

Merged
mmabrouk merged 1 commit intomainfrom
ci/disable-sdk-preview-tests
Mar 31, 2026
Merged

ci: disable SDK preview tests that break Railway environment#4080
mmabrouk merged 1 commit intomainfrom
ci/disable-sdk-preview-tests

Conversation

@mmabrouk
Copy link
Copy Markdown
Member

@mmabrouk mmabrouk commented Mar 30, 2026

Summary

  • Removes the tests-sdk job from the PR preview workflow (14-check-pr-preview.yml) that was added in ci: re-enable auth bootstrap and web tests in PR preview #4072
  • The SDK acceptance tests create accounts via /admin/account and deploy variants to production environment, which corrupts the shared preview state and breaks Playwright auth for the web tests
  • Web-only tests and auth bootstrap remain enabled
  • The concurrency key fix in 44-railway-tests.yml is kept since it's a standalone improvement

Open with Devin

The SDK acceptance tests create accounts and mutate deployment state
(e.g. deploy to production environment), which breaks auth for the
web Playwright tests running in parallel. Remove the tests-sdk job
until the SDK suite is made safe for shared preview environments.
@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)
agenta-documentation Ready Ready Preview, Comment Mar 30, 2026 3:26pm

Request Review

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Mar 30, 2026
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 30, 2026

Railway Preview Environment

Status Destroyed (PR closed)

Updated at 2026-03-31T08:20:16.305Z

@mmabrouk mmabrouk requested review from jp-agenta and junaway March 31, 2026 08:17
@mmabrouk mmabrouk enabled auto-merge March 31, 2026 08:17
@mmabrouk mmabrouk merged commit c834223 into main Mar 31, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants