Skip to content

Migrate from deprecated oc-installer to openshift-tools-installer - #2763

Open
snecklifter wants to merge 1 commit into
stackrox:mainfrom
snecklifter:migrate-oc-installer
Open

Migrate from deprecated oc-installer to openshift-tools-installer#2763
snecklifter wants to merge 1 commit into
stackrox:mainfrom
snecklifter:migrate-oc-installer

Conversation

@snecklifter

@snecklifter snecklifter commented Jul 27, 2026

Copy link
Copy Markdown

Summary

  • Replace redhat-actions/oc-installer@v1 with redhat-actions/openshift-tools-installer@v2 in the multicluster e2e workflow

Why

redhat-actions/oc-installer is deprecated:

  • The download mirror it points to has not been updated since December 2020
  • The action now emits a deprecation warning at runtime (redhat-actions/oc-installer#32)

openshift-tools-installer is the actively maintained successor, supporting semver ranges, arm64, cross-workflow caching, and 18+ tools.

Test plan

  • Verify the multicluster e2e workflow installs oc successfully with the new action

PR summary

This PR replaces redhat-actions/oc-installer@v1 with redhat-actions/openshift-tools-installer@v2 in the multicluster end-to-end workflow. The change addresses the deprecation of oc-installer, whose download mirror has not been updated since December 2020. The replacement action is actively maintained and supports semver ranges, arm64, cross-workflow caching, and more than 18 tools.

Test plan

Verify that the multicluster end-to-end workflow successfully installs oc with the new action.


The following was generated by @coderabbitai and may be updated automatically.

Summary

Updated the multicluster end-to-end workflow to install the latest oc release using the actively maintained redhat-actions/openshift-tools-installer@v2.

Checklist (Definition of Done)

  • Documentation added if necessary (i.e. changes to dev setup, test execution, ...)
  • CI and all relevant tests are passing

Test manual

  1. Trigger the multicluster end-to-end workflow.
  2. Confirm the “Install oc” step completes successfully.
  3. Confirm subsequent end-to-end tests run successfully using the installed oc tool.

`redhat-actions/oc-installer` is deprecated and points to a download
mirror that has not been updated since December 2020. Replace it with
`redhat-actions/openshift-tools-installer@v2`, which is actively
maintained and supports semver ranges, arm64, and cross-workflow caching.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 7772cfe8-af9b-4b6f-b126-b8174e9c6fdb

📥 Commits

Reviewing files that changed from the base of the PR and between 6bd2516 and 802d44c.

📒 Files selected for processing (1)
  • .github/workflows/multicluster-e2e.yaml

📝 Walkthrough

Walkthrough

The multicluster end-to-end GitHub Actions workflow now uses the OpenShift tools installer to install the latest oc CLI.

Changes

Multicluster end-to-end workflow

Layer / File(s) Summary
Update OpenShift CLI installation
.github/workflows/multicluster-e2e.yaml
The “Install oc” step switches to openshift-tools-installer@v2 and explicitly requests oc: "latest".

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: replacing the deprecated oc installer with openshift-tools-installer.
Description check ✅ Passed The description covers the summary, rationale, and test plan, which matches the repository's lightweight template.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ 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.

@openshift-ci

openshift-ci Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: snecklifter
Once this PR has been reviewed and has the lgtm label, please assign mclasmeier for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci

openshift-ci Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Hi @snecklifter. Thanks for your PR.

I'm waiting for a stackrox member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant