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

feat: Update target objects for privilege-granting resources #2688

Merged
merged 4 commits into from
Apr 17, 2024

Conversation

sfc-gh-jcieslak
Copy link
Collaborator

@sfc-gh-jcieslak sfc-gh-jcieslak commented Apr 9, 2024

A follow-up for #2604 (comment). Updated target objects for privilege-granting resources based on documentation (also added references above available object's list).

Pr also contains:

Copy link

github-actions bot commented Apr 9, 2024

Integration tests failure for 98fab6734b441a634406db0203a3beace7e139b0

Copy link

github-actions bot commented Apr 9, 2024

Integration tests failure for 8c956be8551f3609f3ad239f549455917bb325b6

pkg/sdk/grants_validations.go Show resolved Hide resolved
pkg/sdk/object_types.go Show resolved Hide resolved
Copy link

Integration tests failure for 311b1c6a503d0d1d551234794905bc44db2c4a58

docs/resources/tag_association.md Outdated Show resolved Hide resolved
pkg/resources/tag_association.go Outdated Show resolved Hide resolved
Copy link

Integration tests failure for d3b5e01efed96acdf8f7665c90ed35dcbd6cee58

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit 74e2b6b into main Apr 17, 2024
8 of 9 checks passed
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the update-privileges-in-grant-resources branch April 17, 2024 11:09
sfc-gh-jcieslak pushed a commit that referenced this pull request Apr 18, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.89.0](v0.88.0...v0.89.0)
(2024-04-18)


### 🎉 **What's new:**

* Update target objects for privilege-granting resources
([#2688](#2688))
([74e2b6b](74e2b6b))


### 🔧 **Misc**

* Handle generic check destroy in acceptance tests
([#2716](#2716))
([63a5324](63a5324))
* Initialize the SDK client in fewer places
([#2710](#2710))
([382bfc1](382bfc1))


### 🐛 **Bug fixes:**

* Adds case statement for ObjectTypeUser in ShowObjectParameter.
([#2675](#2675))
([23a3341](23a3341))
* diffs always occurring when multiple columns exist
([#2686](#2686))
([3275ad4](3275ad4))
* Fix issues
[#2679](#2679)
[#2721](#2721)
([#2723](#2723))
([b0c9dd4](b0c9dd4))
* Fix several small issues
([#2697](#2697))
([e3f6a15](e3f6a15))
* granting ownership on database roles
([#2703](#2703))
([88944e7](88944e7))
* network policy update
([#2647](#2647))
([8126b28](8126b28))
* Rename grants_redesign_desgin_decisions.md to grants_redesign_design_…
([#2691](#2691))
([5000b2b](5000b2b))
* renames in update operations
([#2702](#2702))
([793c879](793c879))
* showbyid method tests
([#2648](#2648))
([ff5e617](ff5e617))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
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.

2 participants