Skip to content

Enhance namespace handling, update GCS service account script, minor fixes to other provided script examples#20

Closed
drpotters wants to merge 5 commits into
f5devcentral:mainfrom
drpotters:main
Closed

Enhance namespace handling, update GCS service account script, minor fixes to other provided script examples#20
drpotters wants to merge 5 commits into
f5devcentral:mainfrom
drpotters:main

Conversation

@drpotters

Copy link
Copy Markdown
Contributor

Proposed changes

This pull request updates the GCP onboarding documentation and improves the Terraform configuration for F5 XC namespaces. The most important changes are grouped below by theme.

GCP Documentation Improvements:

  • Enhanced the setup instructions in docs/ADSP-UC1-GCP.md to:
    • Use parameterized variables for service account and pool names, and add explicit steps for creating and granting roles to the service account.
    • Add an attribute condition to the Workload Identity Provider to restrict access to a specific GitHub organization.
    • Use gcloud storage cat instead of gsutil cat for retrieving state files, and combine related outputs (e.g., BIG-IP IP and password, internal IPs) into single commands for clarity. [1] [2]
    • Correct the BIG-IP GUI port in usage instructions from 443 to 8443.

Terraform Namespace Handling:

  • Updated f5/xc/namespace.tf to:
    • Add logic for detecting and using an existing Volterra namespace, or creating one if it does not exist, via a new null_resource and data source. This prevents unnecessary resource creation and state changes.
    • Add the null provider to f5/xc/versions.tf to support the new resource logic.

…create namespace if it already exists. Only remove namespace in XC if we created it. This allows us to preserve the namespace if it was created outside of this use case deployment. This is important because if we delete a namespace that was created outside of XC, we could potentially delete resources that are still in use by other tools or users.
Fix to verification and testing to return the correct public_ip.
Change all gsutil references to use gcloud storage.

@knowbase knowbase left a comment

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.

Move the data resource into data.tf

Removed data block for volterra_namespace.
@drpotters drpotters closed this Jun 16, 2026
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.

2 participants