Skip to content
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

AWS deployment #34

Merged
merged 20 commits into from
Sep 5, 2024
Merged

AWS deployment #34

merged 20 commits into from
Sep 5, 2024

Conversation

faisal-fawad
Copy link
Collaborator

@faisal-fawad faisal-fawad commented Aug 18, 2024

Details:

  • AWS CloudFormation templates for deploy, billing and roles
  • Scuffed build_lambda_code.py that builds a .zip file for deployment
  • GitHub actions deploy pipeline which utilizes scripts in aws directory

NOTE: The bandaid mentioned here is simply how the pipeline can be invoked - since on: [workflow_dispatch] means the pipeline can only be manually triggered on the default branch, we need on: [..., pull_request] to test our changes on a feature branch

@faisal-fawad faisal-fawad linked an issue Aug 18, 2024 that may be closed by this pull request
@faisal-fawad
Copy link
Collaborator Author

Will squash and merge once approved LOL. See comment in PR for "bandaid"

Threshold: 100 # 100% of budget is used
Subscribers:
- SubscriptionType: EMAIL
Address: !Ref emailAddress
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes me curious if rate limiting is something we can easily add, just to cover our bases

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will be moved to later PR

Copy link
Collaborator

@zachale zachale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@faisal-fawad faisal-fawad changed the title WIP AWS deployment AWS deployment Sep 5, 2024
@faisal-fawad faisal-fawad merged commit bfc28c2 into main Sep 5, 2024
3 checks passed
@faisal-fawad faisal-fawad deleted the 30-aws-cloudformation-template branch September 5, 2024 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AWS CloudFormation template
3 participants