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

Add UEFI Secure Boot with unsigned kernel case (New) #1440

Merged
merged 3 commits into from
Aug 30, 2024

Conversation

baconYao
Copy link
Contributor

@baconYao baconYao commented Aug 29, 2024

Description

Currently, for secure boot coverage, Checkbox only has miscellanea/secure_boot_mode to check secure boot is enabled from System level.

However, based on the responsibility of publishing image, Canonical is responsible for the signed shim, grub and kernel. Therefore, I add a manual case to replace signed kernel with unsigned kernel to prove the function of UEFI Secure Boot is working as expected.

This is raised from Boashan Project but the testing method is generic. So Canonical QA can follow the step to verify UEFI Secure Boot feature if a project supports UEFI firmware on Classic image.

Resolved issues

Documentation

Tests

Screenshot from 2024-08-30 10-04-30

Screenshot from 2024-08-30 10-04-50

Screenshot from 2024-08-29 15-06-03

@baconYao baconYao added the ce-oem The PR only contains to the ce-oem provider under the contrib area label Aug 29, 2024
@baconYao baconYao requested a review from a team August 29, 2024 07:37
@baconYao baconYao marked this pull request as ready for review August 29, 2024 07:37
@baconYao baconYao force-pushed the dev-uefi-secure-boot branch from 04ae8c8 to 60ce3af Compare August 30, 2024 03:20
@baconYao
Copy link
Contributor Author

@hanhsuan @stanley31huang I revert the commit to fix the issue of 80 length limitation. We should file another PR to fix it.

Copy link
Collaborator

@stanley31huang stanley31huang left a comment

Choose a reason for hiding this comment

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

LGTM

@baconYao baconYao merged commit 0b29388 into main Aug 30, 2024
6 of 10 checks passed
@baconYao baconYao deleted the dev-uefi-secure-boot branch August 30, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ce-oem The PR only contains to the ce-oem provider under the contrib area
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants