search

iterative / terraform-provider-iterative

☁️ Terraform plugin for machine learning workloads: spot instance recovery & auto-termination | AWS, GCP, Azure, Kubernetes
https://registry.terraform.io/providers/iterative/iterative/latest/docs
Apache License 2.0
287 stars 27 forks source link

telemetry: remove unnecessary error from payload #703

Closed casperdcl closed 1 year ago

casperdcl commented 1 year ago

Same as https://github.com/iterative/cml/pull/1236

https://github.com/iterative/terraform-provider-iterative/blob/d53e68f27b3572b047853c3126435fb592b26fd0/iterative/utils/analytics.go#L319-L355

instead logic should be:

 if e != nil { 
    payload["error"] = reflect.TypeOf(e).String()
 } // otherwise, "error" should not appear in payload