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

Greenfield infra code #4687

Merged
merged 7 commits into from Apr 21, 2023
Merged

Greenfield infra code #4687

merged 7 commits into from Apr 21, 2023

Conversation

yonassrobi
Copy link
Contributor

Resolves #

Description

This pull request contains the greenfield infrastructure code that can be used to deploy eAPD application in an AWS ECS Fargate environment. The terraform folder is mainly divided into bootstrapping and ECS folders. The bootstrapping folder contains terraform code needed for creating OIDC roles, ECR repositories, Postgres RDS database, and an EC2 instance for MongoDB needs. The ECS folder contains terraform code for provisioning load balancers, and ECS Fargate infrastructure. This pull request also contains sample GitHub actions for running some of the eAPD tests and for deploying eAPD web & eAPD API to ECS services. The changes in this PR will affect existing functionality and are not intended to run automatically without future intervention.

Chromatic Link

Significant changes or possible side effects

Automated test cases written

Given When Then Type (jest, tap, cypress)

Steps to manually verify this change

This pull request is ready to code review when

  • Automated tests are updated (and all tests are passing)
  • New automated test cases are documented above
  • Chromatic link added above
  • Pull request has been labeled, if applicable with feature, content, bug,
    tests, refactor
  • Associated OpenAPI documentation has been updated
  • The experience passes a basic manual accessibility audit (keyboard nav,
    screenreader, text scaling) OR an exemption is documented

This pull request is ready to test when

  • Code has been reviewed by someone other than the original author

This pull request is ready to review when QA has

  • Verified the functionality related to the change
  • Verified that the change works with Narrator on Windows
  • Verified that the change works with VoiceOver on Mac
  • Verified all updated pages with the WAVE tool
  • Verified tab and keyboard navigation functionality

This pull request can be merged when

  • Design has approved the experience
  • Product has approved the experience

@codecov
Copy link

codecov bot commented Apr 20, 2023

Codecov Report

Merging #4687 (8b8ff15) into main (a949dc8) will decrease coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4687      +/-   ##
==========================================
- Coverage   91.31%   91.26%   -0.06%     
==========================================
  Files         289      289              
  Lines        9525     9525              
  Branches     1916     1916              
==========================================
- Hits         8698     8693       -5     
- Misses        795      800       +5     
  Partials       32       32              
Flag Coverage Δ
api ∅ <ø> (∅)
common 99.17% <ø> (ø)
web 90.83% <ø> (-0.06%) ⬇️

see 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a949dc8...8b8ff15. Read the comment docs.

@cms-eapd-bot
Copy link

cms-eapd-bot commented Apr 20, 2023

This deploy was cleaned up.

Copy link
Contributor

@cthulhuplus cthulhuplus left a comment

Choose a reason for hiding this comment

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

👍

@tbolt tbolt merged commit e3eb85c into main Apr 21, 2023
10 of 12 checks passed
@tbolt tbolt deleted the greenfield-infra branch April 21, 2023 19:06
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.

None yet

5 participants