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

Migrate examples use linear handles (except abitest) #1909

Merged
merged 3 commits into from
Apr 16, 2021

Conversation

wildarch
Copy link
Contributor

@wildarch wildarch commented Mar 12, 2021

Checklist

  • Pull request affects core Oak functionality (e.g. runtime, SDK, ABI)
    • I have written tests that cover the code changes.
    • I have checked that these tests are run by
      Cloudbuild
    • I have updated documentation accordingly.
    • I have raised an issue to
      cover any TODOs and/or unfinished work.
  • Pull request includes prototype/experimental work that is under
    construction.

This leaves just abitest left to migrate.

Based on #1915.

Tracking issue: #1854.

@google-cla google-cla bot added the cla: yes label Mar 12, 2021
@wildarch wildarch force-pushed the migrate-examples-linear-handles branch 3 times, most recently from f631468 to cc928a3 Compare March 19, 2021 14:09
@wildarch wildarch requested a review from rbehjati March 19, 2021 14:59
@wildarch wildarch marked this pull request as ready for review March 19, 2021 14:59
@wildarch wildarch force-pushed the migrate-examples-linear-handles branch from c73a556 to 12ee27f Compare April 1, 2021 08:16
Copy link
Contributor

@rbehjati rbehjati left a comment

Choose a reason for hiding this comment

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

LGTM! Can linear-handles become a default feature now?

@@ -12,17 +12,12 @@ crate-type = ["cdylib"]
anyhow = "*"
log = "*"
oak_abi = "=0.1.0"
oak_io = "=0.1.0"
oak_services = "=0.1.0"
prost = "*"

[dependencies.oak]
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this now be changed to oak = { version = "=0.1.0", features = ["linear-handles"] } to be more consistent throughout.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, good point! Done.

@wildarch
Copy link
Contributor Author

wildarch commented Apr 1, 2021

LGTM! Can linear-handles become a default feature now?

Yes, that will be the next step once we merge this and #1916

@wildarch wildarch force-pushed the migrate-examples-linear-handles branch from b32b4db to df10125 Compare April 9, 2021 11:07
@wildarch wildarch force-pushed the migrate-examples-linear-handles branch from df10125 to af69a75 Compare April 16, 2021 08:06
@wildarch wildarch merged commit 03cea3a into project-oak:main Apr 16, 2021
@wildarch wildarch deleted the migrate-examples-linear-handles branch April 16, 2021 11:54
@tiziano88
Copy link
Collaborator

🎉

@github-actions
Copy link

Reproducibility Index:

2bb36583dc89af9ed53bd472d14f887d071ef877fe86c2c21016fc91d8407722  ./examples/abitest/bin/abitest_0_frontend.wasm
1dee38c4eb66e751639540530bace7e8b8befb38afda120f1e283da55d302021  ./examples/abitest/bin/abitest_1_backend.wasm
f88e4a49982d1abf8a5239639f890d8eecb0fc2734913b1b0a7a8ee1e1f68ee1  ./examples/abitest/bin/abitest_linear_handles.wasm
8757bbf4c470b1c33c830a37fb9ce95feb56fe9b2b1a7003d3150ef21a9ea31a  ./examples/aggregator/bin/aggregator.wasm
0c532354ab48c7dbccc7ae9350f07ad9e2e4d905248d7098cbaeceba92ea4925  ./examples/chat/bin/chat.wasm
138ddd9e68f9c526f0394530dd15df45e692b9d24f1515073667e5d41c4533da  ./examples/hello_world/bin/hello_world.wasm
fd6619cfde74e44f13ae0abd24d5ae6e26abb856a7e5a596d04bf568d744ea36  ./examples/hello_world/bin/translator.wasm
350e7732cf89764361e989b63df67ccc943fbfcc6a9cbfdef777ba11ac5382bf  ./examples/http_server/bin/http_server.wasm
56f34bccf3eda180022d151e98bb4fcc5a651cbc3b9c0910241b91c039206c71  ./examples/injection/bin/injection.wasm
e14e2ed88e58e2da091f4799c599ef6f6b92133575101967d20071711dc892dc  ./examples/private_set_intersection/bin/private_set_intersection.wasm
1acbc6736491a777373727a9a48047b0904461f34bd7711a36cd5a3bade00e86  ./examples/private_set_intersection/bin/private_set_intersection_handler.wasm
f8e62bfc9f5b072f45622bb8983aa65819fc90c5e9e674fc1084fb02e892bcac  ./examples/proxy_attestation/bin/proxy_attestation_example.wasm
fd6619cfde74e44f13ae0abd24d5ae6e26abb856a7e5a596d04bf568d744ea36  ./examples/translator/bin/translator.wasm
4131b3bceb582f1f79e0624794af087ba683401a003e9ec000f4b9c8644626f2  ./examples/trusted_database/bin/trusted_database.wasm
ee61eb8f9c800623a56d6cec8b07dd0cd3b3f24d9899c6584393dbd3eacabede  ./oak_loader/bin/oak_loader
c60d0cbad2e1269aae253e805554ac2bbd70c7ccd5907277e6e90cee98355f09  ./oak_functions/loader/bin/oak_functions_loader

Reproducibility Index diff:

diff --git a/reproducibility_index b/reproducibility_index
index 14ab1dd..f57eca2 100644
--- a/reproducibility_index
+++ b/reproducibility_index
@@ -1,16 +1,16 @@
 2bb36583dc89af9ed53bd472d14f887d071ef877fe86c2c21016fc91d8407722  ./examples/abitest/bin/abitest_0_frontend.wasm
 1dee38c4eb66e751639540530bace7e8b8befb38afda120f1e283da55d302021  ./examples/abitest/bin/abitest_1_backend.wasm
 f88e4a49982d1abf8a5239639f890d8eecb0fc2734913b1b0a7a8ee1e1f68ee1  ./examples/abitest/bin/abitest_linear_handles.wasm
-5df07e7163209d363d9c4733910a71947582403627e2916a0daea661052360c6  ./examples/aggregator/bin/aggregator.wasm
-080c437c6668f050d3d50c76d6f1a21c186bc3fa2bf08a39e45c4cce7b9440fc  ./examples/chat/bin/chat.wasm
+8757bbf4c470b1c33c830a37fb9ce95feb56fe9b2b1a7003d3150ef21a9ea31a  ./examples/aggregator/bin/aggregator.wasm
+0c532354ab48c7dbccc7ae9350f07ad9e2e4d905248d7098cbaeceba92ea4925  ./examples/chat/bin/chat.wasm
 138ddd9e68f9c526f0394530dd15df45e692b9d24f1515073667e5d41c4533da  ./examples/hello_world/bin/hello_world.wasm
-34abe16461c447a1608766aaa5f18098c720244d5c45786cd872f3343eb658b0  ./examples/hello_world/bin/translator.wasm
+fd6619cfde74e44f13ae0abd24d5ae6e26abb856a7e5a596d04bf568d744ea36  ./examples/hello_world/bin/translator.wasm
 350e7732cf89764361e989b63df67ccc943fbfcc6a9cbfdef777ba11ac5382bf  ./examples/http_server/bin/http_server.wasm
 56f34bccf3eda180022d151e98bb4fcc5a651cbc3b9c0910241b91c039206c71  ./examples/injection/bin/injection.wasm
-28b014ccf9f354eb8b3a40845da1a61e9772baea79165abf042e211681278db5  ./examples/private_set_intersection/bin/private_set_intersection.wasm
-214e21108946301304b9de6e10a467d5aa045b171c7903ff5c3cb8f030c508c9  ./examples/private_set_intersection/bin/private_set_intersection_handler.wasm
-e7224705a10468d1b8d9783205161de60d03b49cea5fcb3558e36a4bbcb5a0f8  ./examples/proxy_attestation/bin/proxy_attestation_example.wasm
-34abe16461c447a1608766aaa5f18098c720244d5c45786cd872f3343eb658b0  ./examples/translator/bin/translator.wasm
-16261a9c77cef14ccee46e4e665b23f3e2e747eb8489e57ae2072280516a875d  ./examples/trusted_database/bin/trusted_database.wasm
+e14e2ed88e58e2da091f4799c599ef6f6b92133575101967d20071711dc892dc  ./examples/private_set_intersection/bin/private_set_intersection.wasm
+1acbc6736491a777373727a9a48047b0904461f34bd7711a36cd5a3bade00e86  ./examples/private_set_intersection/bin/private_set_intersection_handler.wasm
+f8e62bfc9f5b072f45622bb8983aa65819fc90c5e9e674fc1084fb02e892bcac  ./examples/proxy_attestation/bin/proxy_attestation_example.wasm
+fd6619cfde74e44f13ae0abd24d5ae6e26abb856a7e5a596d04bf568d744ea36  ./examples/translator/bin/translator.wasm
+4131b3bceb582f1f79e0624794af087ba683401a003e9ec000f4b9c8644626f2  ./examples/trusted_database/bin/trusted_database.wasm
 ee61eb8f9c800623a56d6cec8b07dd0cd3b3f24d9899c6584393dbd3eacabede  ./oak_loader/bin/oak_loader
 c60d0cbad2e1269aae253e805554ac2bbd70c7ccd5907277e6e90cee98355f09  ./oak_functions/loader/bin/oak_functions_loader

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

Successfully merging this pull request may close these issues.

None yet

3 participants