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

DBT requires dataset.update permission when auth view is configured via terraform #2911

Closed
4 tasks
philip-hope opened this issue Nov 24, 2020 · 0 comments
Closed
4 tasks
Labels
bug Something isn't working

Comments

@philip-hope
Copy link

Describe the bug

Access for authorised views is assigned via terraform, when a scheduled DBT task runs our scheduler errors with the below, that DBT doesn't have datasets.update permission. I'm opening the request to enquire as to why DBT should need to have this specific permission if it (the grant_access_to configuration) isn't managed in the view itself

Steps To Reproduce

n/a

Expected behaviour

Since the access is configured in terraform, and the view built in DBT, i would expect that DBT would not need to re-update any access in bigquery. Does DBT require datasets.update to perform a run against an auth view in DBT?

Screenshots and log output

INFO - b'\x1b[33mDatabase Error in model deleted_user_ids (PATH/TO/MODEL)
INFO - b'  Access Denied: Dataset [PROJECT:DATASET]: User does not have bigquery.datasets.update permission for dataset [PROJECT:DATASET]

System information

Which database are you using dbt with?

  • postgres
  • redshift
  • [*] bigquery
  • snowflake
  • other (specify: ____________)

The output of dbt --version:

0.18.1

Additional context

@philip-hope philip-hope added bug Something isn't working triage labels Nov 24, 2020
@jtcohen6 jtcohen6 removed the triage label Nov 24, 2020
gshank added a commit that referenced this issue Aug 14, 2023
gshank added a commit that referenced this issue Jan 16, 2024
* Initial implementation of unit testing (from pr #2911)

Co-authored-by: Michelle Ark <[email protected]>

* 8295 unit testing artifacts (#8477)

* unit test config: tags & meta (#8565)

* Add additional functional test for unit testing selection, artifacts, etc (#8639)

* Enable inline csv format in unit testing (#8743)

* Support unit testing incremental models (#8891)

* update unit test key: unit -> unit-tests (#8988)


* convert to use unit test name at top level key (#8966)

* csv file fixtures (#9044)

* Unit test support for `state:modified` and `--defer` (#9032)

Co-authored-by: Michelle Ark <[email protected]>

* Allow use of sources as unit testing inputs (#9059)

* Use daff for diff formatting in unit testing (#8984)

* Fix #8652: Use seed file from disk for unit testing if rows not specified in YAML config (#9064)

Co-authored-by: Michelle Ark <[email protected]>
Fix #8652: Use seed value if rows not specified

* Move unit testing to test and build commands (#9108)

* Enable unit testing in non-root packages (#9184)

* convert test to data_test (#9201)

* Make fixtures files full-fledged members of manifest and enable partial parsing (#9225)

* In build command run unit tests before models (#9273)

---------

Co-authored-by: Michelle Ark <[email protected]>
Co-authored-by: Michelle Ark <[email protected]>
Co-authored-by: Emily Rockman <[email protected]>
Co-authored-by: Jeremy Cohen <[email protected]>
Co-authored-by: Kshitij Aranke <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants