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

Add CloudFront certificate #516

Merged
merged 5 commits into from
Jul 12, 2024
Merged

Add CloudFront certificate #516

merged 5 commits into from
Jul 12, 2024

Conversation

rhystmills
Copy link
Contributor

@rhystmills rhystmills commented Jul 8, 2024

Follow up to: #515

What does this change?

This PR adds a CloudFront certificate to Tagmanager, to be used in a follow-up PR. Once merged, the ARN for the certificate will be added to Parameter Store and used as part of the CloudFormation for a CloudFront Distribution.

This requires its own RiffRaff project (tag-manager-certificates) and CI step because it must be in the us-east-1 region, while the rest of the project is in eu-west-1.

How to test

Deploy to CODE, as I have done here. If the deploy succeeds, the CloudFormation is valid.

… in an upcoming CloudFront distribution. It requires its own yaml and CI step because

it must be in us-east-1, while the rest of our project is in eu-west-1. Its ARN will be used as a parameter in the main cloudfront yaml
@rhystmills rhystmills changed the base branch from main to rm/add-cname July 8, 2024 16:00
@rhystmills rhystmills changed the title Rm/use cloudfront Use cloudfront Jul 8, 2024
@rhystmills rhystmills force-pushed the rm/use-cloudfront branch 2 times, most recently from 224e619 to 0dee697 Compare July 9, 2024 07:59
@rhystmills rhystmills force-pushed the rm/use-cloudfront branch from 0dee697 to b9ec490 Compare July 9, 2024 08:14
@rhystmills rhystmills force-pushed the rm/use-cloudfront branch from 59a2293 to e6775eb Compare July 9, 2024 09:11
@rhystmills rhystmills changed the title Use cloudfront Add CloudFront certificate Jul 9, 2024
@rhystmills rhystmills marked this pull request as ready for review July 9, 2024 09:47
Copy link
Member

@davidfurey davidfurey left a comment

Choose a reason for hiding this comment

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

lgtm

Base automatically changed from rm/add-cname to main July 12, 2024 08:51
@rhystmills rhystmills merged commit a8be0a8 into main Jul 12, 2024
1 check passed
@rhystmills rhystmills deleted the rm/use-cloudfront branch July 12, 2024 10:15
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