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.

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

fix: Deleting a snowflake_user and their associated snowlfake_role_grant causes an error #1142

Merged
merged 2 commits into from
Aug 10, 2022

Conversation

mwiewior
Copy link
Contributor

@mwiewior mwiewior commented Jul 18, 2022

This is to fix #889 by checking a Snowflake SQL error code and printing a warning instead of failing in case of non-existing user.

Test Plan

  • [ x] acceptance tests

References

@mwiewior mwiewior marked this pull request as draft July 19, 2022 05:50
@mwiewior mwiewior marked this pull request as ready for review July 19, 2022 06:12
@mwiewior mwiewior force-pushed the issue-889 branch 3 times, most recently from e2aa86b to 7b3cc25 Compare July 19, 2022 09:08
@mwiewior
Copy link
Contributor Author

Hi @sfc-gh-swinkler - could you pls kick off gh actions?

@mwiewior
Copy link
Contributor Author

Hi @sfc-gh-jalin maybe you could take a look as well - this is sth that really makes it hard to manage users/roles from Terraform and already a few persons have upvoted this issue as well. Thx!

@sfc-gh-jlove
Copy link
Contributor

Hi @mwiewior

We plan to take a look at this but we currently have very low bandwidth on the team.

@mwiewior
Copy link
Contributor Author

ok, thx - just ping me if can help you somehow

@mwiewior
Copy link
Contributor Author

mwiewior commented Aug 8, 2022

@sfc-gh-jlove hey any chance you will have some time for this one ?

@sfc-gh-swinkler
Copy link
Collaborator

@mwiewior sorry i was on grievance leave. my wife died last month and there wasn't anyone else on the team who could approve PRs. i am catching up now.

@sfc-gh-swinkler
Copy link
Collaborator

/ok-to-test sha=7a986e6

@sfc-gh-swinkler
Copy link
Collaborator

thank you for your contribution

@sfc-gh-swinkler sfc-gh-swinkler merged commit 5f6725a into Snowflake-Labs:main Aug 10, 2022
@github-actions
Copy link

Integration tests success for 7a986e6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deleting a snowflake_user and their associated snowlfake_role_grant causes an error
3 participants