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

Add PNA support to Service Directory module #1960

Merged
merged 2 commits into from
Jan 5, 2024
Merged

Add PNA support to Service Directory module #1960

merged 2 commits into from
Jan 5, 2024

Conversation

stribioli
Copy link
Collaborator

Endpoints in Service Directory can be associated with a VPC. In this case, they can be used by supported Google Cloud products to send requests directly to resources inside a VPC. This feature is called Private Network Access.

The google_service_directory_endpoint resource supports this configuration with a new argument network.
Unfortunately, this argument has an unusual format: it is similar to a standard VPC ID, but instead of the project ID, it expects the project number.


Checklist

If 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

Copy link
Collaborator

@ludoo ludoo left a comment

Choose a reason for hiding this comment

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

Thanks!

modules/service-directory/README.md Outdated Show resolved Hide resolved
@stribioli stribioli enabled auto-merge January 5, 2024 14:57
@stribioli stribioli disabled auto-merge January 5, 2024 14:58
Endpoints in Service Directory can be *associated* with a
VPC. In this case, they can be used by supported Google
Cloud products to send requests directly to resources inside
a VPC. This feature is called Private Network Access.

The `google_service_directory_endpoint` resource supports
this configuration with a new argument `network`.
Unfortunately, this argument has an unusual format: it
is similar to a standard VPC ID, but instead of the project ID,
it expects the project number.
@stribioli stribioli enabled auto-merge January 5, 2024 15:05
@stribioli stribioli merged commit e112810 into GoogleCloudPlatform:master Jan 5, 2024
9 checks passed
@stribioli stribioli deleted the stribioli/sd-pna branch January 5, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants