Skip to content

CI: rename GCP subnets used for integration specs#564

Merged
aramprice merged 1 commit into
ubuntu-jammyfrom
rename-integration-subnets
Apr 20, 2026
Merged

CI: rename GCP subnets used for integration specs#564
aramprice merged 1 commit into
ubuntu-jammyfrom
rename-integration-subnets

Conversation

@aramprice

@aramprice aramprice commented Apr 20, 2026

Copy link
Copy Markdown
Member

Pipeline already reconfigured.

Copilot AI review requested due to automatic review settings April 20, 2026 18:56
@coderabbitai

coderabbitai Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 89358074-4cf9-4b79-b072-2df42e4a388b

📥 Commits

Reviewing files that changed from the base of the PR and between e73c878 and e079e55.

📒 Files selected for processing (2)
  • ci/pipelines/builder.yml
  • ci/pipelines/vars.yml

Walkthrough

This change updates GCP infrastructure naming in pipeline configuration files. The ci/pipelines/builder.yml file has subnet naming conventions updated from bosh-integration- prefix to stemcell-builder-integration- prefix across multiple director deploy tasks and bats configuration parameters. Additionally, ci/pipelines/vars.yml updates the stemcell_details.subnet_int value from "3" to "22", with an inline comment indicating this should reflect the last two digits of the release year.

Suggested reviewers

  • rkoster
  • benjaminguttmann-avtq
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a description explaining the purpose and impact of the GCP subnet renaming changes to provide context for reviewers.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: renaming GCP subnet naming conventions from 'bosh-integration-...' to 'stemcell-builder-integration-...' across pipeline configuration files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rename-integration-subnets

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aramprice aramprice merged commit b62f580 into ubuntu-jammy Apr 20, 2026
16 checks passed
@aramprice aramprice deleted the rename-integration-subnets branch April 20, 2026 18:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Renames the GCP subnets used by the CI integration/bats pipelines to a new stemcell-builder-integration-* naming scheme, and updates the subnet index used to derive CIDRs/IPs.

Changes:

  • Update stemcell_details.subnet_int from "3" to "22" (used for IP/CIDR derivation).
  • Rename GCP_SUBNET_NAME / VARS_SUBNETWORK_DEFAULT values from bosh-integration-* to stemcell-builder-integration-* (including ipv6 variant).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ci/pipelines/vars.yml Updates the subnet index (subnet_int) used across GCP integration jobs.
ci/pipelines/builder.yml Renames the GCP subnet names referenced by integration and BATs jobs (ipv4 + ipv6).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci/pipelines/vars.yml
Comment thread ci/pipelines/builder.yml
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