Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: grantsForUser for Global Resources #5612

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

Open
wants to merge 18 commits into
base: bosorawis-domain-iam-implement-role-grant-scopes-all
Choose a base branch
from

Conversation

elimt
Copy link

@elimt elimt commented Mar 18, 2025

add query to fetch grants for a user for resources that are only globally scoped

Note:
DO NOT MERGE until we can create roles by subType from domain code instead of using setupDB_IamRoles to manually insert. setupDB_IamRoles was used to just validate the query

@elimt elimt added this to the 0.20.x milestone Mar 18, 2025
@elimt elimt requested review from tmessi, bosorawis and dkanney March 18, 2025 16:13
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from c05c24c to ab93354 Compare April 4, 2025 03:00
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from 188ff50 to f577249 Compare April 9, 2025 19:54
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from 01e1975 to c3f45b3 Compare April 9, 2025 20:01
@dkanney dkanney changed the base branch from llb-normalized-grants to bosorawis-sql-split-global-grants-scope-table April 9, 2025 20:01
@dkanney dkanney self-assigned this Apr 9, 2025
@dkanney dkanney marked this pull request as ready for review April 11, 2025 14:18
@dkanney dkanney requested a review from a team as a code owner April 11, 2025 14:18
@bosorawis bosorawis force-pushed the bosorawis-sql-split-global-grants-scope-table branch from 3b3bfca to bbe7bdb Compare April 14, 2025 23:41
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from 122dfb1 to 0508ce5 Compare April 15, 2025 19:45
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from 1280926 to 4f104bb Compare April 21, 2025 21:06
@dkanney dkanney changed the base branch from bosorawis-sql-split-global-grants-scope-table to bosorawis-domain-iam-role-use-new-model April 21, 2025 21:06
Copy link

Database schema diff between bosorawis-sql-split-global-grants-scope-table and elimt-grantsForUser-global-only @ 4f104bb

To understand how these diffs are generated and some limitations see the
documentation of the script.

Functions

Unchanged

Tables

Unchanged

Views

Unchanged

Triggers

Unchanged

Indexes

Unchanged

Constraints

Unchanged

Foreign Key Constraints

Unchanged

@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch 2 times, most recently from 54033ff to 90d7ee5 Compare April 22, 2025 14:52
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch 2 times, most recently from 7c6ab79 to 39dd771 Compare May 1, 2025 22:51
@dkanney dkanney changed the base branch from llb-normalized-grants to bosorawis-domain-iam-implement-role-grant-scopes-all May 1, 2025 23:01
@dkanney
Copy link
Collaborator

dkanney commented May 1, 2025

ℹ️ Changed the base branch to bosorawis-domain-iam-implement-role-grant-scopes-all to get Bo's grant_scope changes

@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from b6e09f7 to e87029d Compare May 2, 2025 07:01
dkanney added 6 commits May 2, 2025 19:29
Remove canonical_grant filter from query. `iam_grant.canonical_grant` is a primary key, so it can't be null anyway -- no need to filter out null canonical grants
No longer using a hard-coded value for roleVersion
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch 3 times, most recently from 84c95bf to 60181c3 Compare May 5, 2025 23:29
@dkanney dkanney requested a review from tmessi May 5, 2025 23:33
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch 2 times, most recently from 5235b09 to 1119387 Compare May 6, 2025 15:22
@dkanney dkanney force-pushed the elimt-grantsForUser-global-only branch from 1119387 to 118a6dd Compare May 6, 2025 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants