Skip to content

Follow jumpbox default branch and branch-neutral APT key URLs - #699

Merged
aramprice merged 1 commit into
mainfrom
codex/five-bosh-defaults-main
Nov 3, 2026
Merged

aramprice merged 1 commit into
mainfrom
codex/five-bosh-defaults-main

Conversation

@aramprice

@aramprice aramprice commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

The read-only jumpbox Git source will follow its repository's default branch. APT and RPM signing-key URLs use HEAD, which works before and after the APT repository rename.

Validation: git diff --check passed. Parsed before/after comparisons confirm only the intended Git branch selectors changed. Modern local strict pipeline validation reports the same pre-existing template/legacy-schema errors before and after this patch; it does not establish runtime compatibility. HTTP downloads of both APT/RPM key URLs through HEAD returned HTTP 200 and matched the corresponding master bytes.

Default-following consumer changes can be merged and, where deployed, applied by their operators before the target rename.

Concourse Git resource contract: a read-only source can omit branch; a source used for put needs an explicit branch.

This PR changes checked-in files only. Deployed pipeline reconfiguration and runtime checks remain operator-owned; no Concourse pipelines or jobs were modified, paused, unpaused, or triggered for this work.

Cutover plans:

Shared community configuration (draft): cloudfoundry/community#1682

@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Walkthrough

The Ubuntu APT and Fedora/RHEL YUM signing-key URLs in the README now use HEAD instead of master. The jumpbox-deployment Git resource no longer specifies a branch; its repository URI remains configured.

Merge Risk: Minimal · up to 59a75

The branch-neutral Git and signing-key references introduce no identified merge-blocking risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly and concisely describes both main changes: following the jumpbox repository default branch and using branch-neutral APT key URLs.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description directly matches the changes. It explains the jumpbox Git source branch change and the APT/RPM signing-key URL updates. It also documents validation results and deployment scope.
Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 831b76e9-2d15-4b5e-9a89-ea828cda7fac
Commits

Reviewing files that changed from the base of the PR and between 4174de6 and 59a75f7.

Files selected for processing: 2
  • README.md
  • ci/pipelines/bosh-bootloader.yml
Files with no reviewable changes: 1
  • ci/pipelines/bosh-bootloader.yml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

@github-project-automation github-project-automation Bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Nov 3, 2026
@aramprice
aramprice merged commit f2bfb4d into main Nov 3, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Nov 3, 2026
@aramprice
aramprice deleted the codex/five-bosh-defaults-main branch November 3, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant