-
Notifications
You must be signed in to change notification settings - Fork 428
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]: Snowpipe Not Being Recreated When Only Integration Changes #2785
Comments
Hey @Joe-Mooney 👋 |
Thanks for the response! I'll keep an eye open for the pipe refactoring in the upcoming quarter, and in the meantime will manually set the |
Great, for now, I'll add a note at the top of the pipe docs. |
- Added note about integration in pipes, because of #2785 - Adjusted account **note** and **warning** to have a proper structure (right now the docs are not treating **note** and **warning** as they should be: [current note and warning](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/account), and [how they should look like](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/grant_privileges_to_account_role))
Ha, I'm on @Joe-Mooney's team and just stumbled into this problem. In this case, the named value used to set the |
Hey @afeld |
Terraform CLI Version
1.8.2
Terraform Provider Version
0.87.2
Terraform Configuration
Category
category:resource
Object type(s)
resource:pipe
Expected Behavior
After having created a snowpipe with the above code, if only the integration value is changed (for example going from "TEST_INTEGRATION" to "TEST_INTEGRATION_V2"), the expectation is that the snowpipe will be updated to change the integration after an apply. At the very least, if this doesn't happen, there should be an error message saying the integration was not updated.
Actual Behavior
Upon running terraform plan, the following was returned:
Upon running terraform apply, the following was returned:
However, in Snowflake itself, the pipe's integration remained unchanged, staying as "TEST_INTEGRATION".
Steps to Reproduce
How much impact is this issue causing?
Medium
Logs
No response
Additional Information
I think this issue revolves around how the parameters are set in the snowflake_pipe resource. In the documentation for ALTER PIPE, it states that:
However, comparing the AWS_SNS_TOPIC and INTEGRATION definitions in the pkg/resources/pipe.go file, I noticed that only AWS_SNS_TOPIC has ForceNew=True:
I believe that integration also needs a ForceNew=True in it as well to force recreation of the pipe when the value of the integration parameters is changed. I am not sure though if this is accurate, or if this is the only change that would need to be made - it is just a theory I have from looking a few hours in the code base.
Would you like to implement a fix?
The text was updated successfully, but these errors were encountered: