-
Notifications
You must be signed in to change notification settings - Fork 19
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
Implement get platform certificates endpoint #619
Conversation
…ficates generated by sevctl. Solution: Set that directory field on settings class and ensure to create the folder on initialization step.
…atform certificates to start the VM key exchange. Solution: Create that endpoint and return the platform certificates generated by the `sevctl` command.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #619 +/- ##
==========================================
+ Coverage 57.95% 58.32% +0.37%
==========================================
Files 60 61 +1
Lines 5406 5464 +58
Branches 601 607 +6
==========================================
+ Hits 3133 3187 +54
- Misses 2133 2137 +4
Partials 140 140 ☔ View full report in Codecov by Sentry. |
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.
Base looks good, got a few comments to improve the code.
…certificates # Conflicts: # src/aleph/vm/conf.py
Problem: There isn't an endpoint to be able to get the confidential platform certificates to start the VM key exchange.
Solution: Create that endpoint and return the platform certificates generated by the
sevctl
command.