Please check our post <a href="https://medium.com/nodle-io/latest-parachain-upgrade-what-everyone-should-know-9d5563e97a70">here</a> describing the incident and its reasons. In this document we will be focused on a detailed description of the steps taken to resolve the issue.
TL;DR
The Nodle Parachain is bricked after its last upgrade failed. This proposal proposes to use the forceSetCode functionality on Polkadot to ensure the prompt restart of the parachain.
Solution
The solution presented below intends to maintain the existing Parachain state, and not require to revert previously approved transactions which would break finality guarantees.
It appears everything in the recent upgrade was sound with all state transitions being correct and intended. However, the migration code associated with the upgrade is taking too long and caused the proof of verification to grow beyond what the relay chain validators tolerates. Because the migration happens right when the runtime is upgraded, this causes the parachain to halt block production as the next block would always be rejected by the Polkadot validators. As such, the head of the parachain did not progress to any undesired state, and thus can be reused.
This means that it should be possible to unbrick the parachain by force setting the current code to a new runtime that is very similar to the stored runtime. As such, <a href="https://github.com/NodleCode/chain/releases/tag/2.2.2-hotfix">a new runtime has been prepared</a> with the changes below:
<ul>
<li>Identical spec version than initially deployed</li>
<li>Keeps the uniques pallet at the same storage prefix</li>
<li>Remove the migration</li>
<li>Remove the new pallet associated to the migration</li>
</ul>
When this code is forced on the relay chain then collators can use the --wasm-runtime-overrides flag to force their nodes to use the appropriate wasm code that is recognized by the relay chain.
Testing and verification
Here are the steps we took to test this proposal:
<ol>
<li>We used polkadot-launch and ran a local rococo relay chain with a few validators and a collator of our eden-dev parachain. All with fresh states starting from scratch. For the relay chain binary we used Polkadot-0.9.42 and for our parachain we used nodle-2.0.30 which is the previously published image for our nodes and contains the exact same runtime as it was on our chain right before the problematic upgrade.</li>
<li>We made runtime wasm out of <a href="https://github.com/NodleCode/chain/blob/sample-failure/runtimes/eden/src/migrations.rs#L25-L26">code</a> that could easily induce the heavy migration step for a local test. We selected the integration of an infinite loop as the fastest way to reproduce the conditions of the issue.</li>
<li>We upgraded the local collators runtime with the runtime wasm which had the intentional failure in its migration. We noticed the node indeed stopped producing blocks with the exact same message as we had seen on our collator &ldquo;Discarding proposal &hellip;took too long&ldquo;. This confirms correct reproduction of the current issue.</li>
<li>We then used Sudo on the local rococo relay chain and called forceSetCurrentCode with the <a href="https://github.com/NodleCode/chain/releases/tag/2.2.2-hotfix">runtime wasm</a> that we are going to use in our proposal. Let&rsquo;s refer to that as the good runtime. We already have discussed the idea behind this code change in the Solution section above.</li>
<li>We then stopped the collator and re-ran it using --wasm-runtime-overrides and setting it to the folder that contained the good runtime. The storage of the collator was maintained and no modification was made there. After the restart the collator started producing blocks and the relay chain backed and included them.</li>
</ol>
Proposal Preimage
Our proposal is paras -&gt; forceSetCurrentCode(para: 2026, newCode: [our good runtime], leading to the preimage hash of &ldquo;0xbdeb173184c3a932473b7921c0feb233900bbe9228c54dc346239a93b186e9cc&rdquo; and preimage length of 1216833 as shown in the following screenshot:
<img src="https://i.ibb.co/p4jJth8/Screenshot-2023-08-24-at-2-05-52-PM.png" alt="" width="640" height="320" />
Prevention Strategy
We intend to work with the Polkadot community to improve existing testing tools to detect similar issues to the one being fixed today. Namely, we would like to extend try-runtime to at least show if the PoV for a runtime upgrade is growing beyond the 5MB limit that is set for validators. Alternatively, this could also take the form of an independent test tool as well.
Regardless of what tool is developed, we will enforce the benchmarking of any migration code in our upgrade process.
Additionally, we would like to investigate whether some patches could be contributed to the Polkadot codebase to optimize for the liveness of parachains by falling back to the last known good runtime in case of upgrade or migration failures. Because this sounds like a much bigger effort given our team expertise, we will need further discussions on this topic.

Set the storage for the parachain validation code immediately.

- The Nodle Parachain had a problem after an upgrade.
- They propose using a function called "forceSetCode" to fix the problem.
- They created a new code that is similar to the old code but without the problem.
- They tested the new code and it worked.
- They want to improve testing tools to prevent similar problems in the future.

#132 Unbrick Nodle

No Active Proposals