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

data-platform-foundations terraform is broken #1260

Closed
dedeco opened this issue Mar 16, 2023 · 3 comments
Closed

data-platform-foundations terraform is broken #1260

dedeco opened this issue Mar 16, 2023 · 3 comments
Assignees

Comments

@dedeco
Copy link
Member

dedeco commented Mar 16, 2023

Following the README.md steps is not working.

Steps:

$ git clone https://github.com/GoogleCloudPlatform/cloud-foundation-fabric.git
$ cd cloud-foundation-fabric/blueprints/data-solutions/data-platform-foundations
$ [edit terraform.tfvars with your variables]
$ terraform init
$ terraform apply

Vars:

prefix = "dp"
parent = "folders/482502016930"
billing_account_id = "1111111-222222-3333333"
organization_domain = "andresousa.dev"
region = "us-central1"

Error:

admin_@cloudshell:~/cloud-foundation-fabric/blueprints/data-solutions/data-platform-foundations$ terraform plan
var.project_config
Provide 'billing_account_id' value if project creation is needed, uses existing 'project_ids' if null. Parent is in 'folders/nnn' or 'organizations/nnn' format.

Enter a value: "0168EE-77421F-419636"


│ Warning: Value for undeclared variable

│ The root module does not declare a variable named "billing_account_id" but a value was found in file "terraform.tfvars". If you meant to use this value, add a "variable" block to the configuration.

│ To silence these warnings, use TF_VAR_... environment variables to provide certain "global" settings to all configurations in your organization. To reduce the verbosity of these warnings, use the -compact-warnings option.


│ Warning: Value for undeclared variable

│ The root module does not declare a variable named "parent" but a value was found in file "terraform.tfvars". If you meant to use this value, add a "variable" block to the configuration.

│ To silence these warnings, use TF_VAR_... environment variables to provide certain "global" settings to all configurations in your organization. To reduce the verbosity of these warnings, use the -compact-warnings option.


│ Error: Invalid value for input variable

│ on variables.tf line 170:
│ 170: variable "project_config" {

│ Unsuitable value for var.project_config set using an interactive prompt: object required.

Uploading Screenshot 2023-03-16 at 20.53.22.png…

@dedeco
Copy link
Member Author

dedeco commented Mar 17, 2023

I had fixed using this *.tfvars

prefix = "dpl"
project_config = {
parent = "folders/903585169921"
billing_account_id = "0111111-2222222-3333333"
}
organization_domain = "andresousa.dev"
region = "us-central1"

@ludoo
Copy link
Collaborator

ludoo commented Mar 17, 2023

@lcaggio can you take a look and fix documentation?

@ludoo
Copy link
Collaborator

ludoo commented Mar 17, 2023

thanks for reporting and sending the fix!

@ludoo ludoo closed this as completed Mar 17, 2023
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

No branches or pull requests

3 participants