Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

feat: Allow users to override secure boot key and password #40

Merged
merged 25 commits into from
Feb 28, 2024

Conversation

noelmiller
Copy link
Member

@noelmiller noelmiller commented Feb 27, 2024

Purpose

This PR is allowing a user to override the secure boot key and secure boot password to allow them to enroll their own key.

Fixes: #32

What I did

  • Updated tests to include a test for secure-boot ISOs
  • Removed unneeded hardcoded files
  • Added Variables for SECURE_BOOT_KEY_URL and ENROLLMENT_PASSWORD
  • Added checks to pull down secure boot key from SECURE_BOOT_KEY_URL

@noelmiller noelmiller self-assigned this Feb 27, 2024
@noelmiller noelmiller added the enhancement New feature or request label Feb 27, 2024
@noelmiller noelmiller marked this pull request as ready for review February 27, 2024 19:39
@noelmiller noelmiller requested a review from JasonN3 as a code owner February 27, 2024 19:39
@noelmiller
Copy link
Member Author

crap, not ready for review. disregard.

@noelmiller
Copy link
Member Author

OK ready for review. For now I am going to stick with a URL since the primary purpose of enrolling a key is to allow our kmods to work. Most users will not require this level of customization and would use our key anyway if they were building off any of our base images with our kmods.

Makefile Outdated Show resolved Hide resolved
@noelmiller noelmiller requested review from bsherman and p5 February 27, 2024 23:51
Copy link

@bsherman bsherman left a comment

Choose a reason for hiding this comment

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

I think it looks good!

@KyleGospo KyleGospo added this pull request to the merge queue Feb 28, 2024
@KyleGospo KyleGospo removed this pull request from the merge queue due to a manual request Feb 28, 2024
@KyleGospo KyleGospo added this pull request to the merge queue Feb 28, 2024
Merged via the queue into main with commit f70e8ea Feb 28, 2024
8 of 10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support action usage outside UBlue
4 participants