Skip to content

ci: run ECS deploys in the build workflow's concurrency group - #156

Merged
AdamBelfki3 merged 1 commit into
mainfrom
ecs-deploy-concurrency
Oct 29, 2026
Merged

AdamBelfki3 merged 1 commit into
mainfrom
ecs-deploy-concurrency

Conversation

@AdamBelfki3

@AdamBelfki3 AdamBelfki3 commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Summary

Puts the Deploy to ECS workflow in the concurrency group main.yml already uses on main (build-images-refs/heads/main), so the two can't run at the same time.

Without this, a deploy started during an image build forces a new ECS deployment on the previous latest digest. It looks like a successful deploy, but the image that's still building isn't on it.

Behavior

  • A deploy started while a build on main is running waits as pending, then runs once the build finishes and ships the new image.
  • A build pushed while a deploy is running waits for the deploy to finish.
  • GitHub keeps at most one pending run per group. If a deploy is pending behind a build and another push to main queues a new build, the pending deploy is cancelled and needs re-running after that build.

Testing

  • actionlint passes on the workflow.

🤖 Generated with Claude Code

Puts deploy-ecs.yml in the same concurrency group main.yml uses on main,
so a deploy started during an image build waits for it to finish and
ships the new image instead of the previous `latest`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
workbench Ready Ready Preview Oct 29, 2026 7:11pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2f259565-4854-40b0-abf9-d5438f5c4905
Commits

Reviewing files that changed from the base of the PR and between 7c98a18 and e2e95ef.

Files selected for processing: 1
  • .github/workflows/deploy-ecs.yml
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@AdamBelfki3
AdamBelfki3 merged commit a6b056c into main Oct 29, 2026
6 of 7 checks passed
@github-actions

github-actions Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

🚀 Preview deployed

@argos-ci

argos-ci Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ⚠️ Changes detected (Review) 13 removed Oct 29, 2026, 7:18 PM

This branch was successfully deployed

1 active deployment
Preview — e2e95efd Deployed Oct 29, 2026 by vercel[bot]
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