terraform-provider-godaddy
terraform-provider-godaddy copied to clipboard
Granular REST API requests to Godaddy, updating individual records only
Code is commented so should be reasonably self-explainatory.
@ records will be submitted in batch because there is no way to separate out them on a subdomain basis
Any other records with both unique type and name will be submitted individually
@kolikons: Is this something you would mind adding to your fork to make it more easily accessible?
Not sure I understand what you mean, my changes are in my fork. For example: https://github.com/jakob-northvolt/terraform-godaddy/blob/master/api/domains.go
@jakob-northvolt Yes, I'm aware of that. But since kolikons code are accessible as a provider from the registry I just thought it would make sense to merge in the changes you made on your fork.
@kolikons: Is this something you would mind adding to your fork to make it more easily accessible?
https://github.com/kolikons/terraform-provider-godaddy/pull/2 if u want to fix makefile =)
Not sure I understand what you mean, my changes are in my fork. For example: https://github.com/jakob-northvolt/terraform-godaddy/blob/master/api/domains.go
@jakob-northvolt Yes, I'm aware of that. But since kolikons code are accessible as a provider from the registry I just thought it would make sense to merge in the changes you made on your fork.
I'm sorry, I did not see first that you pinged kolikons in your comment, I thought it was aimed for me. I understand what you meant now.
Hey @jakob-northvolt - thanks for the PR. Could you merge in the latest code from the main branch?
Yes, np, I'll do it asap
Gents, we started using this provider and stumbled with this issue as well. Is there an estimate for this PR to be merged? Thanks for sharing.
hello @jakob-northvolt could you resolve the merge conflicts?
Unfortunately, I don't think this PR fixes #39. The plan will still show everything changing - Terraform doesn't seem very good at figuring out when two items in a set are the same :(
Without a fix for the issue mentioned - this plugin is severely limited.
This is a sorely needed fix.
Any chance on getting this merged in?
I guess @jakob-northvolt would have to resolve the conflicts first.
Any updates on this? its really impossible to use it, almost gave up and updating manually now.
Any updates on this? its really impossible to use it, almost gave up and updating manually now.
Facing similar issues, I've decided to re-implement provider from scratch to operate on record level.