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

Bridge hub kusama polkadot with dynamic fees #3001

Open
wants to merge 19 commits into
base: bridge-hub-kusama-polkadot
Choose a base branch
from

Conversation

bkontur
Copy link
Contributor

@bkontur bkontur commented Aug 12, 2023

This PR extends PR with BHK/P bridging, adding dynamic fees and back-preassure mechanism.

Detail description here: paritytech/parity-bridges-common#2318

TODO:

  • add Congestion messages with correct encoding

8f86ec78b7 ".git/.scripts/commands/fmt/fmt.sh"
ccf2f9483b Merge remote-tracking branch 'origin/polkadot-staging' into dynamic-fees-v1
f822ebc450 Dynamic fees v1: report congestion status to sending chain (#2318)
add9fb1d53 added/fixed some docs
569a80f233 Rename LocalXcmChannel to XcmChannelStatusProvider (#2319)
dc3618a4a5 Clippy
e7cab6ab49 (Suggestion) Ability to externalize configuration for `ExporterFor` (#2313)
c68467beff fmt
5d76f25311 use saturated_len where possible
7cc1470528 Update modules/xcm-bridge-hub-router/src/lib.rs
8d7a38a409 change log target for xcm bridge router pallet
773f93209f Revert "trigger CI"
48f1ba0323 trigger CI
b26aa98d1e fixing spellcheck, clippy and rustdoc
c467911a37 add new pallet to verify-pallets-build.sh
ed72ebe62b get rid of redundant storage value
522bbc7ec4 benchmarks for pallet-xcm-bridge-hub-router
958243564d extension_reject_call_when_dispatcher_is_inactive
38cd8f3df3 fix other tests in the bridge-runtime-common
b75e64fdf7 tests for new logic in the XcmBlobHaulerAdapter
4c741714cb tests for LocalXcmQueueMessageProcessor
d99420e14c tests for LocalXcmQueueSuspender
084f551bb6 new tests for logic changes in messages pallet
d9515f7317 use LocalXcmChannel in XcmBlobMessageDispatch
d9a0c2e468 improvements and tests for palle-xcm-bridge-router
c24301374a removed commented code
eea610a875 pallet-xcm-bridge-hub-router
1fdac85a14 forbid mesage delivery transactions when the channel between target bridge hub and target asset hub is suspended
3c98c245ac OnMessageDelviered callback
65787da038 LocalXcmQueueManager + more adapters
74b48e2cc3 impl backpressure in the XcmBlobHaulerAdapter

git-subtree-dir: bridges
git-subtree-split: 8f86ec78b7747ba32807e8691f022edb4ad3040d
@bkontur bkontur force-pushed the bridge-hub-kusama-polkadot-with-dynamic-fees branch from 6a8e39d to a08526f Compare August 14, 2023 11:03
@acatangiu acatangiu requested a review from svyatonik August 14, 2023 11:19
@bkontur bkontur force-pushed the bridge-hub-kusama-polkadot-with-dynamic-fees branch from a08526f to a701215 Compare August 14, 2023 13:28
@bkontur bkontur force-pushed the bridge-hub-kusama-polkadot-with-dynamic-fees branch from 8348b99 to 983eb8d Compare August 14, 2023 20:32
Copy link
Contributor

@svyatonik svyatonik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @bkontur! All looks good. Left a couple of questions - I could do the changes if required - please leave a comment for that :)

@bkontur bkontur added B1-note_worthy Changes should be noted in the release notes C7-critical PR touches the given topic and has a critical impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited T1-runtime This PR/Issue is related to the topic “runtime”. labels Aug 16, 2023
… bridge-hub-kusama-polkadot-with-dynamic-fees
… bridge-hub-kusama-polkadot-with-dynamic-fees
@bkontur bkontur marked this pull request as ready for review August 17, 2023 08:01
@bkontur bkontur added the A0-please_review Pull request needs code review. label Aug 17, 2023
Copy link
Contributor

@acatangiu acatangiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

Comment on lines +14 to +15
// limitations under the License.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// limitations under the License.
// limitations under the License.
#![cfg(feature = "bridging")]

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C7-critical PR touches the given topic and has a critical impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants