This repository was archived by the owner on Jun 13, 2025. It is now read-only.
feat(ecs-mcp-server): Add new ECS MCP Server for containerization and deployment#1
Open
karanbokil wants to merge 4 commits into
Open
feat(ecs-mcp-server): Add new ECS MCP Server for containerization and deployment#1karanbokil wants to merge 4 commits into
karanbokil wants to merge 4 commits into
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
Bandit found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
* feat(ecs-mcp-server): Add new ECS MCP Server for containerization and deployment and troubleshooting --------- Co-authored-by: Matthew Goodman <matthewgoodman13@outlook.com>
karanbokil
force-pushed
the
feature/ecs-mcp-server
branch
from
2e75b9a to
8db6e83
Compare
Comment on lines
+158
to
+159
| build_result = subprocess.run( | ||
| buildx_cmd, capture_output=True, text=True, shell=False, check=False |
Check notice
Code scanning / Bandit
subprocess call - check for execution of untrusted input. Note
Comment on lines
+182
to
+183
| build_result = subprocess.run( | ||
| build_cmd, capture_output=True, text=True, shell=False, check=False |
Check notice
Code scanning / Bandit
subprocess call - check for execution of untrusted input. Note
Comment on lines
+199
to
+200
| push_result = subprocess.run( | ||
| push_cmd, capture_output=True, text=True, shell=False, check=False |
Check notice
Code scanning / Bandit
subprocess call - check for execution of untrusted input. Note
Comment on lines
+233
to
+234
| verify_result = subprocess.run( | ||
| verify_cmd, capture_output=True, text=True, shell=False, check=False |
Check notice
Code scanning / Bandit
subprocess call - check for execution of untrusted input. Note
* Serverless MCP server initial commit * Migrate over more tools from typscript package, add more tests, remove api-level tools * Updates to readme, tool descriptions * Remove unused parameters from tools input models * Add new security flags and clean up code * Update README with security controls details * Update configure_domain tool to create certificates and remove unused models. * Add CODEOWNERS for serverless mcp * Let boto3 auto-detect region if not provided * Remove route 53 record set creation from deployWebAppTool * Update README * Migrate from pybars to jinja * Update serverless codeowners * Update unit tests and add docs page * Fix pre-commit errors * Fix bandit errors * Create uv.lock file * Fix unit tests * feat(aws-serverless-mcp-server) : add support for the schema registry * Mock github calls in serverless template test * Fix pre-commit issues * Fix pyright errors * Fix merge conflict with README * Update tests after pyright changes * Remove test for missing param since pyright validates already * Fix pre-commit issues * Add more unit tests for utils * Fix pre-commit issues * Fix pre-commit * update docs related to handle bar to jinja conversion * Add test for update frontend tool * addres pre-commit errors * Include examples in event schem input parmeter description * Disable new line stripping * Add pre-commit file and increase test coverage for server * Fix pre-comit issues * Fix bandit isssues * Flatten input parameters * Update README * Undo changes from README.md * Revert README back to main * Fix README * Fix pre-commmit * Add user agent to boto3 calls * Fix build * Audit secrets * Remove imports from __init__.py * Match init file * Address claude feedback * remove unused file * fix: package installation in README * Set mimetype for s3 upload and return stderr logs for local invoke --------- Co-authored-by: Brandon Xu <brandoxu@amazon.com> Co-authored-by: Scott Schreckengaust <scottschreckengaust@users.noreply.github.com> Co-authored-by: Mohamed MOUNIROU <mmouniro@amazon.com> Co-authored-by: Laith Al-Saadoon <9553966+theagenticguy@users.noreply.github.com>
|
This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds a new ECS MCP Server to the AWS MCP Servers repository. The server enables AI assistants to automatically containerize web applications and deploy them to AWS ECS with Fargate and Application Load Balancers.
Changes
The server provides the following MCP tools:
containerize_app: Provides best practices for docker/finch/compose wrt ECS Fargatecreate_ecs_infrastructure: Creates AWS ECS CloudFormation templates and (optionally) deploys CFN stacks, builds the image, and pushes the imageget_deployment_status: Monitors deployment status and retrieves ALB URLsdelete_ecs_infrastructure: Deletes CFN stacks created fromcreate_ecs_infrastructureUser experience
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Is this a breaking change? (N)
RFC issue number: aws/containers-roadmap#2580
Checklist:
Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.