Skip to content

fix(ci): Fix ECS deployment following attempted task definition separation#6719

Merged
matthewelwell merged 8 commits intomainfrom
fix(ci)/task-def-version-check-failures
Feb 16, 2026
Merged

fix(ci): Fix ECS deployment following attempted task definition separation#6719
matthewelwell merged 8 commits intomainfrom
fix(ci)/task-def-version-check-failures

Conversation

@matthewelwell
Copy link
Contributor

@matthewelwell matthewelwell commented Feb 13, 2026

Changes

This fixes the work started in this PR to use separate task definitions for the newly separated ECS services (admin API vs SDK).

Depends on:

How did you test this code?

Temporarily updated the workflow to deploy this branches code to staging (see here).

@vercel
Copy link

vercel bot commented Feb 13, 2026

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

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Feb 16, 2026 6:50pm
flagsmith-frontend-preview Ignored Ignored Preview Feb 16, 2026 6:50pm
flagsmith-frontend-staging Ignored Ignored Preview Feb 16, 2026 6:50pm

Request Review

@github-actions github-actions bot added the fix label Feb 13, 2026
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.23%. Comparing base (262e4f4) to head (4590094).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6719   +/-   ##
=======================================
  Coverage   98.23%   98.23%           
=======================================
  Files        1311     1311           
  Lines       48474    48474           
=======================================
  Hits        47617    47617           
  Misses        857      857           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@matthewelwell matthewelwell marked this pull request as ready for review February 16, 2026 18:01
@matthewelwell matthewelwell requested review from a team as code owners February 16, 2026 18:01
@matthewelwell matthewelwell requested review from gagantrivedi and removed request for a team February 16, 2026 18:01
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on March 10

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 16, 2026

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-6719 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-6719 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-6719 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-api:pr-6719 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-6719 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-6719 Finished ✅ Results

emyller
emyller previously approved these changes Feb 16, 2026
@matthewelwell matthewelwell merged commit 79b7724 into main Feb 16, 2026
49 of 52 checks passed
@matthewelwell matthewelwell deleted the fix(ci)/task-def-version-check-failures branch February 16, 2026 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants