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

Adjust ESP detection to catch zfsbootmenu #6960

Merged
merged 2 commits into from
Mar 22, 2024
Merged

Adjust ESP detection to catch zfsbootmenu #6960

merged 2 commits into from
Mar 22, 2024

Conversation

superm1
Copy link
Member

@superm1 superm1 commented Mar 22, 2024

Type of pull request:

@hughsie
Copy link
Member

hughsie commented Mar 22, 2024

I'd feel a lot more soft and squishy if this had self tests tbh.

@superm1
Copy link
Member Author

superm1 commented Mar 22, 2024

I'd feel a lot more soft and squishy if this had self tests tbh.

The problem is we need a template for udisks2 in dbusmock. Without that we can't come up with good setups for the crazy disk setups.

martinpitt/python-dbusmock#203

@hughsie
Copy link
Member

hughsie commented Mar 22, 2024

I wondered if we could mock something up just by shoving the FuVolumes into fu_context_get_esp_volumes manually -- e.g. in the uefi-capsule fu-self-test.c

@superm1
Copy link
Member Author

superm1 commented Mar 22, 2024

fu_context_get_esp_volumes

But then we try to go and mount them. Since we made udisks2 mandatory, I don't think this will work.

@hughsie
Copy link
Member

hughsie commented Mar 22, 2024

But then we try to go and mount them.

Hmm, true.

@superm1
Copy link
Member Author

superm1 commented Mar 22, 2024

To add more. We have FWUPD_UEFI_ESP_PATH but this is specifically for the test suite to specify a single ESP path, not a series of paths. We can certainly look for a few fake files under this path but I don't think we can come up with a path that reflects heuristics selection between two groups of paths with various files and which one gets ranked like this.

Not everyone uses a suffix in their filename.  Instead use only filename
prefix and extension for suffix.
Copy link
Member

@hughsie hughsie left a comment

Choose a reason for hiding this comment

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

Thanks

@superm1 superm1 enabled auto-merge (rebase) March 22, 2024 14:48
@superm1 superm1 merged commit 506cd1c into main Mar 22, 2024
16 checks passed
@superm1 superm1 deleted the superm1/esp-list branch March 22, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants