Skip to content

Follow the bosh-deployment default branch - #379

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

aramprice merged 1 commit into
mainfrom
codex/bosh-deployment-main

Conversation

@aramprice

@aramprice aramprice commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Remove the explicit master selector from read-only bosh-deployment Git sources, allowing them to follow the repository default before and after its rename. These changes can be merged and applied while the default is still master.

Concourse Git resource contract: a read-only source can follow the default by omitting branch; a write source requires an explicit branch.

Validation: git diff --check passed. Exact parsed before/after comparison confirms only the intended Git source branch selectors changed. Modern strict offline pipeline validation reports the same existing template/legacy-schema errors before and after; runtime compatibility requires operator verification.

Only checked-in definitions are changed. Deployed pipeline reconfiguration and runtime checks remain operator-owned. No Concourse pause/unpause, configuration, cache clearing, resource checks, job triggers or configure-script execution were performed.

Cutover steps, consumer readiness and rollback: cloudfoundry/bosh-deployment#539

Community configuration (draft): cloudfoundry/community#1686

@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 pipeline no longer specifies the master branch for the bosh-deployment Git resource. The resource now uses its default branch.

Merge Risk: Minimal · up to fab10

The pipeline can follow the repository’s default branch as intended. No merge-blocking issue was identified in the checked-in configuration.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title says the pipeline will follow the bosh-deployment default branch. The change removes the explicit branch: master line from the bosh-deployment Git resource in ci/pipeline.yml, so the resou…
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 states the PR removes the explicit master selector from read-only bosh-deployment Git sources so they follow the repository default. It also gives validation steps and links to cut…
Full details: Title check

Explanation

The title says the pipeline will follow the bosh-deployment default branch. The change removes the explicit branch: master line from the bosh-deployment Git resource in ci/pipeline.yml, so the resource uses the repository default. The title matches the main change and is specific.

Full details: Description check

Explanation

The description states the PR removes the explicit master selector from read-only bosh-deployment Git sources so they follow the repository default. It also gives validation steps and links to cutover and community documentation. This matches the changeset, which removes branch: master from the bosh-deployment Git resource in ci/pipeline.yml. The description is related to the changes, so the check passes.

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
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: b4839af0-28ff-40fc-8623-771da99b5b3b
Commits

Reviewing files that changed from the base of the PR and between 12b7812 and fab1002.

Files selected for processing: 1
  • ci/pipeline.yml
Files with no reviewable changes: 1
  • ci/pipeline.yml

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

@aramprice
aramprice merged commit 2c628da into main Nov 3, 2026
5 checks passed
@aramprice
aramprice deleted the codex/bosh-deployment-main branch November 3, 2026 23:23
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.

1 participant