Skip to content

ci(mcp): render and deploy task definition to ECS#7716

Merged
germangarces merged 6 commits into
mainfrom
feat/mcp-cd-deploy
Jun 10, 2026
Merged

ci(mcp): render and deploy task definition to ECS#7716
germangarces merged 6 commits into
mainfrom
feat/mcp-cd-deploy

Conversation

@germangarces

@germangarces germangarces commented Jun 5, 2026

Copy link
Copy Markdown
Member

Thanks for submitting a PR! Please check the boxes below:

  • I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

Contributes to https://github.com/Flagsmith/pulumi/issues/235

Add AWS Task Definitions and a CD pipeline to deploy the new MCP server.

How did you test this code?

Not tested yet. I'll perform a end-to-end test once this PR is merged.

@germangarces
ci(mcp): render and deploy task definition to ECS
39a4074 
Signed-off-by: germangarces <german.garces@flagsmith.com>
@vercel

vercel Bot commented Jun 5, 2026 •
edited
Loading

Copy link
Copy Markdown

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

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Jun 10, 2026 9:30am
flagsmith-frontend-preview Ignored Ignored Preview Jun 10, 2026 9:30am
flagsmith-frontend-staging Ignored Ignored Preview Jun 10, 2026 9:30am

Request Review

@github-actions github-actions Bot added infrastructure ci-cd Build, test and deployment related and removed infrastructure labels Jun 5, 2026
@codecov

codecov Bot commented Jun 5, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.54%. Comparing base (1946ec5) to head (f5ba526).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #7716      +/-   ##
==========================================
+ Coverage   98.52%   98.54%   +0.02%     
==========================================
  Files        1444     1452       +8     
  Lines       54971    55821     +850     
==========================================
+ Hits        54161    55011     +850     
  Misses        810      810

☔ View full report in Codecov by Harness.
📢 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.

@germangarces
revert(mcp): deploy via force-new-deployment on latest
55b98e5 
Signed-off-by: germangarces <german.garces@flagsmith.com>
@github-actions github-actions Bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Jun 9, 2026
@germangarces
fix env name
5429714 
Signed-off-by: germangarces <german.garces@flagsmith.com>
@germangarces germangarces marked this pull request as ready for review June 10, 2026 07:52
@germangarces germangarces requested a review from a team as a code owner June 10, 2026 07:52
@germangarces germangarces requested review from khvn26 and removed request for a team June 10, 2026 07:52
@github-actions github-actions Bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Jun 10, 2026
@github-actions

github-actions Bot commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Docker builds report

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

@github-actions

github-actions Bot commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  39.7 seconds
commit  5429714
info  🔄 Run: #17371 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  38.7 seconds
commit  5429714
info  🔄 Run: #17371 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  39.4 seconds
commit  5429714
info  🔄 Run: #17371 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  1 minute, 10 seconds
commit  5429714
info  🔄 Run: #17371 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  41.1 seconds
commit  3028d18
info  🔄 Run: #17373 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  45.3 seconds
commit  3028d18
info  🔄 Run: #17373 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  3 passed

Details

stats  3 tests across 3 suites
duration  32.8 seconds
commit  3028d18
info  🔄 Run: #17373 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  1 minute, 14 seconds
commit  3028d18
info  🔄 Run: #17373 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  40.5 seconds
commit  b2b155d
info  🔄 Run: #17374 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  44.5 seconds
commit  b2b155d
info  🔄 Run: #17374 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  4 passed

Details

stats  4 tests across 4 suites
duration  4.5 seconds
commit  b2b155d
info  🔄 Run: #17374 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  54.2 seconds
commit  b2b155d
info  🔄 Run: #17374 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  40.8 seconds
commit  f5ba526
info  🔄 Run: #17376 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  43.5 seconds
commit  f5ba526
info  🔄 Run: #17376 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  42.1 seconds
commit  f5ba526
info  🔄 Run: #17376 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  55.7 seconds
commit  f5ba526
info  🔄 Run: #17376 (attempt 1)

@github-actions

github-actions Bot commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Visual Regression

19 screenshots compared. See report for details.
View full report

khvn26
khvn26 reviewed Jun 10, 2026
View reviewed changes

@khvn26 khvn26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A couple questions/comments

Comment thread .github/workflows/mcp-docker-build-publish.yml Outdated
Comment thread .github/workflows/mcp-docker-build-publish.yml Outdated
Comment thread .github/workflows/mcp-docker-build-publish.yml Outdated
Comment thread .github/workflows/mcp-docker-build-publish.yml Outdated
@germangarces
remove comments
3028d18 
Signed-off-by: germangarces <german.garces@flagsmith.com>
@github-actions github-actions Bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Jun 10, 2026
@germangarces
add worfklow_dispatch and uncomment
b2b155d 
Signed-off-by: germangarces <german.garces@flagsmith.com>
@github-actions github-actions Bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Jun 10, 2026
@germangarces
rename file
f5ba526 
Signed-off-by: germangarces <german.garces@flagsmith.com>
@github-actions github-actions Bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Jun 10, 2026
khvn26
khvn26 approved these changes Jun 10, 2026
View reviewed changes

@khvn26 khvn26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good job 👍

@germangarces germangarces merged commit 9d6105d into main Jun 10, 2026
41 checks passed
@germangarces germangarces deleted the feat/mcp-cd-deploy branch June 10, 2026 09:39
@flagsmith-engineering flagsmith-engineering Bot mentioned this pull request Jun 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@khvn26 khvn26 khvn26 approved these changes

Assignees

@khvn26 khvn26

Labels

ci-cd Build, test and deployment related

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@germangarces @khvn26