docs: add comprehensive AWS ECS self-hosting guide#5302
Conversation
Co-Authored-By: ashwin@infisical.com <ashwin@infisical.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Greptile OverviewGreptile SummaryThis PR adds a comprehensive AWS ECS self-hosting guide that fills a gap in the documentation. The guide covers deploying Infisical on AWS using ECS Fargate, RDS PostgreSQL, ElastiCache Redis, and ALB. Key additions:
Previous review feedback addressed:
The documentation follows best practices and provides clear guidance for production deployments. Health check endpoint Confidence Score: 4/5
Important Files Changed
|
- Fix whitespace/indentation in docs.json - Add note about scoping down KMS permissions for production - Add note about additional ALB security options (CloudFront, WAF, IP allowlists) - Remove broken Terraform module link and clarify example is a starting point Co-Authored-By: ashwin@infisical.com <ashwin@infisical.com>
Co-Authored-By: ashwin@infisical.com <ashwin@infisical.com>
|
@greptileai Can you re-review this PR with all of your comments addressed? |
The docs.json navigation updates will be handled in a separate PR (#5305). Co-Authored-By: ashwin@infisical.com <ashwin@infisical.com>
Context
This PR adds a new comprehensive self-hosting guide for deploying Infisical on AWS using ECS Fargate, RDS PostgreSQL, ElastiCache Redis, and Application Load Balancer. This addresses the missing AWS deployment documentation that was identified as a gap in the self-hosting guides.
The guide includes all the sections requested:
The health check endpoint has been verified as
/api/status(not/api/healthas mentioned in the original PR #5106).Updates since last revision
infisical-terraform-awsrepo doesn't exist) and clarified the example is a starting pointSteps to verify the change
Human review checklist
/api/statusis correct for InfisicalType
Checklist
type(scope): short description(scope is optional, e.g.,fix: prevent crash on syncorfix(api): handle null response).Link to Devin run: https://app.devin.ai/sessions/de8b36ae4d4c4287ace71741aef029fe
Requested by: ashwin@infisical.com