-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
[New Resource] Create delegation signer association resource #33596
Conversation
Community NoteVoting for Prioritization
For Submitters
|
Hey @seanseth7 👋 Thank you very much for your contribution! At times, our maintainers need to make direct edits to pull requests in order to help get it ready to be merged. Your current settings do not allow maintainers to make such edits. To help facilitate this, update your pull request to allow such edits as described in GitHub's Allowing changes to a pull request branch created from a fork documentation. (If you're using a fork owned by an organization, your organization may not allow you to change this setting. If that is the case, let us know.) |
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.
Welcome @seanseth7 👋
It looks like this is your first Pull Request submission to the Terraform AWS Provider! If you haven’t already done so please make sure you have checked out our CONTRIBUTOR guide and FAQ to make sure your contribution is adhering to best practice and has all the necessary elements in place for a successful approval.
Also take a look at our FAQ which details how we prioritize Pull Requests for inclusion.
Thanks again, and welcome to the community! 😃
d939152
to
00bc908
Compare
a76368e
to
e38626a
Compare
f855d56
to
5b5fce4
Compare
0f3e0f2
to
2970110
Compare
2970110
to
cc53375
Compare
Hey @ewbankkit ! I see that you've recently been reviewing the PRs in this repo, and I'm not sure if the maintainers got notified when I marked this ready for review, so just wanted to put this on your radar in case there was no notification. No rush. Thanks! |
@justinretzolk Any chance that someone could review this soon since this has been open for a while? Apologies if you're not the correct person to ask. Please feel free to tag the relevant people if that's the case. |
@seanseth7 Thanks for the contribution 🎉 👏. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
Adds support for creating a Delegation Signer record in the parent DNS zone for domains registered with Route53
Relations
Closes #28749
References
https://docs.aws.amazon.com/Route53/latest/APIReference/API_domains_AssociateDelegationSignerToDomain.html
https://docs.aws.amazon.com/Route53/latest/APIReference/API_domains_DisassociateDelegationSignerFromDomain.html
Output from Acceptance Testing