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

chore: add health cicd #1799

Merged
merged 5 commits into from
Apr 26, 2022
Merged

chore: add health cicd #1799

merged 5 commits into from
Apr 26, 2022

Conversation

gweiying
Copy link
Contributor

@gweiying gweiying commented Mar 31, 2022

Context

How did you solve the problem?

This PR adds the following deployment actions to our GitHub Actions:

  • Deploys GoGov web application to health-staging Elastic Beanstalk
  • Deploys GoGov web application to health-production Elastic Beanstalk
  • Deploys GoGov Serverless migration functions to health-* Lambda
  • Deploys GoGov Serverless migration functions to edu-* Lambda

This PR uses the edu asset variant in the CI/CD pipeline for building the health environment. We will replace the asset variant in a later PR when the health assets are ready.

Tests

What tests should be run to confirm functionality?

Push branch to edge

  • Verify that all deployment actions occur correctly via GitHub Actions console
  • Verify on Elastic Beanstalk console that EB staging applications have been redeployed
  • Verify on Lambda console that Lambda staging applications have been redeployed

Deploy Notes

Notes regarding deployment of the contained body of work. These should note any
new dependencies, new scripts, etc.

  1. This PR uses the edu asset variant in the CI/CD pipeline for building the health environment. We will replace the asset variant in a later PR when the health assets are ready.

  2. health-production and edu-production Lambda functions have not been set up yet. If the branch is pushed to production, the Lambda functions will be generated from scratch, and we will need to log in to configure the network configurations.

@gweiying gweiying marked this pull request as ready for review April 5, 2022 07:12
@gweiying gweiying mentioned this pull request Apr 5, 2022
45 tasks
@gweiying gweiying changed the base branch from chore/ci-cleanup to develop April 26, 2022 07:16
@gweiying gweiying force-pushed the build/health-pipeline branch from e9b501d to 59c46e6 Compare April 26, 2022 07:42
@gweiying gweiying requested a review from orbitalsqwib April 26, 2022 07:42
Copy link
Contributor

@orbitalsqwib orbitalsqwib left a comment

Choose a reason for hiding this comment

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

lgtm!

@gweiying gweiying merged commit d0e9940 into develop Apr 26, 2022
@gweiying gweiying deleted the build/health-pipeline branch April 26, 2022 07:53
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.

2 participants