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

Implement get platform certificates endpoint #619

Merged
merged 10 commits into from
Jun 6, 2024

Conversation

nesitor
Copy link
Member

@nesitor nesitor commented May 28, 2024

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.

nesitor added 2 commits May 28, 2024 21:01
…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.
@nesitor nesitor requested review from olethanh and hoh May 28, 2024 19:08
@nesitor nesitor self-assigned this May 28, 2024
Copy link

codecov bot commented May 28, 2024

Codecov Report

Attention: Patch coverage is 91.52542% with 5 lines in your changes missing coverage. Please review.

Project coverage is 58.32%. Comparing base (b1ca017) to head (722db64).

Files Patch % Lines
src/aleph/vm/orchestrator/supervisor.py 40.00% 3 Missing ⚠️
src/aleph/vm/conf.py 0.00% 1 Missing ⚠️
src/aleph/vm/sevclient.py 92.85% 0 Missing and 1 partial ⚠️
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.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot added the BLUE This PR is simple and straightforward. label May 28, 2024
@aleph-im aleph-im deleted a comment from github-actions bot May 28, 2024
src/aleph/vm/conf.py Show resolved Hide resolved
src/aleph/vm/orchestrator/resources.py Outdated Show resolved Hide resolved
src/aleph/vm/sevclient.py Outdated Show resolved Hide resolved
@nesitor nesitor requested a review from hoh June 4, 2024 16:32
Base automatically changed from andres-feature-add_confidential_directory_setting to main June 5, 2024 08:04
Copy link
Member

@hoh hoh left a 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.

src/aleph/vm/conf.py Show resolved Hide resolved
src/aleph/vm/sevclient.py Outdated Show resolved Hide resolved
src/aleph/vm/sevclient.py Outdated Show resolved Hide resolved
src/aleph/vm/sevclient.py Show resolved Hide resolved
tests/supervisor/test_views.py Outdated Show resolved Hide resolved
tests/supervisor/test_views.py Outdated Show resolved Hide resolved
src/aleph/vm/sevclient.py Outdated Show resolved Hide resolved
src/aleph/vm/orchestrator/resources.py Outdated Show resolved Hide resolved
@nesitor nesitor requested a review from hoh June 5, 2024 11:11
…certificates

# Conflicts:
#	src/aleph/vm/conf.py
@nesitor nesitor requested a review from hoh June 5, 2024 14:09
@hoh hoh merged commit a4e26a8 into main Jun 6, 2024
24 checks passed
@hoh hoh deleted the andres-feature-endpoint_platform_certificates branch June 6, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLUE This PR is simple and straightforward.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants