Skip to content

Follow AWS CPI and certification default branches (ubuntu-jammy) - #788

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

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

Conversation

@aramprice

@aramprice aramprice commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

The read-only AWS CPI and CPI certification Git sources will follow their repositories' default branches. Both selectors can be applied while those defaults are still master and will follow main after the renames.

Validation: git diff --check passed. Parsed before/after comparisons confirm only the intended Git branch selectors changed. Local fly validate-pipeline --strict passed on the rendered YTT template.

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 bosh-cpi-certification-concourse-tasks and bosh-aws-cpi-release Git resources no longer specify branch: master. They now use their repositories’ default branches.

Suggested reviewers: mkocher

Merge Risk: Minimal · up to 75353

The read-only AWS CPI and certification sources will follow their repositories’ default branches. No current incompatibility is established; the change is mergeable subject to normal validation and operator-owned pipeline reconfiguration.

Pre-merge checks: Passed 3 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description clearly explains the change, validation results, scope, operator-owned work, and cutover plans. It does not include the required Merge Forward section with the target branch and merge-… Add the required Merge Forward section. State the oldest applicable target branch for this PR and confirm the merge-forward chain. Add the AI Review Feedback section and resolve or explain any applicable AI review comments.
Passed Passed checks: 3 passed
Check name Status Explanation
Title check Passed The title names two real changes in the diff: the AWS CPI and CPI certification Git resources now follow default branches instead of pinning master. It omits the purpose and the ubuntu-jammy reference…
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.
Full details: Title check

Explanation

The title names two real changes in the diff: the AWS CPI and CPI certification Git resources now follow default branches instead of pinning master. It omits the purpose and the ubuntu-jammy reference is not clearly tied to the diff, but it describes the main change accurately and is specific.

Full details: Description check

Explanation

The description clearly explains the change, validation results, scope, operator-owned work, and cutover plans. It does not include the required Merge Forward section with the target branch and merge-forward chain, or the AI Review Feedback section from the repository template.

Resolution

Add the required Merge Forward section. State the oldest applicable target branch for this PR and confirm the merge-forward chain. Add the AI Review Feedback section and resolve or explain any applicable AI review comments.

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

Warning

Some tools did not complete. Review the errors below.

ast-grep (0.51.1)

ast-grep could not parse rule config: /ast-grep-rules/ast-grep-essentials/rules/ruby/security/dynamic-constantize-params-ruby.yml

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: 69b5f35a-0615-46f3-8fd1-0858427bd20d
Commits

Reviewing files that changed from the base of the PR and between 25f9800 and 75353fd.

Files selected for processing: 1
  • ci/pipeline-template.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.

@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 e5877f8 into ubuntu-jammy Nov 3, 2026
10 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 15:59
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