fix: remove Cirrus CI persistent worker integration - #883
Open
deekay2310 wants to merge 1 commit into
Open
Conversation
…#859) Cirrus CI shut down on June 1, 2026. Remove all dead code: the pkg/integrations/cirrus package, CLI flags (--it-cirrus-pw-token, --it-cirrus-pw-labels), port 3010 security group rules, template snippets, and documentation references. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (35)
💤 Files with no reviewable changes (31)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe pull request removes Cirrus persistent-worker support from command wiring, shared integration management, cloud provisioning, security-group configuration, embedded setup scripts, and documentation. GitHub Actions and GitLab Runner support remain. ChangesCirrus integration removal
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #859.
Cirrus CI shut down on June 1, 2026 when Cirrus Labs joined OpenAI. This PR removes all dead Cirrus CI persistent worker code from the codebase:
pkg/integrations/cirrus/— deleted entirely (Go package, shell/ps1 setup snippets)--it-cirrus-pw-tokenand--it-cirrus-pw-labelsremoved from all 8 command files (AWS rhel/fedora/windows/mac-pool, Azure rhel/windows, IBM power/z) and fromparams.goCirrusPWArgsfield removed fromContextArgsand integration initbootstrap.ps1, Azurerhqp-ci-setup.ps1(including$cirrusTokenparam), and Macrequest.shcirrusTokenremoved from powershell command format stringsibm-power.mdandibm-z.mdmac-pool.gothat listedcirrus-persistent-workeras a future integration mode; this design direction was superseded by the current context-args pattern where integrations are selected viaGHRunnerArgs/GLRunnerArgsfields35 files changed, 437 lines removed. No functional changes to remaining integrations (GitHub Actions, GitLab Runner).
Test plan
go build ./...passes — pure dead-code removal, no logic changes to remaining integrations🤖 Generated with Claude Code