provider/datadog: make datadog_user verified a computed attribute #15034
Conversation
It's read-only and generates supurious diffs for verified users.
| @@ -107,7 +108,7 @@ func resourceDatadogUserRead(d *schema.ResourceData, meta interface{}) error { | |||
| return err | |||
| } | |||
|
|
|||
| log.Printf("[DEBUG] user: %v", u) | |||
| log.Printf("[DEBUG] user: %s", spew.Sdump(u)) | |||
There was a problem hiding this comment.
Can we remove the spew.Dump here? Think this is debug work ;)
There was a problem hiding this comment.
I only left it in as the #v output is useless - it's just a bunch of memory addresses. Should we remove the entire line?
There was a problem hiding this comment.
I just removed the entire debug line as it's pretty much useless :) <3 for the fixing of the debug though
Docs already say this is an output-only attribute so this is just fixing up the code to agree with them! 🔧 https://www.terraform.io/docs/providers/datadog/r/user.html#verified |
|
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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
ghost
locked and limited conversation to collaborators
It's read-only and generates supurious diffs for verified users.