Skip to content
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.

Sign up for GitHub

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

chore: Grant ownership common use cases #3356

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from
+813 −1
Open

chore: Grant ownership common use cases #3356

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
4c89d0a
Add grant ownership examples to the guides
sfc-gh-jcieslak Jan 23, 2025
5d33cca
Add grant ownership examples to the guides
sfc-gh-jcieslak Jan 23, 2025
e59a428
Add grant ownership examples to the guides
sfc-gh-jcieslak Jan 24, 2025
0a9c6ca
Add grant ownership examples to the guides
sfc-gh-jcieslak Jan 24, 2025
a173cb0
Merge branch 'refs/heads/dev' into grant-ownership-common-use-cases
sfc-gh-jcieslak Jan 30, 2025
6c4a634
Changes after review
sfc-gh-jcieslak Jan 30, 2025
4d550a5
Add quoted identifiers ignore case parameter common issue
sfc-gh-jcieslak Jan 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions CREATING_ISSUES.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@
* [Granting on Functions or Procedures](#granting-on-functions-or-procedures)
* [Infinite diffs, empty privileges, errors when revoking on grant resources](#infinite-diffs-empty-privileges-errors-when-revoking-on-grant-resources)
* [Granting PUBLIC role fails](#granting-public-role-fails)
* [Issues with grant_ownership resource](#issues-with-grant_ownership)
* [Using QUOTED_IDENTIFIERS_IGNORE_CASE with the provider](#using-quoted_identifiers_ignore_case-with-the-provider)

This guide was made to aid with creating the GitHub issues, so you can maximize your chances of getting help as quickly as possible.
To correctly report the issue, we suggest going through the following steps.
Expand Down Expand Up @@ -276,3 +278,17 @@ Terraform may fail with:
**Related issues:** [#3001](https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/3001), [#2848](https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2848)

**Solution:** This happens, because the PUBLIC role is a "pseudo-role" (see [docs](https://docs.snowflake.com/en/user-guide/security-access-control-overview#system-defined-roles)) that is already assigned to every role and user, so there is no need to grant it through Terraform. If you have an issue with removing the resources please use `terraform state rm <id>` to remove the resource from the state (and you can safely remove the configuration).

### Issues with grant_ownership

Please read our [guide for grant_ownership](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/guides/grant_ownership_common_use_cases) resource.
It contains common use cases and issues that you may encounter when dealing with ownership transfers.

### Using QUOTED_IDENTIFIERS_IGNORE_CASE with the provider

**Problem:** When `QUOTED_IDENTIFIERS_IGNORE_CASE` parameter is set to true, but resource identifier fields are filled with lowercase letters,
during `terrform apply` they may fail with the `Error: Provider produced inconsistent result after apply` error (removing themselves from the state in the meantime).

**Related issues:** [#2967](https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2967)

**Solution:** Either turn off the parameter or adjust your configurations to use only upper-cased names for identifiers and import back the resources.
Loading
Loading