Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

SubscribeVersion is unable to pass barrier check #4135

Closed
GopherJ opened this issue Oct 24, 2021 · 4 comments
Closed

SubscribeVersion is unable to pass barrier check #4135

GopherJ opened this issue Oct 24, 2021 · 4 comments

Comments

@GopherJ
Copy link
Contributor

GopherJ commented Oct 24, 2021

https://github.com/paritytech/polkadot/blob/master/runtime/kusama/src/lib.rs#L1334-L1340

@GopherJ
Copy link
Contributor Author

GopherJ commented Oct 24, 2021

shall we add AllowSubscriptionsFrom<Everything>?

@GopherJ
Copy link
Contributor Author

GopherJ commented Oct 24, 2021

Logs:

2021-10-24 05:09:06.023 TRACE tokio-runtime-worker xcm::execute_xcm_in_credit: origin: MultiLocation { parents: 0, interior: X1(AccountId32 { network: Kusama, id: [72, 88, 172, 152, 99, 195, 122, 27, 223, 14, 77, 141, 182, 156, 131, 253, 219, 57, 6, 157, 226, 174, 5, 170, 209, 156, 121, 59, 64, 239, 201, 113] }) }, message: Xcm([TransferReserveAsset { assets: MultiAssets([MultiAsset { id: Concrete(MultiLocation { parents: 0, interior: Here }), fun: Fungible(5000000000000000) }]), dest: MultiLocation { parents: 0, interior: X1(Parachain(2085)) }, xcm: Xcm([BuyExecution { fees: MultiAsset { id: Concrete(MultiLocation { parents: 1, interior: Here }), fun: Fungible(5000000000000000) }, weight_limit: Limited(2000000000) }, DepositAsset { assets: Wild(All), max_assets: 1, beneficiary: MultiLocation { parents: 0, interior: X1(AccountId32 { network: Any, id: [72, 88, 172, 152, 99, 195, 122, 27, 223, 14, 77, 141, 182, 156, 131, 253, 219, 57, 6, 157, 226, 174, 5, 170, 209, 156, 121, 59, 64, 239, 201, 113] }) } }]) }]), weight_limit: 1000000000, weight_credit: 1000000000
2021-10-24 05:09:06.023 TRACE tokio-runtime-worker xcm::execute: origin: Some(MultiLocation { parents: 0, interior: X1(AccountId32 { network: Kusama, id: [72, 88, 172, 152, 99, 195, 122, 27, 223, 14, 77, 141, 182, 156, 131, 253, 219, 57, 6, 157, 226, 174, 5, 170, 209, 156, 121, 59, 64, 239, 201, 113] }) }), total_surplus/refunded: 0/0, error_handler_weight: 0
2021-10-24 05:09:06.023 TRACE tokio-runtime-worker xcm_builder::currency_adapter: xcm::currency_adapter deposit_asset MultiAsset { id: Concrete(MultiLocation { parents: 0, interior: Here }), fun: Fungible(5000000000000000) } MultiLocation { parents: 0, interior: X1(Parachain(2085)) }
2021-10-24 05:09:06.024 TRACE tokio-runtime-worker xcm::execute_xcm_in_credit: result: Ok(())
2021-10-24 05:09:06.026  INFO tokio-runtime-worker substrate: ✨ Imported #278 (0x2a66…c197)
2021-10-24 05:09:08.856  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #278 (0x2a66…c197), finalized #275 (0x1416…600b), ⬇ 1.5kiB/s ⬆ 1.3kiB/s
2021-10-24 05:09:12.024  INFO tokio-runtime-worker substrate: ✨ Imported #279 (0x89ee…55d9)
2021-10-24 05:09:13.857  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #279 (0x89ee…55d9), finalized #276 (0x7f8c…21fc), ⬇ 2.0kiB/s ⬆ 1.8kiB/s
2021-10-24 05:09:18.023  INFO tokio-runtime-worker substrate: ✨ Imported #280 (0xd4a5…43e6)
2021-10-24 05:09:18.857  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #280 (0xd4a5…43e6), finalized #277 (0xfd7b…6713), ⬇ 1.7kiB/s ⬆ 1.5kiB/s
2021-10-24 05:09:23.858  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #280 (0xd4a5…43e6), finalized #278 (0x2a66…c197), ⬇ 1.4kiB/s ⬆ 1.5kiB/s
2021-10-24 05:09:24.025 DEBUG tokio-runtime-worker xcm::execute_xcm: origin: MultiLocation { parents: 0, interior: X1(Parachain(2085)) }, message: Xcm([QueryResponse { query_id: 0, response: Version(2), max_weight: 0 }]), weight_limit: 100000000000
2021-10-24 05:09:24.026 TRACE tokio-runtime-worker xcm::execute_xcm_in_credit: origin: MultiLocation { parents: 0, interior: X1(Parachain(2085)) }, message: Xcm([QueryResponse { query_id: 0, response: Version(2), max_weight: 0 }]), weight_limit: 100000000000, weight_credit: 0
2021-10-24 05:09:24.026 TRACE tokio-runtime-worker xcm::should_execute: did not pass barrier: origin: MultiLocation { parents: 0, interior: X1(Parachain(2085)) }, message: Xcm([QueryResponse { query_id: 0, response: Version(2), max_weight: 0 }]), max_weight: 1000000000, weight_credit: 0
2021-10-24 05:09:24.029  INFO tokio-runtime-worker substrate: ✨ Imported #281 (0x2554…093c)
2021-10-24 05:09:28.858  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #281 (0x2554…093c), finalized #279 (0x89ee…55d9), ⬇ 2.5kiB/s ⬆ 2.0kiB/s
2021-10-24 05:09:30.021  INFO tokio-runtime-worker substrate: ✨ Imported #282 (0xe3b6…c7dc)
2021-10-24 05:09:33.859  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #282 (0xe3b6…c7dc), finalized #279 (0x89ee…55d9), ⬇ 1.5kiB/s ⬆ 1.3kiB/s
2021-10-24 05:09:36.032 DEBUG tokio-runtime-worker xcm::execute_xcm: origin: MultiLocation { parents: 0, interior: X1(Parachain(2085)) }, message: Xcm([SubscribeVersion { query_id: 0, max_response_weight: 0 }]), weight_limit: 100000000000
2021-10-24 05:09:36.033 TRACE tokio-runtime-worker xcm::execute_xcm_in_credit: origin: MultiLocation { parents: 0, interior: X1(Parachain(2085)) }, message: Xcm([SubscribeVersion { query_id: 0, max_response_weight: 0 }]), weight_limit: 100000000000, weight_credit: 0
2021-10-24 05:09:36.033 TRACE tokio-runtime-worker xcm::execute: origin: Some(MultiLocation { parents: 0, interior: X1(Parachain(2085)) }), total_surplus/refunded: 0/0, error_handler_weight: 0
2021-10-24 05:09:36.033 TRACE tokio-runtime-worker xcm::execute_xcm_in_credit: result: Ok(())
2021-10-24 05:09:36.035  INFO tokio-runtime-worker substrate: ✨ Imported #283 (0x4a09…8d12)
2021-10-24 05:09:38.859  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #283 (0x4a09…8d12), finalized #280 (0xd4a5…43e6), ⬇ 2.0kiB/s ⬆ 1.2kiB/s
2021-10-24 05:09:42.024  INFO tokio-runtime-worker substrate: ✨ Imported #284 (0x5e14…03b8)
2021-10-24 05:09:43.860  INFO tokio-runtime-worker substrate: 💤 Idle (5 peers), best: #284 (0x5e14…03b8), finalized #281 (0x2554…093c), ⬇ 2.2kiB/s ⬆ 1.9kiB/s

@gavofyork
Copy link
Member

shall we add AllowSubscriptionsFrom<Everything>?

Yes indeed.

@GopherJ
Copy link
Contributor Author

GopherJ commented Oct 29, 2021

should be closed by #4150

@GopherJ GopherJ closed this as completed Oct 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
@gavofyork @GopherJ and others