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

CMDCT-4184 sls-side prepping serverless to retain Cloudfront Distribution when removed #14990

Open
wants to merge 11 commits into
base: pete-sls
Choose a base branch
from

Conversation

peoplespete
Copy link

@peoplespete peoplespete commented Dec 5, 2024

Description

This change is not to be deployed until we are ready to convert to cdk. For full details of the order of events recommended to preserve our Cloudfront Distribution check out this readme: https://github.com/Enterprise-CMCS/macpro-mdct-seds/blob/cmdct-4184-cdk/import_instructions.md
The idea is that by preserving Cloudfront Distribution we will be able to deploy cdk without requiring any external DNS changes. 🎉

Related ticket

CMDCT-4184


How to test

If you want to you can do the following to test this PR:

  1. checkout pete-sls (matching master at the moment) branch
  2. deploy serverless via run script (check that cloudfront distribution points at app front end)
  3. checkout cmdct-4184-sls
  4. deploy serverless via run script (check that cloudfront distribution is stripped down and no longer points at app front end)

Notes

NA


Pre-review checklist

  • I have added thorough tests, if necessary
  • I have updated relevant documentation, if necessary
  • I have performed a self-review of my code
  • I have manually tested this PR in the deployed cloud environment

Pre-merge checklist

Review

  • Design: This work has been reviewed and approved by design, if necessary
  • Product: This work has been reviewed and approved by product owner, if necessary

Security

If either of the following are true, notify the team's ISSO (Information System Security Officer).

  • These changes are significant enough to require an update to the SIA.
  • These changes are significant enough to require a penetration test.

convert to a different template: test → val | val → prod

@peoplespete peoplespete force-pushed the cmdct-4184-sls branch 2 times, most recently from e4d1832 to 17128c3 Compare December 9, 2024 18:30
Copy link

codeclimate bot commented Dec 9, 2024

Code Climate has analyzed commit 8966ff4 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 82.3% (0.0% change).

View more on Code Climate.

@peoplespete peoplespete changed the base branch from master to pete-sls December 9, 2024 19:57
@peoplespete peoplespete changed the title CMDCT-4184 sls-side prepping serverless to retain ApiGateway when removed CMDCT-4184 sls-side prepping serverless to retain Cloudfront Distribution when removed Dec 9, 2024
@peoplespete peoplespete marked this pull request as ready for review December 10, 2024 14:21
@peoplespete peoplespete changed the base branch from pete-sls to master December 31, 2024 15:31
@peoplespete peoplespete changed the base branch from master to pete-sls December 31, 2024 15:33
@JonHolman
Copy link
Contributor

All of this except for modifying the CloudFront distribution can be done immediately. How about we do it now, so that we can get it over with?

@peoplespete
Copy link
Author

All of this except for modifying the CloudFront distribution can be done immediately. How about we do it now, so that we can get it over with?

We decided to leave it here for now since all of the retains would need to be conditional if we merge it soon and rather than add that complexity and since we can't entirely get rid of pete-sls bc of CloudfrontDistribution changes, we'll keep it in the PR and then pete-sls.

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.

4 participants