-
Notifications
You must be signed in to change notification settings - Fork 427
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
Importing a procedure no longer works as described in the docs #2722
Comments
Hey @VenelinMartinov. Thanks for reaching out to us. This is a duplicate of the thread started here: #993 (comment). It is already fixed in #2697 and will be a part of tomorrow's release. Docs will be updated after the release, the note you attached at the end is the correct syntax after 0.85.0 version. |
Thanks for your response @sfc-gh-asawicki, glad to hear the issue is addressed. What about the panic? That's a pretty bad failure mode and really confusing for users - I only found the correct syntax after reading the code. Would it be possible to add a better error message on a malformed import statement? |
This is a valid suggestion. We have it already (better error handling) on our roadmap as part of https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#supporting-all-snowflake-ga-features. |
Okay, much appreciated, thanks for your help. |
Hey @VenelinMartinov. New version 0.89.0 was released. The docs were updated: https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/procedure#import. |
Closing as fixed. Please open a new issue if the problem persists in the newest provider version. |
Terraform CLI and Provider Versions
Terraform Configuration
Expected Behavior
Importing the procedure works.
Actual Behavior
It panics with:
Steps to Reproduce
terraform apply
terraform state rm snowflake_procedure.proc
terraform plan -generate-config-out=generated.tf
How much impact is this issue causing?
High
Logs
No response
Additional Information
note that importing it as
id = "MYDB-2.MYSCHEMA-2.SAMPLEPROC3(VARCHAR,DATE)"
works, but that does not match the docs: https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/procedure#importThe text was updated successfully, but these errors were encountered: