Skip to content

Commit

Permalink
fix(bigquery-dataset): Set dataset_id with underscores
Browse files Browse the repository at this point in the history
  • Loading branch information
anthonyhaussman committed Apr 9, 2024
1 parent a93b3d3 commit 39c00d3
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 16 deletions.
10 changes: 5 additions & 5 deletions modules/bigquery-dataset/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ The access variables are split into `access` and `access_identities` variables,
module "bigquery-dataset" {
source = "./fabric/modules/bigquery-dataset"
project_id = "my-project"
id = "my-dataset"
id = "my_dataset"
access = {
reader-group = { role = "READER", type = "group" }
owner = { role = "OWNER", type = "user" }
Expand All @@ -46,7 +46,7 @@ Access configuration can also be specified via IAM instead of basic roles via th
module "bigquery-dataset" {
source = "./fabric/modules/bigquery-dataset"
project_id = "my-project"
id = "my-dataset"
id = "my_dataset"
iam = {
"roles/bigquery.dataOwner" = ["user:[email protected]"]
}
Expand Down Expand Up @@ -143,7 +143,7 @@ Authorized views can be specified both using the standard `access` options and t
module "bigquery-dataset" {
source = "./fabric/modules/bigquery-dataset"
project_id = "my-project"
id = "my-dataset"
id = "my_dataset"
authorized_views = [
{
project_id = "view_project"
Expand Down Expand Up @@ -176,7 +176,7 @@ Dataset options are set via the `options` variable. all options must be specifie
module "bigquery-dataset" {
source = "./fabric/modules/bigquery-dataset"
project_id = "my-project"
id = "my-dataset"
id = "my_dataset"
options = {
default_table_expiration_ms = 3600000
default_partition_expiration_ms = null
Expand Down Expand Up @@ -227,7 +227,7 @@ locals {
module "bigquery-dataset" {
source = "./fabric/modules/bigquery-dataset"
project_id = "my-project"
id = "my-dataset"
id = "my_dataset"
tables = {
table_a = {
deletion_protection = true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

values:
module.bigquery-dataset.google_bigquery_dataset.default:
dataset_id: my-dataset
dataset_id: my_dataset
project: my-project
module.bigquery-dataset.google_bigquery_dataset_access.authorized_views["view_project_view_dataset_view_id_1"]:
view:
Expand Down
4 changes: 2 additions & 2 deletions tests/modules/bigquery_dataset/examples/iam.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@

values:
module.bigquery-dataset.google_bigquery_dataset.default:
dataset_id: my-dataset
dataset_id: my_dataset
project: my-project
module.bigquery-dataset.google_bigquery_dataset_iam_binding.bindings["roles/bigquery.dataOwner"]:
condition: []
dataset_id: my-dataset
dataset_id: my_dataset
members:
- user:[email protected]
project: my-project
Expand Down
2 changes: 1 addition & 1 deletion tests/modules/bigquery_dataset/examples/options.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

values:
module.bigquery-dataset.google_bigquery_dataset.default:
dataset_id: my-dataset
dataset_id: my_dataset
default_partition_expiration_ms: null
default_table_expiration_ms: 3600000
delete_contents_on_destroy: false
Expand Down
2 changes: 1 addition & 1 deletion tests/modules/bigquery_dataset/examples/partitioning.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

values:
module.bigquery-dataset.google_bigquery_dataset.default:
dataset_id: my-dataset
dataset_id: my_dataset
location: EU
project: my-project
module.bigquery-dataset.google_bigquery_table.default["table_a"]:
Expand Down
12 changes: 6 additions & 6 deletions tests/modules/bigquery_dataset/examples/simple.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,30 +14,30 @@

values:
module.bigquery-dataset.google_bigquery_dataset.default:
dataset_id: my-dataset
dataset_id: my_dataset
description: Terraform managed.
location: EU
project: my-project
module.bigquery-dataset.google_bigquery_dataset_access.group_by_email["reader-group"]:
dataset_id: my-dataset
dataset_id: my_dataset
group_by_email: [email protected]
project: my-project
role: READER
module.bigquery-dataset.google_bigquery_dataset_access.special_group["project_owners"]:
dataset_id: my-dataset
dataset_id: my_dataset
project: my-project
role: OWNER
special_group: projectOwners
module.bigquery-dataset.google_bigquery_dataset_access.user_by_email["owner"]:
dataset_id: my-dataset
dataset_id: my_dataset
project: my-project
role: OWNER
user_by_email: [email protected]
module.bigquery-dataset.google_bigquery_dataset_access.authorized_views["my-project_my-dataset_my-table"]:
dataset_id: my-dataset
dataset_id: my_dataset
project: my-project
view:
- dataset_id: my-dataset
- dataset_id: my_dataset
project_id: my-project
table_id: my-table

Expand Down

0 comments on commit 39c00d3

Please sign in to comment.