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.

Sign up for GitHub

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

Jump to bottom

Update secureboot and mokutil test case #12442

Merged
merged 1 commit into from
May 6, 2021
Merged

Update secureboot and mokutil test case #12442

merged 1 commit into from
May 6, 2021

Conversation

mloviska
Copy link
Contributor

@mloviska mloviska commented Apr 30, 2021 •
edited
Loading

Test objects:

@mloviska mloviska force-pushed the shim_test branch 2 times, most recently from 25ba8c3 to 94f1ef9 Compare April 30, 2021 10:33
@mloviska mloviska added the Ready Ready for review label Apr 30, 2021
@mloviska mloviska marked this pull request as ready for review April 30, 2021 11:28
asmorodskyi
asmorodskyi approved these changes Apr 30, 2021
View reviewed changes
Copy link
Member

@asmorodskyi asmorodskyi left a comment

Choose a reason for hiding this comment

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

LGTM

@pdostal
Copy link
Member

pdostal commented Apr 30, 2021

Did you test this on XEN?

@mloviska
Copy link
Contributor Author

Did you test this on XEN?

Yup, in the past. AFAICT xen + uefi is not really supported by SUSE. But should be doable in the future

@mloviska mloviska added WIP Work in progress and removed Ready Ready for review labels May 3, 2021
@mloviska mloviska force-pushed the shim_test branch 5 times, most recently from b8ac7d7 to 91b0f0b Compare May 5, 2021 13:08
@mloviska mloviska removed the WIP Work in progress label May 5, 2021
@mloviska mloviska force-pushed the shim_test branch 4 times, most recently from 3f69401 to ac423f9 Compare May 6, 2021 09:16
@mloviska mloviska force-pushed the shim_test branch 4 times, most recently from d201054 to 4a842b8 Compare May 6, 2021 14:52
@mloviska mloviska force-pushed the shim_test branch 2 times, most recently from a4ded22 to 0f8a36e Compare May 6, 2021 15:01
@mloviska
Update secureboot and mokutil test case
6227af4 
Test objects:
- mokutil:
    - query enrolled certificates
    - import and remove mock certificate
    - check secureboot status

- sbsignutil
    - verify shim certificate
    - list certificates

- update-bootloader reinit and shim install
@dzedro dzedro merged commit 9bfeeb1 into os-autoinst:master May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cfconrad cfconrad cfconrad left review comments

@pdostal pdostal pdostal approved these changes

@asmorodskyi asmorodskyi asmorodskyi approved these changes

@jlausuch jlausuch Awaiting requested review from jlausuch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@mloviska @pdostal @asmorodskyi @cfconrad @dzedro