-
Notifications
You must be signed in to change notification settings - Fork 3
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
ci(deploy): update deployment script to use artifacts #371
base: main
Are you sure you want to change the base?
Conversation
|
||
BUILD_MODE=$network dfx deploy --network $network app_wallet --with-cycles 2000000000000 $([[ -n "$subnet_type" ]] && echo "--subnet-type $subnet_type") | ||
else | ||
echo "Deploying the app_wallet canister to the '$network' network is not yet supported automatically." |
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.
Why?
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.
Because dfx is not exposing the —no-delete option of icx-asset
echo "Deploying the app_wallet canister to the '$network' network is not yet supported automatically." | ||
echo | ||
echo "Please deploy the app_wallet canister manually using the following command:" | ||
echo "icx-asset --pem YOUR_IDENTITY_PEM_PATH --replica TARGET_REPLICA_HTTP sync --no-delete $canister_id_output artifacts/wallet-dapp/dist" |
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.
Can't we just run this here instead of failing? If we do want to fail, maybe exit 1
would be appropriate here?
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.
Actually I think we could require the identity pem path as an argument and that would enable us to run the command instead of just logging it, I’ll try that.
Co-authored-by: mraszyk <[email protected]>
Co-authored-by: mraszyk <[email protected]>
No description provided.