Skip to content

Build the UI image after tests pass, then ask infra for the deploy PR - #17

Merged
bulbashenko merged 1 commit into
mainfrom
deploy-prs
Oct 12, 2026
Merged

bulbashenko merged 1 commit into
mainfrom
deploy-prs

Conversation

@bulbashenko

Copy link
Copy Markdown
Collaborator

image.yml now runs on workflow_run of test, only for pushes to main where test (UI checks + e2e) succeeded, and builds the tested head_sha. Previously it ran in parallel with the tests, so an image could be promoted for a commit whose e2e failed. A promote job then opens or refreshes the PR deploy/auth in bulbashenko-com/infra (the owner merges it). Cost: the image appears after e2e (~10-30 min) instead of right after the push. workflow_dispatch still builds the selected ref by hand.

Part of bulbashenko-com/infra#102. Merge it after that PR and after the org secret DEPLOY_APP_PRIVATE_KEY and variable DEPLOY_APP_CLIENT_ID exist. Until then the promote job fails, but the build still works.

🤖 Generated with Claude Code

image.yml now runs on workflow_run of test, only for pushes to main where test (UI checks + e2e) succeeded, and builds the tested head_sha. Previously it ran in parallel with the tests, so an image could be promoted for a commit whose e2e failed. A promote job then opens or refreshes the PR deploy/auth in bulbashenko-com/infra (the owner merges it). Cost: the image appears after e2e (~10-30 min) instead of right after the push. workflow_dispatch still builds the selected ref by hand.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bulbashenko
bulbashenko merged commit 36de57e into main Oct 12, 2026
2 checks passed
@bulbashenko
bulbashenko deleted the deploy-prs branch October 12, 2026 17:45
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