Skip to content

chore(mise): replace deprecated ubi: prefix by github: prefix - #923

Merged
pimlock merged 2 commits into
NVIDIA:mainfrom
benoitf:fix-mise-dep
Apr 23, 2026
Merged

chore(mise): replace deprecated ubi: prefix by github: prefix#923
pimlock merged 2 commits into
NVIDIA:mainfrom
benoitf:fix-mise-dep

Conversation

@benoitf

@benoitfbenoitf commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Replace deprecated ubi: prefix by github: prefix 1

When executing the command mise run vm:setup I have an error message

mise ERROR Failed to install ubi:EmbarkStudios/cargo-about@0.8.4: HTTP request to https://api.github.com/repos/EmbarkStudios/cargo-about/releases/tags/v0.8.4 returned an error status

ubi: is assuming the tag has a v prefix but here the tag has not the prefix https://github.com/EmbarkStudios/cargo-about/releases/tag/0.8.4

I'm using mise 2026.4.18 macos-arm64 (2026-04-19)

Related Issue

N/A

Changes

  • mise reports that ubi is deprecated and that github backend should be use instead
  • using github: prefix allows to give the hint that the tag has not the 'v' prefix for EmbarkStudios/cargo-about

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Footnotes

  1. https://mise.jdx.dev/dev-tools/backends/ubi.html

@benoitf
benoitf requested a review from a team as a code ownerApril 22, 2026 20:27
@copy-pr-bot

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

github-actionsBot commented Apr 22, 2026

Copy link
Copy Markdown

All contributors have signed the DCO ✍️ ✅
Posted by the DCO Assistant Lite bot.

@benoitf

This comment was marked as resolved.

Comment threadmise.toml Outdated
@pimlockpimlock self-assigned this Apr 23, 2026
@benoitfbenoitf changed the title chore(mise): replace ubi: prefix by github: prefix for EmbarkStudios/cargo-aboutchore(mise): replace ubi: prefix by github: prefixApr 23, 2026
Comment threadmise.toml Outdated
when executing the command `mise run vm:setup` I have an error message
mise ERROR Failed to install ubi:EmbarkStudios/cargo-about@0.8.4: HTTP request to https://api.github.com/repos/EmbarkStudios/cargo-about/releases/tags/v0.8.4 returned an error status
ubi: is assuming the tag has a v prefix but here the tag has not the prefix https://github.com/EmbarkStudios/cargo-about/releases/tag/0.8.4
github: allows to use github: prefix so that the tag has not the 'v' prefix
mise reports that ubi is deprecated and that github backend should be use instead so replace all ubi prefix as well
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitfbenoitf changed the title chore(mise): replace ubi: prefix by github: prefixchore(mise): replace deprecated ubi: prefix by github: prefixApr 23, 2026
pimlock
pimlock previously approved these changes Apr 23, 2026

@pimlockpimlock left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@pimlock

Copy link
Copy Markdown
Collaborator

I'm looking into the failing workflow: https://github.com/NVIDIA/OpenShell/actions/runs/24847955299/job/72742796860

@pimlock

Copy link
Copy Markdown
Collaborator

/ok to test 6e8b523

Comment threadmise.toml Outdated
@pimlock

Copy link
Copy Markdown
Collaborator

Thanks again @benoitf.
I will take the sccache migration as a separate item, as turns out it's more involved. I just reverted that back to before my suggestion.

TL;DR it looks like the path to make that work with github backend is provide platform specific patterns: https://mise.jdx.dev/dev-tools/backends/github.html#platform-specific-asset-patterns

I also think that we should consider adding locking to mise, so artifacts for all platforms are resolved at lock time (and we have an extra protection in case an artifact was overriden).

@pimlock
pimlock merged commit b19a3dc into NVIDIA:mainApr 23, 2026
9 checks passed
@pimlock

Copy link
Copy Markdown
Collaborator

@benoitf the release job after merging this is failing for unrelated reason, I'm dealing with this as well. Not a happy-CI day today 😅 .

@benoitf

Copy link
Copy Markdown
ContributorAuthor

ah yes :-/ I was not expecting that

Sign up for freeto 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

@benoitf@pimlock