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 SEV-SNP policy for signed UEFI measurements #446

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

deeglaze
Copy link
Collaborator

@deeglaze deeglaze commented May 3, 2024

Depends on PR#445

This adds an extra validation check beyond well-formedness that the verification step checks. If the reference values are available within the SEV-SNP attestation certificate chain, then verify the signature and check the report measurement against the golden values.

@deeglaze
Copy link
Collaborator Author

deeglaze commented Jun 3, 2024

/gcbrun

@deeglaze deeglaze requested a review from jkl73 June 4, 2024 00:51
@deeglaze
Copy link
Collaborator Author

deeglaze commented Jun 4, 2024

@jkl73 not sure what to make of the CS presubmit failure.

@jkl73
Copy link
Contributor

jkl73 commented Jun 4, 2024

@jkl73 not sure what to make of the CS presubmit failure.

just ignore the error now.. I'm still trying to fix the the build

@jkl73
Copy link
Contributor

jkl73 commented Jun 5, 2024

@jkl73 not sure what to make of the CS presubmit failure.

just ignore the error now.. I'm still trying to fix the the build

@deeglaze could you rebase this PR and run the CS presubmit again, I think I fixed it (hopefully..)

@jkl73
Copy link
Contributor

jkl73 commented Jun 5, 2024

/gcbrun

@deeglaze deeglaze requested a review from alexmwu June 6, 2024 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants