Minimal Data Platform - Fix (#1730)

* Fix * Fix README * Fix data platform minimal tests --------- Co-authored-by: Julio Castillo <jccb@google.com>
GoogleCloudPlatform · Oct 4, 2023 · de88395 · de88395
1 parent 20dd3df
commit de88395
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/blueprints/data-solutions/data-platform-minimal/02-composer.tf b/blueprints/data-solutions/data-platform-minimal/02-composer.tf
@@ -24,6 +24,7 @@ locals {
     DP_REGION          = var.region
     LAND_PRJ           = module.land-project.project_id
     LAND_GCS           = module.land-cs-0.url
+    LAND_BQ_DATASET    = module.land-bq-0.dataset_id
     PHS_CLUSTER_NAME   = try(module.processing-dp-historyserver[0].name, "")
     PROCESSING_GCS     = module.processing-cs-0.url
     PROCESSING_PRJ     = module.processing-project.project_id

diff --git a/blueprints/data-solutions/data-platform-minimal/02-processing.tf b/blueprints/data-solutions/data-platform-minimal/02-processing.tf
@@ -105,9 +105,7 @@ module "processing-project" {
   iam_bindings_additive = (
     var.project_config.billing_account_id != null ? {} : local.iam_prc_additive
   )
-  compute_metadata = {
-    enable-oslogin = "false"
-  }
+
   services = [
     "bigquery.googleapis.com",
     "bigqueryreservation.googleapis.com",

diff --git a/blueprints/data-solutions/data-platform-minimal/README.md b/blueprints/data-solutions/data-platform-minimal/README.md
@@ -229,7 +229,7 @@ module "data-platform" {
   prefix = "myprefix"
 }
 
-# tftest modules=23 resources=138
+# tftest modules=23 resources=137
 ```
 
 ## Customizations