Skip to content
This repository was archived by the owner on Jun 8, 2022. It is now read-only.

Documentation for PR #320 #51

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

maugustosilva
Copy link

No description provided.

@@ -0,0 +1,64 @@
User Selected PCR Monitoring
Copy link
Member

Choose a reason for hiding this comment

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

Should this be "Use pre created EK"?

Copy link
Author

Choose a reason for hiding this comment

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

Oops. Fixed

@mpeters
Copy link
Member

mpeters commented Sep 21, 2021

@maugustosilva any chance to fix the (perceived) typo and rebase?

Marcio Silva and others added 12 commits September 22, 2021 09:47
Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
Now that the code change has merged, updating the documentation to
reflect this.

Signed-off-by: Marcio Silva <[email protected]>
* Clarify that tpm_policy is a JSON datastructure
* Rename "blacklist" to something clearer

Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: axelsimon <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
A few api description is not aligned with code.

Signed-off-by: Marcio Silva <[email protected]>
Signed-off-by: Marcio Silva <[email protected]>
@mpeters
Copy link
Member

mpeters commented Sep 22, 2021

@maugustosilva something about your merge into your branch is making this PR have conflicts. The actual changes look fine, but if you look at the "Commits" list it's a lot of unrelated things already merged.

@THS-on
Copy link
Member

THS-on commented Apr 6, 2022

@maugustosilva can you make sure that only your commits are in this PR and rebase on master?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants