-
Notifications
You must be signed in to change notification settings - Fork 381
Add Terraform-based AWS ECS deployment option #3074
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add Terraform-based AWS ECS deployment option #3074
Conversation
This change ensures the entrypoint script has proper execute permissions, preventing exec format errors when running in container environments like ECS. Signed-off-by: shunskkkk <[email protected]>
Support POSTGRES_DB, POSTGRES_USER, and POSTGRES_PASSWORD environment variables while maintaining backward compatibility with default values. This enables flexible database configuration in containerized and cloud environments. Signed-off-by: shunskkkk <[email protected]>
Signed-off-by: shunskkkk <[email protected]>
❌ Deploy Preview for peppy-sprite-186812 failed.
|
|
Thanks for opening your first pull request in the Marquez project! Please check out our contributing guidelines (https://github.com/MarquezProject/marquez/blob/main/CONTRIBUTING.md). |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3074 +/- ##
=========================================
Coverage 81.18% 81.18%
Complexity 1506 1506
=========================================
Files 268 268
Lines 7356 7356
Branches 325 325
=========================================
Hits 5972 5972
Misses 1226 1226
Partials 158 158 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hey @shunskkkk this looks awesome! If you don't mind we will add it here https://github.com/ilum-cloud/marquez |
@thijs-s |
|
merged! ilum-cloud#1 thanks @shunskkkk ! |
Alpine Linux doesn't include curl by default, causing health checks to fail. Use wget which is available in Alpine Linux. Signed-off-by: shunskkkk <[email protected]>
Problem
While Marquez provides deployment solutions like Docker Compose and Helm charts for Kubernetes, there has been consistent demand in the Slack community for AWS ECS deployment support. For organizations that don't require the full complexity of EKS/Kubernetes, ECS offers a simpler and more cost-effective container orchestration solution. However, setting up ECS infrastructure manually requires configuring multiple AWS services, which can be complex and time-consuming.
Solution
This PR adds a complete AWS ECS deployment solution using Terraform to automate the infrastructure setup for organizations that prefer ECS over EKS for its simplicity and lower operational overhead. The solution includes:
The deployment creates a production-ready Marquez setup with:
Important Note: Docker images must be pushed to ECR after Terraform creates the repositories but before ECS services can start. This is clearly documented in the README.
One-line summary: Add AWS ECS deployment configuration with Terraform for production-ready infrastructure
Checklist
CHANGELOG.md.sqldatabase schema migration according to Flyway's naming convention (N/A - no database schema changes)