Feature/s3 bucket - #14
Merged
Merged
Conversation
- add random provider for stable UUID - with encryption, lifecycle, and access block
- Create custom DB parameter group - Create DB subnet group from private subnets - Allow ingress from app_sg on 5432 - Outputs: endpoint and port"
- Write DB connection variables into .env on boot - Restart webapp service with updated RDS settings
- Created IAM role and instance profile for EC2 - Defined trust policy allowing EC2 to assume the role - Added least-privilege S3 access policy (List/Get/Put/Delete on specific bucket)
- Added AWS_REGION and S3_BUCKET variables in user_data.sh - Passed new vars via templatefile in ec2.tf - Used random_uuid to keep S3 bucket name stable across applies
- resolve undefined variable issue and add logging to /var/log/user_data_setup.log in user_data.sh
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.