chore: Remove deprecation from unsafe execute (#2941)

- Remove the deprecation message from the unsafe_execute resource because it's planned to be supported in V1 (references: #2934) - Add #2938 to the list of known issues for the user object.
Snowflake-Labs · Jul 17, 2024 · ed712d7 · ed712d7
1 parent 4ce662d
commit ed712d7
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 11 deletions.
diff --git a/docs/index.md b/docs/index.md
@@ -240,7 +240,6 @@ The Snowflake provider will use the following order of precedence when determini
 - [snowflake_oauth_integration](./docs/resources/oauth_integration)
 - [snowflake_role](./docs/resources/role) - use [snowflake_account_role](./docs/resources/account_role) instead
 - [snowflake_saml_integration](./docs/resources/saml_integration) - use [snowflake_saml2_integration](./docs/resources/saml2_integration) instead
-- [snowflake_unsafe_execute](./docs/resources/unsafe_execute)
 
 ## Currently deprecated datasources
 

diff --git a/docs/resources/unsafe_execute.md b/docs/resources/unsafe_execute.md
@@ -3,20 +3,20 @@
 page_title: "snowflake_unsafe_execute Resource - terraform-provider-snowflake"
 subcategory: ""
 description: |-
-  Experimental resource used for testing purposes only. Allows to execute ANY SQL statement.
+  Experimental resource allowing execution of ANY SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Use at your own risk.
 ---
 
 # snowflake_unsafe_execute (Resource)
 
-!> **Warning** This is a dangerous resource that allows executing **ANY** SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Will be deleted in the upcoming versions. Use at your own risk.
+!> **Warning** This is a dangerous resource that allows executing **ANY** SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Use at your own risk.
+
+~> **Note** This resource will be included in the V1 (check [here](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/v1-preparations/ESSENTIAL_GA_OBJECTS.MD)) but may be slightly modified before. Design decisions and changes will be listed in the [migration guide](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/MIGRATION_GUIDE.md#migration-guide).
 
 ~> **Note** It can be theoretically used to manage resource that are not supported by the provider. This is risky and may brake other resources if used incorrectly.
 
 ~> **Note** Use `query` parameter with caution. It will fetch **ALL** the results returned by the query provided. Try to limit the number of results by writing query with filters. Query failure does not stop resource creation; it simply results in `query_results` being empty.
 
-~> **Deprecation** Experimental resource. Will be deleted in the upcoming versions. Use at your own risk. <deprecation>
-
-Experimental resource used for testing purposes only. Allows to execute ANY SQL statement.
+Experimental resource allowing execution of ANY SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Use at your own risk.
 
 ## Example Usage
 

diff --git a/examples/additional/deprecated_resources.MD b/examples/additional/deprecated_resources.MD
@@ -4,4 +4,3 @@
 - [snowflake_oauth_integration](./docs/resources/oauth_integration)
 - [snowflake_role](./docs/resources/role) - use [snowflake_account_role](./docs/resources/account_role) instead
 - [snowflake_saml_integration](./docs/resources/saml_integration) - use [snowflake_saml2_integration](./docs/resources/saml2_integration) instead
-- [snowflake_unsafe_execute](./docs/resources/unsafe_execute)
diff --git a/pkg/resources/unsafe_execute.go b/pkg/resources/unsafe_execute.go
@@ -51,8 +51,7 @@ func UnsafeExecute() *schema.Resource {
 
 		Schema: unsafeExecuteSchema,
 
-		DeprecationMessage: "Experimental resource. Will be deleted in the upcoming versions. Use at your own risk.",
-		Description:        "Experimental resource used for testing purposes only. Allows to execute ANY SQL statement.",
+		Description: "Experimental resource allowing execution of ANY SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Use at your own risk.",
 
 		CustomizeDiff: func(_ context.Context, diff *schema.ResourceDiff, _ interface{}) error {
 			if diff.HasChange("query") {

diff --git a/templates/resources/unsafe_execute.md.tmpl b/templates/resources/unsafe_execute.md.tmpl
@@ -12,7 +12,9 @@ description: |-
 
 # {{.Name}} ({{.Type}})
 
-!> **Warning** This is a dangerous resource that allows executing **ANY** SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Will be deleted in the upcoming versions. Use at your own risk.
+!> **Warning** This is a dangerous resource that allows executing **ANY** SQL statement. It may destroy resources if used incorrectly. It may behave incorrectly combined with other resources. Use at your own risk.
+
+~> **Note** This resource will be included in the V1 (check [here](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/v1-preparations/ESSENTIAL_GA_OBJECTS.MD)) but may be slightly modified before. Design decisions and changes will be listed in the [migration guide](https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/MIGRATION_GUIDE.md#migration-guide).
 
 ~> **Note** It can be theoretically used to manage resource that are not supported by the provider. This is risky and may brake other resources if used incorrectly.