-
Notifications
You must be signed in to change notification settings - Fork 417
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
Fedora 40 fwupdmgr: partition kind required #7298
Comments
I'm wondering if it's this perhaps? b42ad1d -- would a COPR build be okay for testing? |
If so, there's a new build in https://copr.fedorainfracloud.org/coprs/rhughes/fwupd/ available in a few minutes. |
Doesn't seem to fix it.. what can I do to give you more info?
|
And
|
The fwupdtool output (I chose to look at /boot/efi partition at the choice) |
Looks like md127 is xfs. Same kind of problem we saw with btrfs. I guess time for us to switch to an allow list. |
yeah. I went with / as xfs because it was installed from Fedora Server. The EFI partition is not on a raid partition because I only had 1 nvme in the system at the time. I can run an updated rpm when ready |
IMO - Something is totally wrong with the Fedora installer that it's making partitions of kind We can add a workaround in fwupd, but who knows what else breaks from this. |
I wonder if I did something odd when installing the system, but I just checked another Fedora 38 system which I did a straight install on and it is also OK searching on that string gave me from 2017 storaged-project/blivet#538 which says that is the expected partition type |
but I am swimming way outside of my expertise so not sure where this bug would need to be or what type is expected. |
Yeah that's the expected partition type for an ESP but if you look at the logs you shared from fwupdtool you can see your MD partition was marked as that. Which isn't expected for fwupd. |
That is weird. Looking at the udisk.. it isn't labeled at all but looking at the fwupd I see it thinks it is two different partition types:
Can it be labeled with 2 different types? Or is there something else going on |
Oh good point! Let me dig in more. |
I think it's a side effect of 7c1df3d. |
I added an extra commit to the PR to help clarify that. It looks like it's not an installer bug; it's our own doing in that we tried to help support people who put EFI partition on RAID 1 arrays that caused all of this. |
thanks.. sometimes our past helpful selves are future selfs worst enemy 😆 |
There should be a new build in the COPR now. |
Sadly.. it doesn't seem to have fixed the issue:
|
Looking at the output of the newer fwupdtool I get
|
@hughsie can you please double check the commits you included in that test build? I notice this:
But with those commits we should have seen the word
|
Describe the bug
I have a Dell Precision 3630 with an outstanding firmware update. When running the commands, I get a small error but no idea on how to fix it:
I added more
-v
to the call and gotbut I don't know if that helps. I did look to see if the /boot/efi was seen as an EFI partition
Steps to Reproduce
On a Dell Precision 3630, run fwupdmgr to get updates
Expected behavior
I expected either the update to go through or a more clear message on what kind of partition was not matching.
fwupd version information
Version of the fwupdmgr package:
Please note how you installed it (
apt
,dnf
,pacman
, source, etc):This was installed via dnf on Fedora 40
Please provide the output of the fwupd devices recognized in your system.
Additional questions
The text was updated successfully, but these errors were encountered: