paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK
https://polkadot.network/
1.82k stars 661 forks source link

Improve XCM debuggability #644

Open xlc opened 1 year ago

xlc commented 1 year ago

Reopen for paritytech/polkadot#7079

I just spent many hours to debug an issue caused by XCM v3 -> v2 -> v3 dropped proof size from weights and it is just hard to debug. Again, more logging will be helpful.

https://twitter.com/XiliangChen/status/1654320992712544261

xlc commented 1 year ago

https://github.com/paritytech/polkadot/blob/eb8dee045a75db91b7f4cb6a187f0bdd5e92ab47/xcm/src/v2/mod.rs#L1051

This silently drop the proof size. It should log something if proof size is not zero.

Polkadot-Forum commented 1 year ago

This issue has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/discussion-on-xcm-v2-v3-support/2796/2