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 the ability to update CNAME and other DNS records #80

Closed
mmckane opened this issue Jan 8, 2021 · 9 comments · Fixed by #128
Closed

Add the ability to update CNAME and other DNS records #80

mmckane opened this issue Jan 8, 2021 · 9 comments · Fixed by #128
Assignees
Labels
enhancement New feature or request

Comments

@mmckane
Copy link

mmckane commented Jan 8, 2021

Requiring a delete on a resource to update DNS is not efficient and makes using terraform to manage DNS useless. Please add the ability to update records with terraform, or provide any info as to why this is technically not possible.

@donwlewis
Copy link

@mmckane would you consider forcing a new resource on a change sufficient enough? We had a discussion on this in the previous maintainer project, and a quick fix was purposed with this pull request:

hashicorp/terraform-provider-infoblox#41

@mmckane
Copy link
Author

mmckane commented Jan 19, 2021

@donwlewis I think that could be a good quick fix. could that be set for the other record types that couldn't be updated as well, as looking this might not be unique to CNAMEs?

I did dig enough and found thus issue/feature request in the underlying go client infobloxopen/infoblox-go-client#110

@PhilipWhiteside
Copy link

@donwlewis for my use that scenario would fit. I'm not precious about the original entries, just the result of name -> name. The entry is managed by Terraform, so nothing else should be tied to it outside of Terraform. However, performing a destroy, or taint, is challenging when we use a pipeline tied to git to push the changes into IB rather than local Terraform CLI.

I see that hashicorp/terraform-provider-infoblox#41 would support that. This is a MR into the old repo, is that going to be a problem as this repo has moved to here?

@PhilipWhiteside
Copy link

@deromemont You created this MR hashicorp/terraform-provider-infoblox#41 in an old repo which has been migrated here. The MR has not moved across with it. Are you able to push to this repo, or are you happy for me to copy your changes and do so on your behalf?

@deromemont
Copy link

@PhilipWhiteside you can copy my changes if you want ;)

@somashekhar
Copy link
Contributor

We are working on this requirement.
We need to write update on infoblox go client and then implement the same at terraform.

@somashekhar somashekhar added the enhancement New feature or request label Mar 5, 2021
@somashekhar somashekhar self-assigned this Apr 22, 2021
@jfreeland
Copy link

What is the status of this?

@somashekhar
Copy link
Contributor

@jfreeland work on this is almost complete and we are testing it along with other features. Can expect it under develop branch soon.

@somashekhar
Copy link
Contributor

Now the feature is available at master branch. Will be available as part of v2.0.0

@somashekhar somashekhar linked a pull request Jul 26, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants