Sam Levenick
Sam Levenick
> ``` > Error: google-beta/services/networkconnectivity/resource_network_connectivity_hub_test.go:28:29: undefined: testAccCheckNetworkConnectivityHubDestroyProducer > ``` > > Without examples, the destroy producer function is not being generated. I'd recommend adding an example or copying the format...
Adding @trodge as reviewer as I'm going to be out
> > For one, this function is being used in 6 places across this file. Are we sure that we should be adding those fields to the initial plan for...
Let me know when this is ready for review
Needs rebase to fix conflicts, but looks fine to me
`Ideally, failing to delete an existing App Engine resource (or create a new one in its place) would not cause Terraform operations (apply or destroy) to fail.` I don't think...
The first usage of `terraform apply` will create the App Engine application as it does not exist within the project until explicitly created, so I'm not sure what your question...
Ah, well you can allow `terraform import` to fail if it doesn't find the application, which would mean Terraform would attempt to create the application It feels like you are...
Yeah, so unfortunately I don't think we will be able to make any significant changes to how this resource works. I'd like to clarify that `terraform destroy` will always succeed,...
That's a possibility. I'll mark this as an enhancement to trigger it to be triaged. This request on Terraform core would make this a lot more possible, and likely matches...