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

BridgeHubs: XCM ExportMessage benchmark - just Rococo for now #2353

Closed
wants to merge 7 commits into from

Conversation

acatangiu
Copy link
Contributor

@acatangiu acatangiu commented Mar 21, 2023

@acatangiu acatangiu added B0-silent Changes should not be mentioned in any release notes A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. labels Mar 21, 2023
@acatangiu acatangiu self-assigned this Mar 21, 2023
@paritytech-ci paritytech-ci requested review from a team March 21, 2023 09:30
@acatangiu
Copy link
Contributor Author

bot bench $ xcm bridge-hub-rococo bridge-hubs pallet_xcm_benchmarks::generic

@acatangiu
Copy link
Contributor Author

bot cancel

@ggwpez
Copy link
Member

ggwpez commented Mar 21, 2023

Sorry commented it in the wrong MR 🤦

@acatangiu
Copy link
Contributor Author

Sorry commented it in the wrong MR facepalm

You can make it up with a review of paritytech/polkadot#6923 if you're familiar with the XCM benchmarking code 😆

@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from command-bot bot Mar 22, 2023
@paritytech paritytech deleted a comment from ggwpez Mar 22, 2023
@paritytech paritytech deleted a comment from paritytech-cicd-pr Mar 22, 2023
@ggwpez
Copy link
Member

ggwpez commented Mar 22, 2023

bot clean

☝️ is your friend

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/cumulus/-/jobs/2584500

@acatangiu
Copy link
Contributor Author

I pushed relevant changes from here directly to the bridges branch. Closing this PR.

@acatangiu acatangiu closed this Mar 24, 2023
@acatangiu acatangiu deleted the bench-xcm-export-message branch March 25, 2023 08:35
bkontur added a commit that referenced this pull request Aug 16, 2023
277f0d5496 Dynamic fees for bridges-v1 (#2294)
b1c51f7dd2 Finality loop refactoring (#2357)
620db2b10f Add equivocation detector crate and implement clients (#2348) (#2353)
3fe4b13eb4 Add basic equivocation detection pipeline schema (#2338) (#2341)

git-subtree-dir: bridges
git-subtree-split: 277f0d54961c800b231d8123c6445f378b1deb89
paritytech-processbot bot pushed a commit that referenced this pull request Aug 17, 2023
* Squashed 'bridges/' changes from edf33a2c85..277f0d5496

277f0d5496 Dynamic fees for bridges-v1 (#2294)
b1c51f7dd2 Finality loop refactoring (#2357)
620db2b10f Add equivocation detector crate and implement clients (#2348) (#2353)
3fe4b13eb4 Add basic equivocation detection pipeline schema (#2338) (#2341)

git-subtree-dir: bridges
git-subtree-split: 277f0d54961c800b231d8123c6445f378b1deb89

* [dynfees] Rococo/Wococo does not need congestion and dynamic fees (for now)

* Fix

* ".git/.scripts/commands/fmt/fmt.sh"

* Forgotten bridges/Cargo.lock

---------

Co-authored-by: command-bot <>
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. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants