-
Notifications
You must be signed in to change notification settings - Fork 20
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
Implement dnsimple_domain_delegation
resource
#99
Conversation
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.
Some suggestions and comments were left. The id
as a required attribute would need to be changed to domain
. We will still need to keep the id around as a read-only attribute.
QAd and the resource works exactly as expected. After the comments are addressed this should be good to go. |
Co-authored-by: Ivan Bakalov <ivan.bakalov@dnsimple.com>
Co-authored-by: Ivan Bakalov <ivan.bakalov@dnsimple.com>
Co-authored-by: Ivan Bakalov <ivan.bakalov@dnsimple.com>
This creates a resource that will take control of a domain's delegation settings and allow management via Terraform. When the resource is destroyed, only the Terraform state is removed; the domain delegation is kept as is and will no longer be managed by Terraform.