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

Grant read access to all free and full-access datasets to user's BQ connection #16266

Conversation

rafatower
Copy link
Contributor

@rafatower rafatower commented Apr 20, 2021

@shortcut-integration
Copy link

@rafatower rafatower force-pushed the feature/ch148637/grant-read-access-to-all-free-and-full-access branch from 7ddeff9 to 782f0ff Compare April 20, 2021 10:52
@rafatower rafatower force-pushed the feature/ch148637/grant-read-access-to-all-free-and-full-access branch from 782f0ff to ebe5ee0 Compare April 20, 2021 13:01
@rafatower rafatower marked this pull request as ready for review April 20, 2021 14:04
@rafatower
Copy link
Contributor Author

Some traces from tests in staging:
image

image

image

(please note my oauth user is not the best for tests because it has too many privileges)

@rafatower
Copy link
Contributor Author

rafatower commented Apr 20, 2021

In my tests everything worked as expected (oauth and service account), but permissions can take a little bit to propagate in GCP (up to a few minutes).

Copy link
Contributor

@jgoizueta jgoizueta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏁
I was a little confused at first and thought that this might fit better as part of the spatial extension setup but now it make sense to me as it is. 👍

@rafatower rafatower merged commit b7a0230 into master Apr 21, 2021
@rafatower rafatower deleted the feature/ch148637/grant-read-access-to-all-free-and-full-access branch April 21, 2021 06:37
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