Skip to content

Conversation

@jhanavi
Copy link
Contributor

@jhanavi jhanavi commented Nov 10, 2025

No description provided.

@jhanavi jhanavi requested review from a team and imrannayer as code owners November 10, 2025 08:09
@apeabody
Copy link
Contributor

Thanks! After this PR is merged, you will want to open a PR to add:

  lint_env = {
    ENABLE_BPMETADATA = "1"
  }

at https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/blob/b0c6241ea40cd940b153aa3c1e64b53711b4fde0/infra/terraform/test-org/org/locals.tf for this repo

@apeabody
Copy link
Contributor

/gcbrun

@jhanavi
Copy link
Contributor Author

jhanavi commented Nov 11, 2025

Thanks! After this PR is merged, you will want to open a PR to add:

  lint_env = {
    ENABLE_BPMETADATA = "1"
  }

at https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/blob/b0c6241ea40cd940b153aa3c1e64b53711b4fde0/infra/terraform/test-org/org/locals.tf for this repo

Thank you! Looks like it's already enabled in the file you pointed for this repo. See line 467.

@apeabody
Copy link
Contributor

Thanks! After this PR is merged, you will want to open a PR to add:

  lint_env = {
    ENABLE_BPMETADATA = "1"
  }

at https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/blob/b0c6241ea40cd940b153aa3c1e64b53711b4fde0/infra/terraform/test-org/org/locals.tf for this repo

Thank you! Looks like it's already enabled in the file you pointed for this repo. See line 467.

Great! Ahh, I see metadata was already enabled, this PR is just adding display metadata. Thanks!

@q2w q2w enabled auto-merge (squash) November 12, 2025 04:08
Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

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

Approved by @q2w

@q2w q2w merged commit 06432c0 into terraform-google-modules:main Nov 12, 2025
4 checks passed
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

Successfully merging this pull request may close these issues.

4 participants