Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Topgrade attempts to upgrade bluefin-cli via distrobox and fails #88

Closed
tbjers opened this issue May 4, 2024 · 4 comments
Closed

Topgrade attempts to upgrade bluefin-cli via distrobox and fails #88

tbjers opened this issue May 4, 2024 · 4 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@tbjers
Copy link

tbjers commented May 4, 2024

When running ujust update the Topgrade process attempts to perform an in-place upgrade of the bluefin-cli container. The upgrade process fails with the following output:

── 16:11:40 - Distrobox ────────────────────────────────────────────────────────
 Upgrading bluefin-cli...
/sbin/su-exec
distrobox: Installing basic packages...
/usr/local/bin/xdg-open
BusyBox v1.36.1 (2024-03-27 13:51:06 UTC) multi-call binary.

Usage: su [-lmp] [-s SH] [-] [USER [FILE ARGS | -c 'CMD' [ARG0 ARGS]]]

Run shell under USER (by default, root)

        -,-l    Clear environment, go to home dir, run shell as login shell
        -p,-m   Do not set new $HOME, $SHELL, $USER, $LOGNAME
        -c CMD  Command to pass to 'sh -c'
        -s SH   Shell to use instead of user's default
distrobox: Setting up devpts mounts...
distrobox: Setting up read-only mounts...
distrobox: Setting up read-write mounts...
/usr/bin/entrypoint: line 1465: container_user_name: parameter not set
Error: An error occurred
su-exec: -S: No such file or directory

Jorge mentioned that we do not want to perform this in-place upgrade on this container as it is built via CI.

@castrojo
Copy link
Member

I think we'll need to just adjust the topgrade.toml to ignore this box. I'll try to find time investigating.

@castrojo castrojo added good first issue Good for newcomers help wanted Extra attention is needed labels May 11, 2024
@m2Giles
Copy link
Member

m2Giles commented May 16, 2024

We'll need to get real su for the wolfi boxes. BusyBox su is causing this problem most likely.

@tbjers
Copy link
Author

tbjers commented May 16, 2024

Yeah, BusyBox is a bit quirky. The integrated less command is atrocious, I always wipe it and install directly from apk.

@m2Giles
Copy link
Member

m2Giles commented May 17, 2024

This seems to be fixed now:
image

@m2Giles m2Giles closed this as completed May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants