-
Notifications
You must be signed in to change notification settings - Fork 5
feat(sync): add plugin system for extensible post-deployment sync steps #513
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
Draft
lwshang
wants to merge
25
commits into
main
Choose a base branch
from
lwshang/sync_plugin
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
949ad2a
feat(sync-plugin): scaffold plugin sync step with Component Model int…
lwshang b841457
feat(sync-plugin): implement wasmtime Component Model runtime and POC…
lwshang 84deefa
feat(sync-plugin): add icp-sync-plugin example with Rust CDK canister
lwshang 3cd68f9
feat(sync-plugin): switch to WASI preopens and add manifest files field
lwshang f69b85b
feat(sync-plugin): move Candid encoding into the plugin
lwshang 6dd648f
chore(deps): upgrade wasmtime from 30 to 41
lwshang 55193a1
chore: ignore all target/ dirs from root gitignore
lwshang 399d6bd
refactor(sync-plugin): replace custom stdio streams with MemoryOutput…
lwshang 529a51b
refactor(sync-plugin): move sync-plugin.wit into icp-sync-plugin crate
lwshang 42a8c5f
docs(sync-plugin): update SANDBOX.md for buffered stdio and moved WIT
lwshang 51e96be
refactor(sync-plugin): move PoC plugin into examples/icp-sync-plugin …
lwshang 5159b3a
docs(sync-plugin): replace outdated docs with DESIGN.md and TODO.md
lwshang 7194a6c
feat(sync-plugin): forward proxy to sync_many and add direct flag to …
lwshang f15b540
feat(sync-plugin): exercise direct flag with set_uploader and identit…
lwshang e4bfae8
docs(sync-plugin): add README for icp-sync-plugin example
lwshang 9a5c99e
feat(sync): add --proxy flag to icp sync command
lwshang e71b5f8
chore: merge main into lwshang/sync_plugin
lwshang ebeefd5
chore: upgrade wasmtime to 43 and bump toolchain to 1.91.0
lwshang 277a547
Merge remote-tracking branch 'origin/main' into lwshang/sync_plugin
lwshang 504a6c7
refactor: centralize wasm fetch/cache in canister::wasm module
lwshang 56abfd1
test(icp-sync-plugin): add unit tests for run_plugin error paths and …
lwshang dd02be6
test(icp-sync-plugin): add e2e integration test for sync plugin happy…
lwshang 561c75a
docs: update changelog for sync plugin system and icp sync --proxy
lwshang 541b44b
fix: update stale test expectations and add wasm targets to toolchain
lwshang 97efbd9
fix: use manifest path in wasm read log and error messages
lwshang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| /target | ||
| target/ | ||
| .DS_Store | ||
| .cursor | ||
|
|
||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Changelog entry says
icp sync --proxyroutes “management canister calls”, but in this PR the new behavior is primarily routing sync plugin calls to the target canister through the proxy canister. Consider rewording to reflect the actual effect of--proxyforicp sync.