-
Notifications
You must be signed in to change notification settings - Fork 418
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
[Bug]: snowflake_task resource error_integration got unconvertible type 'sdk.AccountObjectIdentifier' #3136
Comments
Hey @FrelliBB 👋 |
Thanks for the quick reply. |
Just confirmed that the error occurs in the "old" version and the current refactored version works. I'll try to see if there's any workaround for this in the older version (If I won't find it today, I'll check tomorrow). |
The following workarounds for now:
|
We are happy to skip the upgrade 0.97.0 and wait for a correct fix. Would that be possible or would the fix be targeting GA and require a longer wait? For context we spotted this issue on automated dependabot upgrades from 0.96.0 to 0.97.0, so we can skip this version for now. |
Ok 👍 the fix should be a part of 0.98.0 or 0.99.0, but it will most likely come with additional changes (e.g. field renames, etc.) that have to be migrated. That's because the refactor of tasks is done as part of preparing GA objects for v1, and all the changes will be listed in the migration guide. |
Terraform CLI Version
1.9.6
Terraform Provider Version
0.97.0
Terraform Configuration
Category
category:resource
Object type(s)
resource:task
Expected Behavior
Resource is created normally.
Actual Behavior
Terraform plan fails with.
Steps to Reproduce
Use the configuration and run Terraform apply.
How much impact is this issue causing?
Medium
Logs
No response
Additional Information
It looks like it might be related to https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/3086/files#diff-cb7f0da2beafb1e661c9b5cfd7df2f625e57b4e8af1aa913257fa4901bfd9846R466. I checked the migration guide but didn't see anything about this change in it.
Would you like to implement a fix?
The text was updated successfully, but these errors were encountered: