Skip to content

registry: remove openshift-install#9372

Merged
jdx merged 1 commit intomainfrom
codex/remove-openshift-install-registry
Apr 24, 2026
Merged

registry: remove openshift-install#9372
jdx merged 1 commit intomainfrom
codex/remove-openshift-install-registry

Conversation

@jdx
Copy link
Copy Markdown
Owner

@jdx jdx commented Apr 24, 2026

Summary

Remove the openshift-install shorthand from the registry. The registry entry points only at asdf:mise-plugins/mise-openshift-install, and that plugin currently fails normal version discovery.

mise ls-remote openshift-install fails with error running list-all: exited with code 1, and mise install openshift-install@stable fails for the same reason. Exact numeric versions can still install, but the shorthand gives a broken registry experience for normal discovery and channel-style usage.

Validation

  • rg -n "openshift-install" registry docs src e2e settings.toml
  • git diff --check
  • pre-commit hk lint suite during commit, including cargo fmt --all -- --check, cargo check --all-features, taplo, schema, shellcheck, shfmt, actionlint, markdownlint, prettier, stylua, and lua-language-server

This PR was generated by an AI coding assistant.


Note

Low Risk
Low risk: this only removes a broken registry alias and does not change runtime logic or data handling.

Overview
Removes the openshift-install registry entry (which pointed to asdf:mise-plugins/mise-openshift-install), effectively disabling the shorthand and avoiding a broken ls-remote/channel install experience.

Reviewed by Cursor Bugbot for commit f961e0c. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the backend configuration and description for the OpenShift 4.x cluster installation tool. I have no feedback to provide.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 24, 2026

Greptile Summary

Removes the openshift-install shorthand from the mise registry by deleting its single-entry TOML file. The plugin (asdf:mise-plugins/mise-openshift-install) fails list-all, breaking ls-remote and channel-style installs, so removing the registry entry is the appropriate fix.

Confidence Score: 5/5

Safe to merge — single-file deletion with no remaining references in the mise source tree.

The change is a clean two-line deletion. A codebase-wide search confirms no other references to openshift-install exist in mise's own source, docs, or e2e tests. The only remaining occurrence is in the vendored aqua-registry which is unaffected. No logic, config, or schema changes are involved.

No files require special attention.

Important Files Changed

Filename Overview
registry/openshift-install.toml File deleted — removes the broken openshift-install registry entry; no other references remain in the mise source tree.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User: mise ls-remote openshift-install] -->|Before PR| B[Registry lookup → registry/openshift-install.toml]
    B --> C[asdf:mise-plugins/mise-openshift-install]
    C --> D[list-all fails → exit code 1 ❌]
    A -->|After PR| E[Registry lookup → entry not found]
    E --> F[Clear 'unknown tool' error ✅]
Loading

Reviews (1): Last reviewed commit: "registry: remove openshift-install" | Re-trigger Greptile

@jdx jdx merged commit 90d73d2 into main Apr 24, 2026
25 checks passed
@jdx jdx deleted the codex/remove-openshift-install-registry branch April 24, 2026 19:45
@github-actions
Copy link
Copy Markdown

Hyperfine Performance

mise x -- echo

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.20 x -- echo 17.8 ± 0.5 16.5 21.1 1.00
mise x -- echo 18.6 ± 0.4 17.7 21.5 1.04 ± 0.04

mise env

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.20 env 17.4 ± 0.6 16.0 22.4 1.00
mise env 18.4 ± 0.4 17.3 21.3 1.06 ± 0.04

mise hook-env

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.20 hook-env 18.1 ± 0.3 17.2 19.3 1.00
mise hook-env 18.8 ± 3.1 17.4 86.2 1.04 ± 0.17

mise ls

Command Mean [ms] Min [ms] Max [ms] Relative
mise-2026.4.20 ls 16.7 ± 0.4 15.6 18.0 1.00
mise ls 17.5 ± 0.5 16.2 18.8 1.05 ± 0.04

xtasks/test/perf

Command mise-2026.4.20 mise Variance
install (cached) 120ms 123ms -2%
ls (cached) 65ms 66ms -1%
bin-paths (cached) 66ms 68ms -2%
task-ls (cached) 720ms 705ms +2%

mise-en-dev added a commit that referenced this pull request Apr 25, 2026
### 🚀 Features

- **(registry)** add --security flag to include security info in JSON
output by @jdx in [#9364](#9364)

### 🐛 Bug Fixes

- **(config)** limit resolved backend opts to aliases by @risu729 in
[#9315](#9315)
- **(docs)** stack banner message and link on mobile by @jdx in
[#9362](#9362)
- **(github)** prefer shortest asset name as tiebreaker in
auto-detection by @jdx in [#9361](#9361)
- **(java)** newer zulu versions use a different directory structure by
@roele in [#9365](#9365)
- **(prune)** respect tracked lockfiles by @jdx in
[#9373](#9373)
- **(task)** skip tool install for missing naked tasks by @jdx in
[#9374](#9374)
- **(trust)** add untrust command by @jdx in
[#9370](#9370)
- fix - flux-operator-mcp aqua path by @monotek in
[#9357](#9357)

### 📚 Documentation

- update ruby compile msg by @fladson in
[#9338](#9338)

### 📦️ Dependency Updates

- update ubuntu docker tag to v26 by @renovate[bot] in
[#9347](#9347)
- update ghcr.io/jdx/mise:deb docker digest to 1af5a69 by @renovate[bot]
in [#9352](#9352)
- update taiki-e/install-action digest to 787505c by @renovate[bot] in
[#9354](#9354)
- update ghcr.io/jdx/mise:rpm docker digest to 7015ff3 by @renovate[bot]
in [#9353](#9353)
- update ghcr.io/jdx/mise:copr docker digest to da63a0f by
@renovate[bot] in [#9351](#9351)
- update ghcr.io/jdx/mise:alpine docker digest to 461700f by
@renovate[bot] in [#9350](#9350)
- bump communique 1.0.3 → 1.0.4 by @jdx in
[#9378](#9378)

### 📦 Registry

- remove openshift-install by @jdx in
[#9372](#9372)
- remove go-sdk by @jdx in
[#9371](#9371)

### Chore

- **(npm-publish)** use aube publish instead of npm publish by @jdx in
[#9328](#9328)

### New Contributors

- @fladson made their first contribution in
[#9338](#9338)
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