Skip to content
This repository was archived by the owner on Aug 24, 2026. It is now read-only.

feat: preconditions for installing code - #5728

Closed
mraszyk wants to merge 3 commits into
masterfrom
mraszyk/install-code-preconditions
Closed

feat: preconditions for installing code#5728
mraszyk wants to merge 3 commits into
masterfrom
mraszyk/install-code-preconditions

Conversation

@mraszyk

@mraszyk mraszyk commented May 9, 2025

Copy link
Copy Markdown
Contributor

This PR specifies preconditions for installing canister code via the corresponding management canister endpoints that must be satisfied (if provided) for the call to succeed. The motivation is to prevent canisters that are not stopped from being installed and to enable compare-and-swap semantics for installing canisters (by atomically "comparing" the canister history before "swapping" the canister code).

@mraszyk
mraszyk requested a review from a team as a code owner May 9, 2025 09:57
@github-actions github-actions Bot added the interface-spec Changes to the IC Interface Specification label May 9, 2025

@Dfinity-Bjoern Dfinity-Bjoern left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, please proceed and merge in line with roll-out

@oggy-dfin

Copy link
Copy Markdown
Contributor

Should we do the same for load_canister_snapshot_args?

@mraszyk

mraszyk commented Aug 4, 2025

Copy link
Copy Markdown
Contributor Author

Should we do the same for load_canister_snapshot_args?

Makes sense to me.

@marc0olo

Copy link
Copy Markdown
Member

@mraszyk can we merge this?

@mraszyk

mraszyk commented Aug 13, 2025

Copy link
Copy Markdown
Contributor Author

@mraszyk can we merge this?

Not really because it is not implemented in the protocol (this PR is merely a suggestion for a new protocol feature).

@mraszyk
mraszyk marked this pull request as draft August 13, 2025 11:23
@marc0olo

Copy link
Copy Markdown
Member

This repository has been replaced by dfinity/developer-docs, live at docs.internetcomputer.org, and is being archived, so no further merges will happen here. Closing this PR — if the change is still relevant, please open an equivalent PR against dfinity/developer-docs. Thanks for the contribution.

@marc0olo marc0olo closed this Aug 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

interface-spec Changes to the IC Interface Specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants