Skip to content

Fast 0-init stage#3701

Closed
apichick wants to merge 7 commits into
masterfrom
fast-0-init
Closed

Fast 0-init stage#3701
apichick wants to merge 7 commits into
masterfrom
fast-0-init

Conversation

@apichick

@apichick apichick commented Feb 4, 2026

Copy link
Copy Markdown
Collaborator

Checklist

I applicable, I acknowledge that I have:

  • Read the contributing guide
  • Ran terraform fmt on all modified files
  • Regenerated the relevant README.md files using tools/tfdoc.py
  • Made sure all relevant tests pass

description = "Flag indicating whether a temporary project needs be created."
type = object({
id = optional(string)
create = optional(bool, true)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this is not used, at the moment.

Comment on lines +61 to +64
"${local.prefix}-init-${random_string.suffix.result}"
)
parent = var.default_project_config.create ? local.parent : null
prefix = local.prefix

@juliocc juliocc Feb 5, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Make sure prefix is handled correctly here. We're setting it explicitely in line 61 and again in line 64.

output "project_id" {
description = "ID of the project to be used as billing project."
value = <<-EOF
x = [

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

update value

output "existing_org_policies" {
description = "Variable containing existing organization policies to be added to the terraform.tfvars of the 0-org-setup stage."
value = <<-EOF
x = [

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

update variable name (I think it's org_policies_import)

terraform output existing_org_policies > terraform.tfvars
```

Now you are ready to go to the `0-org-setup` stage!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

add the default vars/outputs tables.

@juliocc

juliocc commented Mar 20, 2026

Copy link
Copy Markdown
Collaborator

@apichick what's the plan with this PR?

@juliocc juliocc closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants