-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat!: XCM Rate Limiter #636
Conversation
Crate versions that have been updated:
Runtime version has been increased. |
@@ -228,6 +229,10 @@ impl cumulus_pallet_xcmp_queue::Config for ParachainRuntime { | |||
type ControllerOriginConverter = XcmOriginToCallOrigin; | |||
type PriceForSiblingDelivery = (); | |||
type WeightInfo = (); | |||
type ExecuteDeferredOrigin = EnsureRoot<AccountId>; | |||
type MaxDeferredMessages = ConstU32<100>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happens to the messages if this buffer is filled?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix tests and LGTM
Reason for this is that tests are run with |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #636 +/- ##
==========================================
- Coverage 49.64% 49.59% -0.06%
==========================================
Files 60 60
Lines 3857 3861 +4
==========================================
Hits 1915 1915
- Misses 1942 1946 +4
☔ View full report in Codecov by Sentry. |
Add the xcm-rate-limiter pallet from galacticcouncil/hydration-node#561 to Basilisk.
Checklist:
Todos: