-
Notifications
You must be signed in to change notification settings - Fork 28
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
Ubuntu Desktop check fails #54
Comments
dpkg --get-selections | grep linux-image | grep -v deinstall
Another option (and this is complete conjecture) would be to check for the |
Thank you for the feedback Andrew. The script supports both 22.04 and 24.04 so I'll need to look for a solution that works for both. Alternatively, I could put in separate checks for each, but I'd like to avoid that. |
Thanks! I don't think checking for an apt package and snap as a backup should be an issue - both should be fairly explicitly showing that the desktop version is being used. If the apt package is missing, but snap exists, that's pretty clear what's going on. Both package names are very clear that they are for desktop. Is the desktop check of particular importance? How critical would things go wrong in the (IMO very unlikely state, if not impossible) this check has if it contains a false positive? Does that risk override supporting the latest Ubuntu version? |
I have just successfully tested the following to install a server install in Virtualbox. Were you installing on bare metal or using a VM? I noticed you said you did an update/upgrade in the Live environment before running the script. I didn't do that. |
You can confirm that Canonical has, indeed, made this change on their desktop installer bootstrap repository starting in 24.04.01 (and to answer your question, yes this is the current version on the Ubuntu website and what I used): https://github.com/canonical/ubuntu-desktop-provision
This is the new graphical installer for the LiveUSB. I believe if you choose "Try Ubuntu Desktop" instead of "Install Ubuntu Desktop" the Per your questions, if I choose "Try" and use the RAM filesystem created to bootstrap the server (as I think is intended?), the Edit 2: I see where I wrote I was using the prior non-point release in my OP. I don't think that's correct, and may have been a typo, but I can confirm later today when I get access to my USB key. |
The image I was using was, indeed the |
Following up - I tested this in a VM (via UTM) with the same image. The |
Hi Andrew, thank you for the updates from your testing. That's interesting that booting the same iso in bare metal from a usb key vs booting in a VM presents a different environment. It certainly looks like I'll need to adapt the code. I'm just surprised no-one has flagged the issue yet. Perhaps most users are creating 22.04 instead of 24.04 installs. |
Maybe! Let me confirm for you it's not user error. Although I think the snap check shouldn't create any false positives. |
Hi Andrew, have you additional feedback? I can't easily test the script on bare metal so you're my only feedback for bare metal installs at the moment. I introduced the check because a user incorrectly used the server iso to install from. It had a differently formatted apt sources file so the script didn't work. Note to lurkers: If you are experiencing the issue please let us know. |
Thanks for checking up - not yet, unfortunately. The server I was provisioning is up and running, so I lost my device I was targeting. That said, I should have another here soon to test. |
Ideally we will identify a way to virtualise whatever you are seeing on the bare metal install so I can test for it in future using a VM. |
There doesn't seem to be an
ubuntu-desktop
package any more with the Ubuntu Desktop LiveUSB (v24.04), so the Ubuntu Desktop check fails:https://github.com/Sithuk/ubuntu-server-zfsbootmenu/blob/main/ubuntu_server_encrypted_root_zfs.sh#L112-L125
I am using the LiveUSB from
ubuntu-24.04-desktop-amd64.iso
after performing a fullapt update/upgrade
.The
casper
check does seem to still work as-expected.The text was updated successfully, but these errors were encountered: