-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add tf iac #415
Add tf iac #415
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few comments!
terragrunt init -upgrade | ||
terragrunt validate | ||
terragrunt plan -out tfplan | ||
# terragrunt apply tfplan |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we remove these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are just disabled temporarily so we can verify the production release workflow without actually overwriting the production environment
release-production: | ||
name: Release to Production | ||
needs: deploy-iac | ||
if: startsWith(github.ref_name, 'r-') # the prefix we have added to the tag |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this branch or tag name? so a tag name of r-05-21-24
would trigger it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tag name, it's created by the create-release job in the deploy-staging workflow. We can change how it's named if desired
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Naw it's good, thanks!
# terragrunt apply tfplan | ||
|
||
# - name: Deploy frontend to S3 | ||
# run: aws s3 sync ./frontend/frontend/dist "s3://${{ needs.deploy-iac.outputs.bucket }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm guessing this is temp?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just disabled temporarily so we can verify the production release workflow without actually overwriting the production environment
Description of the Change
Adds Tofu/Terragrunt IaC and github actions workflows to manage release & deployment
Benefits
Applicable Issues