Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Auto-update of all weights for 2025-01-30-1738265602 #7406

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Auto-update of all weights for 2025-01-30-1738265602 #7406

wants to merge 3 commits into from

Conversation

paritytech-cmd-bot-polkadot-sdk[bot]
Copy link

Auto-update of all weights for 2025-01-30-1738265602.

Subweight results:

@paritytech-cmd-bot-polkadot-sdk paritytech-cmd-bot-polkadot-sdk bot added the R0-silent Changes should not be mentioned in any release notes label Jan 31, 2025
@paritytech-cmd-bot-polkadot-sdk paritytech-cmd-bot-polkadot-sdk bot requested review from a team January 31, 2025 10:11
@mordamax mordamax mentioned this pull request Jan 31, 2025
Merged
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13072420140
Failed job name: test-linux-stable

1 similar comment
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13072420140
Failed job name: test-linux-stable

@bkontur
Copy link
Contributor

bkontur commented Jan 31, 2025

@claravanstaden Clara, can you please check those failed tests: https://github.com/paritytech/polkadot-sdk/actions/runs/13072420140/job/36476909198

  TRY 6 FAIL [   0.600s] bridge-hub-rococo-integration-tests tests::snowbridge::send_eth_asset_from_asset_hub_to_ethereum_and_back
  TRY 6 FAIL [   0.736s] bridge-hub-westend-integration-tests tests::snowbridge::send_weth_asset_from_asset_hub_to_ethereum

Maybe this hard-coded fee amount depends on some weights? If that is the case, we should fix that assert somehow.

@mordamax mordamax mentioned this pull request Jan 31, 2025
Open
@claravanstaden claravanstaden mentioned this pull request Feb 3, 2025
Closed
github-merge-queue bot pushed a commit that referenced this pull request Feb 3, 2025
@mordamax
add allow(dead_code) to substrate weight templates (#7408)
4f4f6f8 
address failed CI after full regeneration

Example #7406
Failed CI
https://github.com/paritytech/polkadot-sdk/actions/runs/13070646240

Monkey-patched weights which have been overridden by automation

![image](https://github.com/user-attachments/assets/ecf69173-f4dd-4113-a319-4f29d779ecae)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acatangiu acatangiu Awaiting requested review from acatangiu acatangiu will be requested when the pull request is marked ready for review acatangiu is a code owner

@cheme cheme Awaiting requested review from cheme cheme will be requested when the pull request is marked ready for review cheme is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bkontur @mordamax