Skip to content
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

255 prepare foucoco runtime upgrade to include zenlinklptoken changes in the runtime #277

Conversation

adelarja
Copy link
Contributor

@adelarja adelarja commented Jul 17, 2023

This PR was created to perform a runtime upgrade in the Foucoco testnet.

The Foucoco chain encountered a halt while attempting to execute the runtime upgrade. In this PR, we can find evidence of all the changes made in case we ever need to investigate what happened to the bricked chain.

The code changes include:

  • Code refactoring
  • Version bump from 15 to 18

During the runtime upgrade process, some modifications were made to the management account, resulting in a temporary halt of our testnet (this was an expected situation). There is a possibility that we performed the runtime upgrade simultaneously with the ongoing management account change.

While executing the upgrade, I was able to run the authorizeUpgrade extrinsic successfully, but I encountered issues when attempting to execute the enactAuthorizedUpgrade extrinsic. The enactAuthorizedUpgrade was never processed, and a pop-up indicating that the transaction was rejected/banned appeared.

Attached to this PR, you can find the compressed wasm file that we've used to perform the upgrade: foucoco_runtime_halted_chain.compact.compressed.zip

The latest changes only involve a bump of the spec_version.

This is the Foucoco wasm file that will be used for the next upgrade: foucoco_runtime.compact.compressed.zip

b-yap and others added 30 commits June 12, 2023 16:27
The order of the pallets during runtime construction has changed.
see KILTprotocol/kilt-node#416
adelarja and others added 19 commits June 19, 2023 16:23
deleted unnecessary dependencies.
Updated spacewalk dependencies.
…token-for-use-with-the-zenlink-pallet' of github.com:pendulum-chain/pendulum into 241-update-spacewalk-dependencies-and-configure-the-lp-token-for-use-with-the-zenlink-pallet
… asset chain_id and the parachain_id are the same.
…token-for-use-with-the-zenlink-pallet' into 255-prepare-foucoco-runtime-upgrade-to-include-zenlinklptoken-changes-in-the-runtime
…tion, allowing us to use a discriminant of 3 for the Zenlink LP token.
@ebma
Copy link
Member

ebma commented Jul 24, 2023

@adelarja we can close this PR without merging it because the important changes are already included in #282, correct?

@adelarja
Copy link
Contributor Author

@adelarja we can close this PR without merging it because the important changes are already included in #282, correct?

@ebma yes it is correct! thanks

@adelarja
Copy link
Contributor Author

We close this PR without merging it. All the changes are included in #282

@adelarja adelarja closed this Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prepare Foucoco runtime upgrade to include ZenlinkLPToken changes in the runtime
3 participants