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 Distribution managed resource #654
Conversation
Hi, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ulucinar Looks pretty close to merging, could you respond to the remaining comments?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after fixing that panic possibility and squashing to single commit. Thanks @ulucinar !
Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
…s-scrub Update docs terminology and typos
…-externalname update the externalname configuration for aws_lambda_alias to reflect…
Description of your changes
This PR aims to introduce a managed resource for CloudFront's Distribution API.
Fixes #236
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
Manually deployed a distribution via
v1alpha1.Distribution
resource. Updated itscomment
field. Disabled and reenabled it and disabled it via itsenabled
field. Finally, deleted it.