Skip to content

feat: enable asynchronous canister creation with subnet ID in the URL#9982

Merged
mraszyk merged 11 commits into
masterfrom
mraszyk/request-status-via-subnet-read-state
Apr 23, 2026
Merged

feat: enable asynchronous canister creation with subnet ID in the URL#9982
mraszyk merged 11 commits into
masterfrom
mraszyk/request-status-via-subnet-read-state

Conversation

@mraszyk
Copy link
Copy Markdown
Contributor

@mraszyk mraszyk commented Apr 22, 2026

This PR enables to retrieve the status of an (asynchronous) update call via the (existing) public HTTP endpoint /api/v3/subnet/<effective_subnet_id>/read_state on the replica. It is specified in this PR.

In more detail, this PR is needed if the synchronous update call endpoint /api/v4/subnet/.../call returns 202 and thus the user needs to process the update call (to create canister as nothing else is allowed on /api/v4/subnet/.../call) asynchronously (by polling /api/v3/subnet/.../read_state for the /request_status/...).

@github-actions github-actions Bot added the feat label Apr 22, 2026
@mraszyk mraszyk marked this pull request as ready for review April 23, 2026 05:11
@mraszyk mraszyk requested a review from a team as a code owner April 23, 2026 05:11
Copy link
Copy Markdown
Contributor

@eichhorl eichhorl left a comment

Choose a reason for hiding this comment

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

The PR title doesn't seem to match the changes

Comment thread rs/tests/networking/read_state_test.rs Outdated
@mraszyk
Copy link
Copy Markdown
Contributor Author

mraszyk commented Apr 23, 2026

The PR title doesn't seem to match the changes

What I meant to say is that this PR is needed if the synchronous update call endpoint /api/v4/subnet/.../call returns 202 and thus the user needs to process the update call (to create canister as nothing else is allowed on /api/v4/subnet/.../call) asynchronously (by polling /api/v3/subnet/.../read_state for the /request_status).

Comment thread rs/http_endpoints/public/src/read_state.rs
Comment thread rs/tests/networking/read_state_test.rs Outdated
@mraszyk mraszyk requested a review from a team as a code owner April 23, 2026 14:15
@mraszyk mraszyk added this pull request to the merge queue Apr 23, 2026
Merged via the queue into master with commit 920df80 Apr 23, 2026
37 checks passed
@mraszyk mraszyk deleted the mraszyk/request-status-via-subnet-read-state branch April 23, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants