-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
google_dns_record_set always wants to update #5497
Comments
|
@edwardmedia, when you reproed were you able to get DEBUG logs that you could share? It looks like it has to do with escaping the quotes but it would be good to know for sure whether it's the API doing it or something in our code. @matti, thanks for the report! Next time, please fill out the bug template- it asks for information that make it easier to figure out what's going on on our side. |
@danawillow Which particular piece of information did I leave out by not following the template? Honest question. |
Debug logs and provider version |
Oh and there's also some metadata in the template that helps our automation label issues so that our teammates don't have to hunt down issues without a label. |
Okay, the template is missing information on how to get DEBUG logs - I have used terraform and google provider for 3 years and I have no idea what is DEBUG log |
The template has the following block in the debug logs section:
If that documentation is confusing or could be improved, I'd recommend filing an issue against https://github.com/hashicorp/terraform so they can try to improve clarity there. Or if you'd just prefer that our bug template be phrased in a different way, you're welcome to propose a PR to change it (the source is at https://github.com/terraform-providers/terraform-provider-google/blob/master/.github/ISSUE_TEMPLATE/bug.md) or file an issue against us to do so (no issue template needed in that case) |
It is still not 100% clear
I am assuming TF_LOG=DEBUG ?
the template could just say:
Debug logs (TF_LOG=DEBUG terraform apply ...):
?
… On 29. Jan 2020, at 22.33, Dana Hoffman ***@***.***> wrote:
The template has the following block in the debug logs section:
To obtain the debug output, see the Terraform documentation on debugging.
If that documentation is confusing or could be improved, I'd recommend filing an issue against https://github.com/hashicorp/terraform so they can try to improve clarity there. Or if you'd just prefer that our bug template be phrased in a different way, you're welcome to propose a PR to change it (the source is at https://github.com/terraform-providers/terraform-provider-google/blob/master/.github/ISSUE_TEMPLATE/bug.md) or file an issue against us to do so (no issue template needed in that case)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Hey @matti, aside from the perpetual diff, do the record sets actually get created correctly? Our documentation says:
which was added after #232, which implies the record might be invalid with the extra quotes. I don't know if that's the case with a record like your example or just SPF, do you know? |
I came here for the same issue on TXT records (DKIM/SPF, etc..) - the quotes worked but it seems like a really odd syntax, e.g.
I'm really unclear on the defaults here, this provider splits strings within a list into a further list (so a list of lists?)? Why would it do that by default? It makes no sense to me, I'd maybe suggest this default behaviour is changed as it's completely counterintuitive. |
@andrew-s that requirement comes straight from the GCP documentation on record sets (https://cloud.google.com/dns/records/json-record). The provider is just passing along the strings exactly as they're sent. I'm going to go ahead and close this issue for now since it's been open for a few months without a response, but feel free to comment or file a new issue if you encounter any other issues or questions. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
like in #863
The text was updated successfully, but these errors were encountered: