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

Implement dnsimple_domain_delegation resource #99

Merged
17 commits merged into from
Apr 20, 2023
Merged

Conversation

ghost
Copy link

@ghost ghost commented Apr 13, 2023

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.

@ghost ghost requested a review from DXTimer April 13, 2023 10:33
@ghost ghost self-assigned this Apr 13, 2023
@ghost ghost self-requested a review April 13, 2023 10:33
Copy link
Contributor

@DXTimer DXTimer left a 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.

docs/resources/domain_delegation.md Outdated Show resolved Hide resolved
docs/resources/domain_delegation.md Outdated Show resolved Hide resolved
docs/resources/domain_delegation.md Outdated Show resolved Hide resolved
internal/framework/resources/domain_delegation_resource.go Outdated Show resolved Hide resolved
@DXTimer
Copy link
Contributor

DXTimer commented Apr 19, 2023

QAd and the resource works exactly as expected. After the comments are addressed this should be good to go.

Wilson Lin and others added 5 commits April 20, 2023 14:14
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>
@ghost ghost requested a review from DXTimer April 20, 2023 04:31
@ghost ghost merged commit ff607cb into main Apr 20, 2023
@ghost ghost deleted the feature/domain-delegation branch April 20, 2023 04:52
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant