Skip to content
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

snowflake_account resources can now be deleted #1973

Closed
jrobison-sb opened this issue Jul 24, 2023 · 5 comments
Closed

snowflake_account resources can now be deleted #1973

jrobison-sb opened this issue Jul 24, 2023 · 5 comments
Labels
bug Used to mark issues with provider's incorrect behavior

Comments

@jrobison-sb
Copy link
Contributor

Provider Version

$ terraform version
Terraform v1.5.3
on darwin_arm64
+ provider registry.terraform.io/snowflake-labs/snowflake v0.67.0

Terraform Version

$ terraform version
Terraform v1.5.3
on darwin_arm64
+ provider registry.terraform.io/snowflake-labs/snowflake v0.67.0

Describe the bug

At the time when the snowflake_account resource was first added to this provider, presumably it wasn't possible to delete accounts by way of the API. The only option was to delete accounts by way of a support ticket.

It has now become possible to DROP ACCOUNT, so the provider should support this.
https://docs.snowflake.com/en/release-notes/2023-07#deleting-an-account-self-service-preview
https://docs.snowflake.com/en/user-guide/organizations-manage-accounts-delete

Expected behavior

You should be able to delete an account using Terraform.

@jrobison-sb jrobison-sb added the bug Used to mark issues with provider's incorrect behavior label Jul 24, 2023
@sfc-gh-swinkler
Copy link
Collaborator

@jrobison-sb thank you for sharing this information. I have made a note of your feature request, and we will implement it when we have resources available.

@ben-wilson-peak
Copy link

@jrobison-sb thank you for sharing this information. I have made a note of your feature request, and we will implement it when we have resources available.

Hey @sfc-gh-swinkler, is there any movement internally on this feature? We note that the deletion API requires a grace period, which would need to be either hard coded in the provider (not that desirable) or defaulted and overridable as a param in the resource itself.

@sfc-gh-asawicki
Copy link
Collaborator

Hey @ben-wilson-peak. Yes, there is an open PR #2065 that supports account dropping.

@ben-wilson-peak
Copy link

Hey @ben-wilson-peak. Yes, there is an open PR #2065 that supports account dropping.

Amazing thank you for the heads up, totally missed that!

@jrobison-sb
Copy link
Contributor Author

#2065

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to mark issues with provider's incorrect behavior
Projects
None yet
Development

No branches or pull requests

4 participants