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.

Sign up for GitHub

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

Jump to bottom

Enclave Prototype #8869

Open
wants to merge 165 commits into
base: dev
Choose a base branch
from
Open

Enclave Prototype #8869

wants to merge 165 commits into from

Conversation

itstauq
Copy link
Member

@itstauq itstauq commented May 30, 2024 •
edited by snwagh
Loading

Description

Todo:

  • Copy reviews (Review all the text that a user would see; for Product team?)
  • Handle backward-compatible objects migrations

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@itstauq itstauq self-assigned this May 30, 2024
@madhavajay madhavajay added 0.8.7 and removed 0.8.7 labels Jun 3, 2024
@itstauq itstauq marked this pull request as draft June 5, 2024 13:58
@itstauq itstauq changed the title [WIP] Enclave Prototype Enclave Prototype Jun 5, 2024
@rasswanth-s
Copy link
Collaborator

Merging in DEV, as I require the skip_save_to_blob_store functionality from dev.

rasswanth-s and others added 26 commits July 10, 2024 14:53
@rasswanth-s
Merge branch 'tauquir/enclave-prototype' into rasswanth/model_hosting_v2
61a6654
@rasswanth-s
added model ref action object to dynamically load models
4ff7877
@rasswanth-s
add ModelRef as a ProxyActionObject to load model
0d3fc26 
add ModelRef to upload dataset
fixes model inference on eval dataset
@itstauq
Make Enclave V2 notebook work with remote enclaves
dce33df
@itstauq
Add get_enclave_attestation method to ProjectCode class
4ac959b
@itstauq
Get raw attestation token from enclave
0037200
@rasswanth-s
Merge branch 'tauquir/enclave-prototype' into rasswanth/model_hosting_v2
e699853
@rasswanth-s
Merge branch 'dev' into tauquir/enclave-prototype
7f3bd4a
@rasswanth-s
Merge branch 'tauquir/enclave-prototype' into rasswanth/model_hosting_v2
22251c8
@rasswanth-s
modify submit model code to action object
234044d
@rasswanth-s
Shifted to Nested Paradigm of Model Initialization in UserCode
0ffb1ee 
modify submit_model to code_action_id
@rasswanth-s
Added combine assets transfer for enclaves
281f233 
Add utility functions to wrap and unwrap initialized model
Created a new notebook, for model-hosting + enclaves
@itstauq
Add view_attestation_report method to ProjectCode class
bc6ac78
@itstauq
Add a TODO to hardcode the JWKS URL to prevent vulnerability
849bfdd
@itstauq
Add GPU Enclave attestation support
7d61889
@rasswanth-s
update model hosting notebooks
902fa83
@itstauq
Add test-gpu Enclave endpoint
b9cae0e
@itstauq
Update notebook to use CPU enclave instead of GPU
6ffc91c
@rasswanth-s
upload to enclave blob storage before transferring assets to enclave
e3e8ed8 
add client argument to save to blob storage
@rasswanth-s
cleanup test notebooks format
0f13614
@itstauq
Merge pull request #9036 from OpenMined/tauquir/enclave-attestations
21d7391 
View Enclave attestation report
@itstauq
CI fixes - Part 1
ac2d118
@itstauq
Merge pull request #9031 from OpenMined/rasswanth/model_hosting_v2
c68df91 
Model Hosting Revamp
@itstauq
Fix linters
cf5bce3
@itstauq
Comment out print statement with sensitive information
d96d695
@itstauq
Fix CI
b3b03f1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@snwagh snwagh Awaiting requested review from snwagh

@BrendanSchell BrendanSchell Awaiting requested review from BrendanSchell

@rasswanth-s rasswanth-s Awaiting requested review from rasswanth-s

At least 1 approving review is required to merge this pull request.

Assignees

@itstauq itstauq

Labels
Enclave Product Review Purple Team
Projects
None yet
Milestone
24Q3 - G2M1 Enclave Python
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@itstauq @rasswanth-s @madhavajay @snwagh @BrendanSchell @shubham3121 @yashgorana