Skip to content

remove_ruby_references_from_ci - #361

Merged
neddp merged 1 commit into
cloudfoundry:masterfrom
dudejas:remove-ruby-cpi
Sep 9, 2026
Merged

remove_ruby_references_from_ci#361
neddp merged 1 commit into
cloudfoundry:masterfrom
dudejas:remove-ruby-cpi

Conversation

@dudejas

@dudejas dudejas commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Remove Ruby CPI jobs and workflows from CI

The Go CPI is now the default in bosh-deployment#528 and has been the active implementation for all known landscapes. This PR stops exercising the Ruby CPI in CI — no source is deleted yet, just the CI infrastructure that tests it.

Changes

  • Removed job run-devstack-tests-ruby-cpi — ran lifecycle tests and BATS with USE_GOLANG_CPI: "false"; redundant now that the Go path is the only supported one
  • Removed job bump-ruby-deps — bumped Ruby gems in src/bosh_openstack_cpi; no longer needed
  • Simplified bump-bosh-packages — removed the bump-ruby-package task (ruby-3.4) and the Ruby run-unit-specs step; only bump-golang-packages + run-golang-unit-specs remain
  • Fixed passed: gatespromote-candidate and automatically-release-new-patch now gate solely on run-devstack-tests (Go)
  • Removed unused resources ruby-release and bosh-ruby-release-registry-image
  • Deleted .github/workflows/ruby.yml — triggered on src/bosh_openstack_cpi/** changes

The pipeline now has zero references to Ruby. Go jobs (run-specs, run-devstack-tests, bump-go-deps, bump-bosh-packages) are unaffected.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: bff57fea-8e6f-4c7d-bbf5-db41bd0fe893

📥 Commits

Reviewing files that changed from the base of the PR and between dfe8a0c and 3a24d19.

📒 Files selected for processing (2)
  • .github/workflows/ruby.yml
  • ci/pipeline.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/ruby.yml

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


Walkthrough

Removed Ruby workflow validation and Ruby CPI DevStack testing. Release and candidate promotion gates now use the Go DevStack tests. Dependency bumping and package updates now run Go checks only. Ruby release resources and Ruby-specific pipeline jobs were removed.

Suggested reviewers: neddp

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 3a24d

Ruby CI and packaging automation are removed while release promotion retains Go validation. No actionable merge-blocking risk is currently identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so the relationship between the changes and their purpose cannot be assessed from the description. Add a brief description that explains the removal of Ruby CI workflows, jobs, dependencies, and release resources.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Title check ✅ Passed The title clearly summarizes the primary change: removing Ruby references from CI configuration and workflows.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@dudejas
dudejas marked this pull request as ready for review September 9, 2026 13:15
@neddp
neddp merged commit efc59ed into cloudfoundry:master Sep 9, 2026
5 checks passed
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