-
Notifications
You must be signed in to change notification settings - Fork 29
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
Support reinstalling canisters (SNS) #228
Conversation
b72649f
to
b004c02
Compare
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.
LGTM
Can you also add a changelog entry and update the PR description please?
b004c02
to
c751dcf
Compare
@@ -62,6 +63,11 @@ pub struct MakeUpgradeCanisterProposalOpts { | |||
/// Path to the binary file containing argument to post-upgrade method of the new canister WASM. | |||
#[arg(long, conflicts_with = "canister_upgrade_arg")] | |||
canister_upgrade_arg_path: Option<String>, | |||
|
|||
/// The install mode. | |||
#[arg(long, short, default_value("install"), |
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.
This disagrees with If the flag is not passed, the canister is upgraded instead (the old behavior).
from the PR description. Also, it changes the default behaviour. IMO we need a very good reason to change what it does by default
c751dcf
to
1649509
Compare
1649509
to
ed77290
Compare
Co-authored-by: Adam Spofford <[email protected]>
Description
Previously there was no way to send a UpgradeSnsControlledCanister proposal that reinstalls a canister with quill. This MR adds a
--mode
flag that can be used for this purpose. The flag is required.Fixes NNS1-3063
How Has This Been Tested?
Automated tests have been adjusted to test all 3 cases
Checklist