-
Notifications
You must be signed in to change notification settings - Fork 42
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
Make SGX registration a bit more robust #480
Conversation
g2flyer
commented
Mar 15, 2024
•
edited
Loading
edited
- Handle missing SGX verification status CONFIGURATION_AND_SW_HARDENING_NEEDED
- Honor PDO_LOG_LEVEL in registration script
- Removes some misleading "ERROR" messages ...
4c560fc
to
c31a1af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good and necessary to support latest revision of IAS.
Ok to merge this before or after #477 .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes in the final commit look good to me.
definitely not before in its current form. the last commit of the three is the unique. the earlier ones are duplicates of 477 which have not yet been approved. |
a8c434a
to
c343c7a
Compare
c343c7a
to
7e5f682
Compare
* Handle missing SGX verification status CONFIGURATION_AND_SW_HARDENING_NEEDED Signed-off-by: Michael Steiner <[email protected]>
7e5f682
to
b174bf9
Compare
* Make script PDO_LOG_LEVEL aware * Remove unnecessary env check which leads to misleading ERROR messages when running no-config docker Signed-off-by: Michael Steiner <[email protected]>
b174bf9
to
9300a3f
Compare