cdk-rs icon indicating copy to clipboard operation
cdk-rs copied to clipboard

feat: add management canister methods for interacting with the chunk store

Open hpeebles opened this issue 2 years ago • 1 comments

Description

Adds upload_chunk, stored_chunks, clear_chunk_store and install_chunked_code to the management canister API.

How Has This Been Tested?

I have tested this locally by uploading a wasm comprised of 3 chunks, the I queried stored_chunks which returned the chunks, then I called install_chunked_code which completed successfully, then I called clear_chunk_store, then I queried stored_chunks again and this time it was empty.

Checklist:

  • [x] The title of this PR complies with Conventional Commits.
  • [x] I have edited the CHANGELOG accordingly.
  • [ ] I have made corresponding changes to the documentation.

hpeebles avatar Feb 01 '24 11:02 hpeebles

@hpeebles

For the inconsistency between the interface-spec and the replica implementation, the conclusion is that the replica will be modified to align with the spec. So we have to wait for that before we can complete this PR.

And since it will be pub chunk_hashes_list: Vec<ChunkHash> in the end, we don't have to implement the custom serde methods now.

Turning this PR back to draft now.

lwshang avatar Feb 29 '24 17:02 lwshang