Add migration guide and docs

Snowflake-Labs · Nov 8, 2024 · 1f0c231 · 1f0c231
1 parent 16fca6b
commit 1f0c231
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 0 deletions.
diff --git a/MIGRATION_GUIDE.md b/MIGRATION_GUIDE.md
@@ -167,6 +167,18 @@ Note: Because [bcr 2024_07](https://docs.snowflake.com/en/release-notes/bcr-bund
 
 Connected issues: [#3125](https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/3125)
 
+### *(bugfix)* Handle user import correctly
+
+#### Context before the change
+
+Password is empty after the `snowflake_user` import; we can't read it from the config or from Snowflake.
+During the next terraform plan+apply it's updated to the "same" value.
+It results in an error on Snowflake side: `New password rejected by current password policy. Reason: 'PRIOR_USE'.`
+
+#### After the change
+
+The error will be ignored on the provider side (after all, it means that the password in state is the same as on Snowflake side). Still, plan+apply is needed after importing user.
+
 ## v0.96.0 ➞ v0.97.0
 
 ### *(new feature)* snowflake_stream_on_table, snowflake_stream_on_external_table resource

diff --git a/docs/resources/legacy_service_user.md b/docs/resources/legacy_service_user.md
@@ -1011,3 +1011,5 @@ Import is supported using the following syntax:
 ```shell
 terraform import snowflake_legacy_service_user.example '"<user_name>"'
 ```
+
+Note: terraform plan+apply may be needed after successful import to fill out all the missing fields (like `password`) in state.
diff --git a/docs/resources/user.md b/docs/resources/user.md
@@ -1021,3 +1021,5 @@ Import is supported using the following syntax:
 ```shell
 terraform import snowflake_user.example '"<user_name>"'
 ```
+
+Note: terraform plan+apply may be needed after successful import to fill out all the missing fields (like `password`) in state.
diff --git a/templates/resources/legacy_service_user.md.tmpl b/templates/resources/legacy_service_user.md.tmpl
@@ -41,3 +41,5 @@ Import is supported using the following syntax:
 
 {{ codefile "shell" (printf "examples/resources/%s/import.sh" .Name)}}
 {{- end }}
+
+Note: terraform plan+apply may be needed after successful import to fill out all the missing fields (like `password`) in state.
diff --git a/templates/resources/user.md.tmpl b/templates/resources/user.md.tmpl
@@ -41,3 +41,5 @@ Import is supported using the following syntax:
 
 {{ codefile "shell" (printf "examples/resources/%s/import.sh" .Name)}}
 {{- end }}
+
+Note: terraform plan+apply may be needed after successful import to fill out all the missing fields (like `password`) in state.
Original file line number	Diff line number	Diff line change
Expand Up		@@ -41,3 +41,5 @@ Import is supported using the following syntax:

		{{ codefile "shell" (printf "examples/resources/%s/import.sh" .Name)}}
		{{- end }}

		Note: terraform plan+apply may be needed after successful import to fill out all the missing fields (like `password`) in state.