From 0e1717e55242f6c08efc51eb5cb7ecc39445a5a2 Mon Sep 17 00:00:00 2001
From: Maksym H <1177472+mordamax@users.noreply.github.com>
Date: Tue, 8 Oct 2024 15:20:19 +0100
Subject: [PATCH 01/10] [DNM] Update README.md
---
README.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/README.md b/README.md
index b8ddf8427c9fd..1952c3f29d1e7 100644
--- a/README.md
+++ b/README.md
@@ -1,4 +1,4 @@
-
+1

From bcf5064f5fc68d4d9db6dd86f9852732c5d27999 Mon Sep 17 00:00:00 2001
From: GitHub Action
Date: Tue, 8 Oct 2024 14:23:35 +0000
Subject: [PATCH 02/10] Update from mordamax running command 'prdoc --audience
node_dev --bump patch'
---
prdoc/pr_5977.prdoc | 6 ++++++
1 file changed, 6 insertions(+)
create mode 100644 prdoc/pr_5977.prdoc
diff --git a/prdoc/pr_5977.prdoc b/prdoc/pr_5977.prdoc
new file mode 100644
index 0000000000000..f22027ac31d7e
--- /dev/null
+++ b/prdoc/pr_5977.prdoc
@@ -0,0 +1,6 @@
+title: '[DNM] testing /cmd'
+doc:
+- audience:
+ - Node Dev
+ description: null
+crates: []
From 75f511e810ead8a94dd01805a1d483ca6f2cce1e Mon Sep 17 00:00:00 2001
From: Maksym H
Date: Tue, 8 Oct 2024 22:42:03 +0100
Subject: [PATCH 03/10] flip continue-on-fail to fail-fast, save output to file
+ artifacts
---
.github/scripts/cmd/cmd.py | 47 ++++++++++++++++++---------
.github/scripts/cmd/test_cmd.py | 20 ++++++------
.github/workflows/cmd.yml | 33 +++++++++++++++++--
docs/contributor/commands-readme.md | 5 ---
docs/contributor/weight-generation.md | 34 ++++++++++---------
5 files changed, 89 insertions(+), 50 deletions(-)
diff --git a/.github/scripts/cmd/cmd.py b/.github/scripts/cmd/cmd.py
index 01f36ea375c5b..6a624bf4237b5 100755
--- a/.github/scripts/cmd/cmd.py
+++ b/.github/scripts/cmd/cmd.py
@@ -15,12 +15,21 @@
runtimeNames = list(map(lambda x: x['name'], runtimesMatrix))
common_args = {
- '--continue-on-fail': {"action": "store_true", "help": "Won't exit(1) on failed command and continue with next steps. "},
'--quiet': {"action": "store_true", "help": "Won't print start/end/failed messages in PR"},
'--clean': {"action": "store_true", "help": "Clean up the previous bot's & author's comments in PR"},
'--image': {"help": "Override docker image '--image docker.io/paritytech/ci-unified:latest'"},
}
+def print_and_log(message, output_file='/tmp/cmd/command_output.log'):
+ print(message)
+ with open(output_file, 'a') as f:
+ f.write(message + '\n')
+
+def setup_logging():
+ if not os.path.exists('/tmp/cmd'):
+ os.makedirs('/tmp/cmd')
+ open('/tmp/cmd/command_output.log', 'w')
+
parser = argparse.ArgumentParser(prog="/cmd ", description='A command runner for polkadot-sdk repo', add_help=False)
parser.add_argument('--help', action=_HelpAction, help='help for help if you need some help') # help for help
for arg, config in common_args.items():
@@ -28,6 +37,8 @@
subparsers = parser.add_subparsers(help='a command to run', dest='command')
+setup_logging()
+
"""
BENCH
"""
@@ -39,8 +50,8 @@
Runs benchmarks for pallet_balances and pallet_multisig for all runtimes which have these pallets. **--quiet** makes it to output nothing to PR but reactions
%(prog)s --pallet pallet_balances pallet_xcm_benchmarks::generic --quiet
- Runs bench for all pallets for westend runtime and continues even if some benchmarks fail
- %(prog)s --runtime westend --continue-on-fail
+ Runs bench for all pallets for westend runtime and fails fast on first failed benchmark
+ %(prog)s --runtime westend --fail-fast
Does not output anything and cleans up the previous bot's & author command triggering comments in PR
%(prog)s --runtime westend rococo --pallet pallet_balances pallet_multisig --quiet --clean
@@ -53,6 +64,7 @@
parser_bench.add_argument('--runtime', help='Runtime(s) space separated', choices=runtimeNames, nargs='*', default=runtimeNames)
parser_bench.add_argument('--pallet', help='Pallet(s) space separated', nargs='*', default=[])
+parser_bench.add_argument('--fail-fast', help='Fail fast on first failed benchmark', action='store_true')
"""
FMT
@@ -156,7 +168,9 @@ def main():
manifest_path = os.popen(search_manifest_path).read()
if not manifest_path:
print(f'-- pallet {pallet} not found in dev runtime')
- exit(1)
+ if args.fail_fast:
+ print_and_log(f'Error: {pallet} not found in dev runtime')
+ sys.exit(1)
package_dir = os.path.dirname(manifest_path)
print(f'-- package_dir: {package_dir}')
print(f'-- manifest_path: {manifest_path}')
@@ -186,8 +200,9 @@ def main():
f"{config['bench_flags']}"
print(f'-- Running: {cmd} \n')
status = os.system(cmd)
- if status != 0 and not args.continue_on_fail:
- print(f'Failed to benchmark {pallet} in {runtime}')
+
+ if status != 0 and args.fail_fast:
+ print_and_log(f'❌ Failed to benchmark {pallet} in {runtime}')
sys.exit(1)
# Otherwise collect failed benchmarks and print them at the end
@@ -198,14 +213,14 @@ def main():
successful_benchmarks[f'{runtime}'] = successful_benchmarks.get(f'{runtime}', []) + [pallet]
if failed_benchmarks:
- print('❌ Failed benchmarks of runtimes/pallets:')
+ print_and_log('❌ Failed benchmarks of runtimes/pallets:')
for runtime, pallets in failed_benchmarks.items():
- print(f'-- {runtime}: {pallets}')
+ print_and_log(f'-- {runtime}: {pallets}')
if successful_benchmarks:
- print('✅ Successful benchmarks of runtimes/pallets:')
+ print_and_log('✅ Successful benchmarks of runtimes/pallets:')
for runtime, pallets in successful_benchmarks.items():
- print(f'-- {runtime}: {pallets}')
+ print_and_log(f'-- {runtime}: {pallets}')
elif args.command == 'fmt':
command = f"cargo +nightly fmt"
@@ -213,8 +228,8 @@ def main():
nightly_status = os.system(f'{command}')
taplo_status = os.system('taplo format --config .config/taplo.toml')
- if (nightly_status != 0 or taplo_status != 0) and not args.continue_on_fail:
- print('❌ Failed to format code')
+ if (nightly_status != 0 or taplo_status != 0):
+ print_and_log('❌ Failed to format code')
sys.exit(1)
elif args.command == 'update-ui':
@@ -222,15 +237,15 @@ def main():
print(f'Updating ui with `{command}`')
status = os.system(f'{command}')
- if status != 0 and not args.continue_on_fail:
- print('❌ Failed to format code')
+ if status != 0:
+ print_and_log('❌ Failed to update ui')
sys.exit(1)
elif args.command == 'prdoc':
# Call the main function from ./github/scripts/generate-prdoc.py module
exit_code = generate_prdoc.main(args)
- if exit_code != 0 and not args.continue_on_fail:
- print('❌ Failed to generate prdoc')
+ if exit_code != 0:
+ print_and_log('❌ Failed to generate prdoc')
sys.exit(exit_code)
print('🚀 Done')
diff --git a/.github/scripts/cmd/test_cmd.py b/.github/scripts/cmd/test_cmd.py
index 0316c7ff1bb48..faad3f261b9ae 100644
--- a/.github/scripts/cmd/test_cmd.py
+++ b/.github/scripts/cmd/test_cmd.py
@@ -96,7 +96,7 @@ def test_bench_command_normal_execution_all_runtimes(self):
command='bench',
runtime=list(map(lambda x: x['name'], mock_runtimes_matrix)),
pallet=['pallet_balances'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -153,7 +153,7 @@ def test_bench_command_normal_execution(self):
command='bench',
runtime=['westend'],
pallet=['pallet_balances', 'pallet_staking'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -196,7 +196,7 @@ def test_bench_command_normal_execution_xcm(self):
command='bench',
runtime=['westend'],
pallet=['pallet_xcm_benchmarks::generic'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -232,7 +232,7 @@ def test_bench_command_two_runtimes_two_pallets(self):
command='bench',
runtime=['westend', 'rococo'],
pallet=['pallet_balances', 'pallet_staking'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -290,7 +290,7 @@ def test_bench_command_one_dev_runtime(self):
command='bench',
runtime=['dev'],
pallet=['pallet_balances'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -327,7 +327,7 @@ def test_bench_command_one_cumulus_runtime(self):
command='bench',
runtime=['asset-hub-westend'],
pallet=['pallet_assets'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -362,7 +362,7 @@ def test_bench_command_one_cumulus_runtime_xcm(self):
command='bench',
runtime=['asset-hub-westend'],
pallet=['pallet_xcm_benchmarks::generic', 'pallet_assets'],
- continue_on_fail=False,
+ fail_fast=True,
quiet=False,
clean=False,
image=None
@@ -400,7 +400,7 @@ def test_bench_command_one_cumulus_runtime_xcm(self):
self.mock_system.assert_has_calls(expected_calls, any_order=True)
- @patch('argparse.ArgumentParser.parse_known_args', return_value=(argparse.Namespace(command='fmt', continue_on_fail=False), []))
+ @patch('argparse.ArgumentParser.parse_known_args', return_value=(argparse.Namespace(command='fmt'), []))
@patch('os.system', return_value=0)
def test_fmt_command(self, mock_system, mock_parse_args):
with patch('sys.exit') as mock_exit:
@@ -410,7 +410,7 @@ def test_fmt_command(self, mock_system, mock_parse_args):
mock_system.assert_any_call('cargo +nightly fmt')
mock_system.assert_any_call('taplo format --config .config/taplo.toml')
- @patch('argparse.ArgumentParser.parse_known_args', return_value=(argparse.Namespace(command='update-ui', continue_on_fail=False), []))
+ @patch('argparse.ArgumentParser.parse_known_args', return_value=(argparse.Namespace(command='update-ui'), []))
@patch('os.system', return_value=0)
def test_update_ui_command(self, mock_system, mock_parse_args):
with patch('sys.exit') as mock_exit:
@@ -419,7 +419,7 @@ def test_update_ui_command(self, mock_system, mock_parse_args):
mock_exit.assert_not_called()
mock_system.assert_called_with('sh ./scripts/update-ui-tests.sh')
- @patch('argparse.ArgumentParser.parse_known_args', return_value=(argparse.Namespace(command='prdoc', continue_on_fail=False), []))
+ @patch('argparse.ArgumentParser.parse_known_args', return_value=(argparse.Namespace(command='prdoc'), []))
@patch('os.system', return_value=0)
def test_prdoc_command(self, mock_system, mock_parse_args):
with patch('sys.exit') as mock_exit:
diff --git a/.github/workflows/cmd.yml b/.github/workflows/cmd.yml
index f8bc7cb5b606d..00fec329e49f9 100644
--- a/.github/workflows/cmd.yml
+++ b/.github/workflows/cmd.yml
@@ -368,6 +368,19 @@ jobs:
git status
git diff
+ if [ -f /tmp/cmd/command_output.log ]; then
+ CMD_OUTPUT=$(cat /tmp/cmd/command_output.log)
+ echo "cmd_output=$CMD_OUTPUT" >> $GITHUB_OUTPUT
+ echo "$CMD_OUTPUT" >> $GITHUB_STEP_SUMMARY
+ fi
+
+ - name: Upload command output
+ if: ${{ always() }}
+ uses: actions/upload-artifact@v4
+ with:
+ name: command-output
+ path: /tmp/cmd/command_output.log
+
- name: Commit changes
run: |
if [ -n "$(git status --porcelain)" ]; then
@@ -413,35 +426,49 @@ jobs:
uses: actions/github-script@v7
env:
SUBWEIGHT: "${{ steps.subweight.outputs.result }}"
+ CMD_OUTPUT: "${{ steps.cmd.outputs.cmd_output }}"
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
let runUrl = ${{ steps.build-link.outputs.run_url }}
let subweight = process.env.SUBWEIGHT;
+ let cmdOutput = process.env.CMD_OUTPUT;
- let subweightCollapsed = subweight
+ let subweightCollapsed = subweight.trim() !== ''
? `\n\nSubweight results:
\n\n${subweight}\n\n `
: '';
+ let cmdOutputCollapsed = cmdOutput.trim() !== ''
+ ? `\n\nCommand output:
\n\n${cmdOutput}\n\n `
+ : '';
+
github.rest.issues.createComment({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
- body: `Command "${{ steps.get-pr-comment.outputs.group2 }}" has finished ✅ [See logs here](${runUrl})${subweightCollapsed}`
+ body: `Command "${{ steps.get-pr-comment.outputs.group2 }}" has finished ✅ [See logs here](${runUrl})${subweightCollapsed}${cmdOutputCollapsed}`
})
- name: Comment PR (Failure)
if: ${{ failure() && !contains(github.event.comment.body, '--quiet') }}
uses: actions/github-script@v7
+ env:
+ CMD_OUTPUT: "${{ steps.cmd.outputs.cmd_output }}"
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
let jobUrl = ${{ steps.build-link.outputs.job_url }}
+ let cmdOutput = process.env.CMD_OUTPUT;
+
+ let cmdOutputCollapsed = cmdOutput.trim() !== ''
+ ? `\n\nCommand output:
\n\n${cmdOutput}\n\n `
+ : '';
+
github.rest.issues.createComment({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
- body: `Command "${{ steps.get-pr-comment.outputs.group2 }}" has failed ❌! [See logs here](${jobUrl})`
+ body: `Command "${{ steps.get-pr-comment.outputs.group2 }}" has failed ❌! [See logs here](${jobUrl})${cmdOutputCollapsed}`
})
- name: Add 😕 reaction on failure
diff --git a/docs/contributor/commands-readme.md b/docs/contributor/commands-readme.md
index 3a0fadc3bb253..52c554cc70987 100644
--- a/docs/contributor/commands-readme.md
+++ b/docs/contributor/commands-readme.md
@@ -24,11 +24,6 @@ By default, the Start and End/Failure of the command will be commented with the
If you want to avoid, use this flag. Go to
[Action Tab](https://github.com/paritytech/polkadot-sdk/actions/workflows/cmd.yml) to see the pipeline status.
-2.`--continue-on-fail` to continue running the command even if something inside a command
-(like specific pallet weight generation) are failed.
-Basically avoids interruption in the middle with `exit 1`
-The pipeline logs will include what is failed (like which runtimes/pallets), then you can re-run them separately or not.
-
3.`--clean` to clean up all yours and bot's comments in PR relevant to `/cmd` commands. If you run too many commands,
or they keep failing, and you're rerunning them again, it's handy to add this flag to keep a PR clean.
diff --git a/docs/contributor/weight-generation.md b/docs/contributor/weight-generation.md
index 77a570c94530c..d5b407d5562d3 100644
--- a/docs/contributor/weight-generation.md
+++ b/docs/contributor/weight-generation.md
@@ -19,51 +19,53 @@ In a PR run the actions through comment:
To regenerate all weights (however it will take long,
so don't do it unless you really need it), run the following command:
+
```sh
/cmd bench
```
To generate weights for all pallets in a particular runtime(s), run the following command:
+
```sh
/cmd bench --runtime kusama polkadot
```
For Substrate pallets (supports sub-modules too):
+
```sh
/cmd bench --runtime dev --pallet pallet_asset_conversion_ops
```
> **📝 Note**: The action is not being run right-away, it will be queued and run in the next available runner.
-So might be quick, but might also take up to 10 mins (That's in control of Github).
-Once the action is run, you'll see reaction 👀 on original comment, and if you didn't pass `--quiet` -
-it will also send a link to a pipeline when started, and link to whole workflow when finished.
+> So might be quick, but might also take up to 10 mins (That's in control of Github).
+> Once the action is run, you'll see reaction 👀 on original comment, and if you didn't pass `--quiet` -
+> it will also send a link to a pipeline when started, and link to whole workflow when finished.
----
+> **📝 Note**: It will try keep benchmarking even if some pallets failed, with the result of failed/successful pallets.
+>
+> If you want to fail fast on first failed benchmark, add `--fail-fast` flag to the command.
-> **💡Hint #1** : if you run all runtimes or all pallets, it might be that some pallet in the middle is failed
-to generate weights, thus it stops (fails) the whole pipeline.
-> If you want, you can make it to continue running, even if some pallets are failed, add `--continue-on-fail`
-flag to the command. The report will include which runtimes/pallets have failed, then you can re-run
-them separately after all is done.
+---
This way it runs all possible runtimes for the specified pallets, if it finds them in the runtime
+
```sh
/cmd bench --pallet pallet_balances pallet_xcm_benchmarks::generic pallet_xcm_benchmarks::fungible
```
If you want to run all specific pallet(s) for specific runtime(s), you can do it like this:
+
```sh
/cmd bench --runtime bridge-hub-polkadot --pallet pallet_xcm_benchmarks::generic pallet_xcm_benchmarks::fungible
```
-
-> **💡Hint #2** : Sometimes when you run too many commands, or they keep failing and you're rerunning them again,
-it's handy to add `--clean` flag to the command. This will clean up all yours and bot's comments in PR relevant to
-/cmd commands.
+> **💡Hint #1** : Sometimes when you run too many commands, or they keep failing and you're rerunning them again,
+> it's handy to add `--clean` flag to the command. This will clean up all yours and bot's comments in PR relevant to
+> /cmd commands.
```sh
-/cmd bench --runtime kusama polkadot --pallet=pallet_balances --clean --continue-on-fail
+/cmd bench --runtime kusama polkadot --pallet=pallet_balances --clean
```
-> **💡Hint #3** : If you have questions or need help, feel free to tag @paritytech/opstooling (in github comments)
-or ping in [matrix](https://matrix.to/#/#command-bot:parity.io) channel.
+> **💡Hint #2** : If you have questions or need help, feel free to tag @paritytech/opstooling (in github comments)
+> or ping in [matrix](https://matrix.to/#/#command-bot:parity.io) channel.
From a4d56188742d689d8fc2f79107fe474816380348 Mon Sep 17 00:00:00 2001
From: Maksym H
Date: Tue, 8 Oct 2024 22:45:45 +0100
Subject: [PATCH 04/10] Update README.md
---
README.md | 1 -
1 file changed, 1 deletion(-)
diff --git a/README.md b/README.md
index 1952c3f29d1e7..8016b6b37301a 100644
--- a/README.md
+++ b/README.md
@@ -1,4 +1,3 @@
-1

From 23c65529e45b2b7676050b583fcaa6c4aee5891c Mon Sep 17 00:00:00 2001
From: Maksym H
Date: Tue, 8 Oct 2024 22:46:24 +0100
Subject: [PATCH 05/10] Delete pr_5977.prdoc
---
prdoc/pr_5977.prdoc | 6 ------
1 file changed, 6 deletions(-)
delete mode 100644 prdoc/pr_5977.prdoc
diff --git a/prdoc/pr_5977.prdoc b/prdoc/pr_5977.prdoc
deleted file mode 100644
index f22027ac31d7e..0000000000000
--- a/prdoc/pr_5977.prdoc
+++ /dev/null
@@ -1,6 +0,0 @@
-title: '[DNM] testing /cmd'
-doc:
-- audience:
- - Node Dev
- description: null
-crates: []
From 6c5cb681524ef396124530dca7d136066a5ad26e Mon Sep 17 00:00:00 2001
From: Maksym H
Date: Wed, 9 Oct 2024 12:44:35 +0100
Subject: [PATCH 06/10] add timeout to cmd
---
.github/workflows/cmd.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/.github/workflows/cmd.yml b/.github/workflows/cmd.yml
index 00fec329e49f9..d87aaa55137a8 100644
--- a/.github/workflows/cmd.yml
+++ b/.github/workflows/cmd.yml
@@ -279,6 +279,7 @@ jobs:
echo "The repository is ${{ steps.get-pr.outputs.repo }}"
cmd:
+ timeout-minutes: 1800 # 30 hours as it could take a long time to run all the runtimes/pallets
needs: [set-image, get-pr-branch]
env:
JOB_NAME: "cmd"
From a61d700c6e6f199e901015788698a94eda7219c0 Mon Sep 17 00:00:00 2001
From: Javier Viola <363911+pepoviola@users.noreply.github.com>
Date: Tue, 8 Oct 2024 13:16:36 -0300
Subject: [PATCH 07/10] bump zombienet version and set request for k8s (#5968)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Bump zombienet version, including fixes for `ci` and set _resources
requests_ for the runner.
Thx!
Co-authored-by: Bastian Köcher
---
.gitlab/pipeline/zombienet.yml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/.gitlab/pipeline/zombienet.yml b/.gitlab/pipeline/zombienet.yml
index b02d857f3f241..2a7e2e7559a03 100644
--- a/.gitlab/pipeline/zombienet.yml
+++ b/.gitlab/pipeline/zombienet.yml
@@ -1,12 +1,14 @@
.zombienet-refs:
extends: .build-refs
variables:
- ZOMBIENET_IMAGE: "docker.io/paritytech/zombienet:v1.3.113"
+ ZOMBIENET_IMAGE: "docker.io/paritytech/zombienet:v1.3.114"
PUSHGATEWAY_URL: "http://zombienet-prometheus-pushgateway.managed-monitoring:9091/metrics/job/zombie-metrics"
DEBUG: "zombie,zombie::network-node,zombie::kube::client::logs"
ZOMBIE_PROVIDER: "k8s"
RUST_LOG: "info,zombienet_orchestrator=debug"
RUN_IN_CI: "1"
+ KUBERNETES_CPU_REQUEST: "512m"
+ KUBERNETES_MEMORY_REQUEST: "512M"
timeout: 60m
include:
From aecfebf61487df6eaeb1cd6c3ebee5921c95d976 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bastian=20K=C3=B6cher?=
Date: Tue, 8 Oct 2024 22:56:13 +0200
Subject: [PATCH 08/10] Bump some dependencies (#5886)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
This bumps `ethbloom`, `ethereum-types`, `primitive-types` and `rlp` to
their latest version.
Fixes: https://github.com/paritytech/polkadot-sdk/issues/5870
---------
Co-authored-by: command-bot <>
Co-authored-by: ggwpez
Co-authored-by: Oliver Tale-Yazdi
Co-authored-by: Shawn Tabrizi
Co-authored-by: Dónal Murray
---
Cargo.lock | 224 ++++++++++--------
Cargo.toml | 9 +-
bridges/primitives/polkadot-core/Cargo.toml | 2 -
.../polkadot-core/src/parachains.rs | 5 +-
prdoc/pr_5886.prdoc | 18 ++
.../client/consensus/babe/src/authorship.rs | 3 +-
substrate/frame/babe/src/mock.rs | 2 +-
substrate/frame/revive/src/wasm/runtime.rs | 4 +-
substrate/frame/sassafras/src/mock.rs | 2 +-
substrate/primitives/runtime/src/testing.rs | 8 +-
10 files changed, 156 insertions(+), 121 deletions(-)
create mode 100644 prdoc/pr_5886.prdoc
diff --git a/Cargo.lock b/Cargo.lock
index 4d5565e433b76..34e30d79e98b0 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -900,7 +900,7 @@ dependencies = [
"parity-scale-codec",
"polkadot-parachain-primitives",
"polkadot-runtime-common",
- "primitive-types",
+ "primitive-types 0.13.1",
"rococo-runtime-constants",
"scale-info",
"serde_json",
@@ -1033,7 +1033,7 @@ dependencies = [
"parity-scale-codec",
"polkadot-parachain-primitives",
"polkadot-runtime-common",
- "primitive-types",
+ "primitive-types 0.13.1",
"scale-info",
"serde_json",
"snowbridge-router-primitives",
@@ -2048,7 +2048,6 @@ dependencies = [
"frame-system",
"hex",
"parity-scale-codec",
- "parity-util-mem",
"scale-info",
"serde",
"sp-core 28.0.0",
@@ -5756,9 +5755,9 @@ dependencies = [
[[package]]
name = "ethabi-decode"
-version = "1.0.0"
+version = "1.1.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "09d398648d65820a727d6a81e58b962f874473396a047e4c30bafe3240953417"
+checksum = "f9af52ec57c5147716872863c2567c886e7d62f539465b94352dbc0108fe5293"
dependencies = [
"ethereum-types",
"tiny-keccak",
@@ -5766,33 +5765,33 @@ dependencies = [
[[package]]
name = "ethbloom"
-version = "0.13.0"
+version = "0.14.1"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c22d4b5885b6aa2fe5e8b9329fb8d232bf739e434e6b87347c63bdd00c120f60"
+checksum = "8c321610643004cf908ec0f5f2aa0d8f1f8e14b540562a2887a1111ff1ecbf7b"
dependencies = [
"crunchy",
"fixed-hash",
- "impl-codec",
+ "impl-codec 0.7.0",
"impl-rlp",
- "impl-serde",
+ "impl-serde 0.5.0",
"scale-info",
"tiny-keccak",
]
[[package]]
name = "ethereum-types"
-version = "0.14.1"
+version = "0.15.1"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "02d215cbf040552efcbe99a38372fe80ab9d00268e20012b79fcd0f073edd8ee"
+checksum = "1ab15ed80916029f878e0267c3a9f92b67df55e79af370bf66199059ae2b4ee3"
dependencies = [
"ethbloom",
"fixed-hash",
- "impl-codec",
+ "impl-codec 0.7.0",
"impl-rlp",
- "impl-serde",
- "primitive-types",
+ "impl-serde 0.5.0",
+ "primitive-types 0.13.1",
"scale-info",
- "uint",
+ "uint 0.10.0",
]
[[package]]
@@ -7610,6 +7609,15 @@ dependencies = [
"parity-scale-codec",
]
+[[package]]
+name = "impl-codec"
+version = "0.7.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b67aa010c1e3da95bf151bd8b4c059b2ed7e75387cdb969b4f8f2723a43f9941"
+dependencies = [
+ "parity-scale-codec",
+]
+
[[package]]
name = "impl-num-traits"
version = "0.1.2"
@@ -7618,16 +7626,27 @@ checksum = "951641f13f873bff03d4bf19ae8bec531935ac0ac2cc775f84d7edfdcfed3f17"
dependencies = [
"integer-sqrt",
"num-traits",
- "uint",
+ "uint 0.9.5",
+]
+
+[[package]]
+name = "impl-num-traits"
+version = "0.2.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "803d15461ab0dcc56706adf266158acbc44ccf719bf7d0af30705f58b90a4b8c"
+dependencies = [
+ "integer-sqrt",
+ "num-traits",
+ "uint 0.10.0",
]
[[package]]
name = "impl-rlp"
-version = "0.3.0"
+version = "0.4.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f28220f89297a075ddc7245cd538076ee98b01f2a9c23a53a4f1105d5a322808"
+checksum = "54ed8ad1f3877f7e775b8cbf30ed1bd3209a95401817f19a0eb4402d13f8cf90"
dependencies = [
- "rlp",
+ "rlp 0.6.1",
]
[[package]]
@@ -7639,6 +7658,15 @@ dependencies = [
"serde",
]
+[[package]]
+name = "impl-serde"
+version = "0.5.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "4a143eada6a1ec4aefa5049037a26a6d597bfd64f8c026d07b77133e02b7dd0b"
+dependencies = [
+ "serde",
+]
+
[[package]]
name = "impl-trait-for-tuples"
version = "0.2.2"
@@ -8366,7 +8394,7 @@ dependencies = [
"pallet-example-tasks",
"parity-scale-codec",
"polkadot-sdk",
- "primitive-types",
+ "primitive-types 0.13.1",
"scale-info",
"serde_json",
"static_assertions",
@@ -8779,7 +8807,7 @@ dependencies = [
"sha2 0.10.8",
"smallvec",
"thiserror",
- "uint",
+ "uint 0.9.5",
"unsigned-varint 0.7.2",
"void",
]
@@ -9242,7 +9270,7 @@ dependencies = [
"tokio-tungstenite",
"tokio-util",
"tracing",
- "uint",
+ "uint 0.9.5",
"unsigned-varint 0.8.0",
"url",
"x25519-dalek",
@@ -9277,15 +9305,6 @@ dependencies = [
"value-bag",
]
-[[package]]
-name = "lru"
-version = "0.8.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b6e8aaa3f231bb4bd57b84b2d5dc3ae7f350265df8aa96492e0bc394a1571909"
-dependencies = [
- "hashbrown 0.12.3",
-]
-
[[package]]
name = "lru"
version = "0.11.0"
@@ -10628,7 +10647,7 @@ dependencies = [
"pallet-assets",
"pallet-balances",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.13.1",
"scale-info",
"sp-api 26.0.0",
"sp-arithmetic 23.0.0",
@@ -10649,7 +10668,7 @@ dependencies = [
"pallet-assets",
"pallet-balances",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.13.1",
"scale-info",
"sp-arithmetic 23.0.0",
"sp-core 28.0.0",
@@ -12313,7 +12332,7 @@ dependencies = [
"polkavm 0.12.0",
"polkavm-common 0.12.0",
"pretty_assertions",
- "rlp",
+ "rlp 0.6.1",
"scale-info",
"serde",
"sp-api 26.0.0",
@@ -13223,7 +13242,7 @@ checksum = "4e69bf016dc406eff7d53a7d3f7cf1c2e72c82b9088aac1118591e36dd2cd3e9"
dependencies = [
"bitcoin_hashes 0.13.0",
"rand",
- "rand_core 0.6.4",
+ "rand_core 0.5.1",
"serde",
"unicode-normalization",
]
@@ -13281,35 +13300,6 @@ dependencies = [
"syn 1.0.109",
]
-[[package]]
-name = "parity-util-mem"
-version = "0.12.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "0d32c34f4f5ca7f9196001c0aba5a1f9a5a12382c8944b8b0f90233282d1e8f8"
-dependencies = [
- "cfg-if",
- "ethereum-types",
- "hashbrown 0.12.3",
- "impl-trait-for-tuples",
- "lru 0.8.1",
- "parity-util-mem-derive",
- "parking_lot 0.12.3",
- "primitive-types",
- "smallvec",
- "winapi",
-]
-
-[[package]]
-name = "parity-util-mem-derive"
-version = "0.1.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f557c32c6d268a07c921471619c0295f5efad3a0e76d4f97a05c091a51d110b2"
-dependencies = [
- "proc-macro2 1.0.86",
- "syn 1.0.109",
- "synstructure 0.12.6",
-]
-
[[package]]
name = "parity-wasm"
version = "0.45.0"
@@ -13481,7 +13471,7 @@ dependencies = [
"polkadot-parachain-primitives",
"polkadot-primitives",
"polkadot-runtime-common",
- "primitive-types",
+ "primitive-types 0.12.2",
"scale-info",
"smallvec",
"sp-api 26.0.0",
@@ -16523,12 +16513,26 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "0b34d9fd68ae0b74a41b21c03c2f62847aa0ffea044eee893b4c140b37e244e2"
dependencies = [
"fixed-hash",
- "impl-codec",
- "impl-num-traits",
+ "impl-codec 0.6.0",
+ "impl-num-traits 0.1.2",
+ "impl-serde 0.4.0",
+ "scale-info",
+ "uint 0.9.5",
+]
+
+[[package]]
+name = "primitive-types"
+version = "0.13.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d15600a7d856470b7d278b3fe0e311fe28c2526348549f8ef2ff7db3299c87f5"
+dependencies = [
+ "fixed-hash",
+ "impl-codec 0.7.0",
+ "impl-num-traits 0.2.0",
"impl-rlp",
- "impl-serde",
+ "impl-serde 0.5.0",
"scale-info",
- "uint",
+ "uint 0.10.0",
]
[[package]]
@@ -17640,6 +17644,16 @@ dependencies = [
"rustc-hex",
]
+[[package]]
+name = "rlp"
+version = "0.6.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "fa24e92bb2a83198bb76d661a71df9f7076b8c420b8696e4d3d97d50d94479e3"
+dependencies = [
+ "bytes",
+ "rustc-hex",
+]
+
[[package]]
name = "rocksdb"
version = "0.21.0"
@@ -17950,10 +17964,10 @@ dependencies = [
"num-bigint",
"num-traits",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.12.2",
"proptest",
"rand",
- "rlp",
+ "rlp 0.5.2",
"ruint-macro",
"serde",
"valuable",
@@ -19883,7 +19897,7 @@ checksum = "e98f3262c250d90e700bb802eb704e1f841e03331c2eb815e46516c4edbf5b27"
dependencies = [
"derive_more",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.12.2",
"scale-bits",
"scale-decode-derive",
"scale-type-resolver",
@@ -19910,7 +19924,7 @@ checksum = "4ba0b9c48dc0eb20c60b083c29447c0c4617cb7c4a4c9fef72aa5c5bc539e15e"
dependencies = [
"derive_more",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.12.2",
"scale-bits",
"scale-encode-derive",
"scale-type-resolver",
@@ -20872,7 +20886,7 @@ dependencies = [
"hex",
"hex-literal",
"parity-scale-codec",
- "rlp",
+ "rlp 0.6.1",
"scale-info",
"serde",
"snowbridge-ethereum",
@@ -20920,7 +20934,7 @@ dependencies = [
"parity-bytes",
"parity-scale-codec",
"rand",
- "rlp",
+ "rlp 0.6.1",
"scale-info",
"serde",
"serde-big-array",
@@ -21478,7 +21492,7 @@ dependencies = [
"integer-sqrt",
"num-traits",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.13.1",
"rand",
"scale-info",
"serde",
@@ -21718,7 +21732,7 @@ dependencies = [
"futures",
"hash-db",
"hash256-std-hasher",
- "impl-serde",
+ "impl-serde 0.4.0",
"itertools 0.11.0",
"k256",
"libsecp256k1",
@@ -21728,7 +21742,7 @@ dependencies = [
"parity-scale-codec",
"parking_lot 0.12.3",
"paste",
- "primitive-types",
+ "primitive-types 0.13.1",
"rand",
"regex",
"scale-info",
@@ -21767,7 +21781,7 @@ dependencies = [
"futures",
"hash-db",
"hash256-std-hasher",
- "impl-serde",
+ "impl-serde 0.4.0",
"itertools 0.10.5",
"k256",
"libsecp256k1",
@@ -21777,7 +21791,7 @@ dependencies = [
"parity-scale-codec",
"parking_lot 0.12.3",
"paste",
- "primitive-types",
+ "primitive-types 0.12.2",
"rand",
"scale-info",
"schnorrkel 0.11.4",
@@ -21814,7 +21828,7 @@ dependencies = [
"futures",
"hash-db",
"hash256-std-hasher",
- "impl-serde",
+ "impl-serde 0.4.0",
"itertools 0.10.5",
"k256",
"libsecp256k1",
@@ -21824,7 +21838,7 @@ dependencies = [
"parity-scale-codec",
"parking_lot 0.12.3",
"paste",
- "primitive-types",
+ "primitive-types 0.12.2",
"rand",
"scale-info",
"schnorrkel 0.11.4",
@@ -21861,7 +21875,7 @@ dependencies = [
"futures",
"hash-db",
"hash256-std-hasher",
- "impl-serde",
+ "impl-serde 0.4.0",
"itertools 0.11.0",
"k256",
"libsecp256k1",
@@ -21871,7 +21885,7 @@ dependencies = [
"parity-scale-codec",
"parking_lot 0.12.3",
"paste",
- "primitive-types",
+ "primitive-types 0.12.2",
"rand",
"scale-info",
"schnorrkel 0.11.4",
@@ -22516,7 +22530,7 @@ dependencies = [
"bytes",
"impl-trait-for-tuples",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.12.2",
"sp-externalities 0.19.0",
"sp-runtime-interface-proc-macro 11.0.0",
"sp-std 8.0.0",
@@ -22534,7 +22548,7 @@ dependencies = [
"impl-trait-for-tuples",
"parity-scale-codec",
"polkavm-derive 0.9.1",
- "primitive-types",
+ "primitive-types 0.13.1",
"rustversion",
"sp-core 28.0.0",
"sp-externalities 0.25.0",
@@ -22560,7 +22574,7 @@ dependencies = [
"impl-trait-for-tuples",
"parity-scale-codec",
"polkavm-derive 0.8.0",
- "primitive-types",
+ "primitive-types 0.12.2",
"sp-externalities 0.27.0",
"sp-runtime-interface-proc-macro 18.0.0",
"sp-std 14.0.0 (registry+https://github.com/rust-lang/crates.io-index)",
@@ -22580,7 +22594,7 @@ dependencies = [
"impl-trait-for-tuples",
"parity-scale-codec",
"polkavm-derive 0.9.1",
- "primitive-types",
+ "primitive-types 0.12.2",
"sp-externalities 0.28.0",
"sp-runtime-interface-proc-macro 18.0.0",
"sp-std 14.0.0 (registry+https://github.com/rust-lang/crates.io-index)",
@@ -22821,7 +22835,7 @@ name = "sp-storage"
version = "13.0.0"
source = "git+https://github.com/paritytech/polkadot-sdk#82912acb33a9030c0ef3bf590a34fca09b72dc5f"
dependencies = [
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
"ref-cast",
"serde",
@@ -22833,7 +22847,7 @@ dependencies = [
name = "sp-storage"
version = "19.0.0"
dependencies = [
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
"ref-cast",
"serde",
@@ -22846,7 +22860,7 @@ version = "20.0.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "8dba5791cb3978e95daf99dad919ecb3ec35565604e88cd38d805d9d4981e8bd"
dependencies = [
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
"ref-cast",
"serde",
@@ -22860,7 +22874,7 @@ version = "21.0.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "99c82989b3a4979a7e1ad848aad9f5d0b4388f1f454cc131766526601ab9e8f8"
dependencies = [
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
"ref-cast",
"serde",
@@ -23061,7 +23075,7 @@ dependencies = [
name = "sp-version"
version = "29.0.0"
dependencies = [
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
"parity-wasm",
"scale-info",
@@ -23079,7 +23093,7 @@ version = "35.0.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "6ff74bf12b4f7d29387eb1caeec5553209a505f90a2511d2831143b970f89659"
dependencies = [
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
"parity-wasm",
"scale-info",
@@ -23413,7 +23427,7 @@ dependencies = [
"polkadot-primitives",
"polkadot-runtime-parachains",
"polkadot-test-runtime",
- "primitive-types",
+ "primitive-types 0.13.1",
"scale-info",
"sp-arithmetic 23.0.0",
"sp-core 28.0.0",
@@ -24038,11 +24052,11 @@ dependencies = [
"frame-metadata 16.0.0",
"futures",
"hex",
- "impl-serde",
+ "impl-serde 0.4.0",
"instant",
"jsonrpsee 0.22.5",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.12.2",
"reconnecting-jsonrpsee-ws-client",
"scale-bits",
"scale-decode",
@@ -24095,9 +24109,9 @@ dependencies = [
"frame-metadata 16.0.0",
"hashbrown 0.14.5",
"hex",
- "impl-serde",
+ "impl-serde 0.4.0",
"parity-scale-codec",
- "primitive-types",
+ "primitive-types 0.12.2",
"scale-bits",
"scale-decode",
"scale-encode",
@@ -25448,6 +25462,18 @@ dependencies = [
"static_assertions",
]
+[[package]]
+name = "uint"
+version = "0.10.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "909988d098b2f738727b161a106cfc7cab00c539c2687a8836f8e565976fb53e"
+dependencies = [
+ "byteorder",
+ "crunchy",
+ "hex",
+ "static_assertions",
+]
+
[[package]]
name = "unarray"
version = "0.1.4"
diff --git a/Cargo.toml b/Cargo.toml
index bd37dee51460b..a52cdb99b0e92 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -741,8 +741,8 @@ env_logger = { version = "0.11.2" }
environmental = { version = "1.1.4", default-features = false }
equivocation-detector = { path = "bridges/relays/equivocation" }
ethabi = { version = "1.0.0", default-features = false, package = "ethabi-decode" }
-ethbloom = { version = "0.13.0", default-features = false }
-ethereum-types = { version = "0.14.1", default-features = false }
+ethbloom = { version = "0.14.1", default-features = false }
+ethereum-types = { version = "0.15.1", default-features = false }
exit-future = { version = "0.2.0" }
expander = { version = "2.0.0" }
fatality = { version = "0.1.1" }
@@ -1003,7 +1003,6 @@ parachains-relay = { path = "bridges/relays/parachains" }
parachains-runtimes-test-utils = { path = "cumulus/parachains/runtimes/test-utils", default-features = false }
parity-bytes = { version = "0.1.2", default-features = false }
parity-db = { version = "0.4.12" }
-parity-util-mem = { version = "0.12.0" }
parity-wasm = { version = "0.45.0" }
parking_lot = { version = "0.12.1", default-features = false }
partial_sort = { version = "0.2.0" }
@@ -1078,7 +1077,7 @@ polkavm-derive = "0.9.1"
polkavm-linker = "0.9.2"
portpicker = { version = "0.1.1" }
pretty_assertions = { version = "1.3.0" }
-primitive-types = { version = "0.12.1", default-features = false }
+primitive-types = { version = "0.13.1", default-features = false, features = ["num-traits"] }
proc-macro-crate = { version = "3.0.0" }
proc-macro-warning = { version = "1.0.0", default-features = false }
proc-macro2 = { version = "1.0.86" }
@@ -1106,7 +1105,7 @@ relay-substrate-client = { path = "bridges/relays/client-substrate" }
relay-utils = { path = "bridges/relays/utils" }
remote-externalities = { path = "substrate/utils/frame/remote-externalities", default-features = false, package = "frame-remote-externalities" }
reqwest = { version = "0.11", default-features = false }
-rlp = { version = "0.5.2", default-features = false }
+rlp = { version = "0.6.1", default-features = false }
rococo-emulated-chain = { path = "cumulus/parachains/integration-tests/emulated/chains/relays/rococo" }
rococo-parachain-runtime = { path = "cumulus/parachains/runtimes/testing/rococo-parachain" }
rococo-runtime = { path = "polkadot/runtime/rococo" }
diff --git a/bridges/primitives/polkadot-core/Cargo.toml b/bridges/primitives/polkadot-core/Cargo.toml
index 366ee7aa948e8..295fb281e9bbc 100644
--- a/bridges/primitives/polkadot-core/Cargo.toml
+++ b/bridges/primitives/polkadot-core/Cargo.toml
@@ -12,7 +12,6 @@ workspace = true
[dependencies]
codec = { features = ["derive"], workspace = true }
-parity-util-mem = { optional = true, workspace = true }
scale-info = { features = ["derive"], workspace = true }
serde = { optional = true, features = [
"derive",
@@ -42,7 +41,6 @@ std = [
"codec/std",
"frame-support/std",
"frame-system/std",
- "parity-util-mem",
"scale-info/std",
"serde",
"sp-core/std",
diff --git a/bridges/primitives/polkadot-core/src/parachains.rs b/bridges/primitives/polkadot-core/src/parachains.rs
index d54ee108386ed..a8b1cf6eebf49 100644
--- a/bridges/primitives/polkadot-core/src/parachains.rs
+++ b/bridges/primitives/polkadot-core/src/parachains.rs
@@ -32,9 +32,6 @@ use sp_std::vec::Vec;
#[cfg(feature = "std")]
use serde::{Deserialize, Serialize};
-#[cfg(feature = "std")]
-use parity_util_mem::MallocSizeOf;
-
/// Parachain id.
///
/// This is an equivalent of the `polkadot_parachain_primitives::Id`, which is a compact-encoded
@@ -71,7 +68,7 @@ impl From for ParaId {
#[derive(
PartialEq, Eq, Clone, PartialOrd, Ord, Encode, Decode, RuntimeDebug, TypeInfo, Default,
)]
-#[cfg_attr(feature = "std", derive(Serialize, Deserialize, Hash, MallocSizeOf))]
+#[cfg_attr(feature = "std", derive(Serialize, Deserialize, Hash))]
pub struct ParaHead(pub Vec);
impl ParaHead {
diff --git a/prdoc/pr_5886.prdoc b/prdoc/pr_5886.prdoc
new file mode 100644
index 0000000000000..f5e5972811978
--- /dev/null
+++ b/prdoc/pr_5886.prdoc
@@ -0,0 +1,18 @@
+title: Bump some dependencies
+doc:
+- audience: Runtime Dev
+ description: |-
+ This bumps `ethbloom`, `ethereum-types`, `primitive-types` and `rlp` to their latest version.
+
+ Fixes: https://github.com/paritytech/polkadot-sdk/issues/5870
+crates:
+- name: sc-consensus-babe
+ bump: patch
+- name: pallet-babe
+ bump: patch
+- name: pallet-revive
+ bump: patch
+- name: sp-runtime
+ bump: patch
+- name: bp-polkadot-core
+ bump: major
diff --git a/substrate/client/consensus/babe/src/authorship.rs b/substrate/client/consensus/babe/src/authorship.rs
index 57ee706a04f6b..06394b84f4700 100644
--- a/substrate/client/consensus/babe/src/authorship.rs
+++ b/substrate/client/consensus/babe/src/authorship.rs
@@ -108,7 +108,8 @@ pub(super) fn secondary_slot_author(
return None
}
- let rand = U256::from((randomness, slot).using_encoded(sp_crypto_hashing::blake2_256));
+ let rand =
+ U256::from_little_endian(&(randomness, slot).using_encoded(sp_crypto_hashing::blake2_256));
let authorities_len = U256::from(authorities.len());
let idx = rand % authorities_len;
diff --git a/substrate/frame/babe/src/mock.rs b/substrate/frame/babe/src/mock.rs
index 4e4052b2b566e..d416e31b25f71 100644
--- a/substrate/frame/babe/src/mock.rs
+++ b/substrate/frame/babe/src/mock.rs
@@ -291,7 +291,7 @@ pub fn new_test_ext_with_pairs(
authorities_len: usize,
) -> (Vec, sp_io::TestExternalities) {
let pairs = (0..authorities_len)
- .map(|i| AuthorityPair::from_seed(&U256::from(i).into()))
+ .map(|i| AuthorityPair::from_seed(&U256::from(i).to_little_endian()))
.collect::>();
let public = pairs.iter().map(|p| p.public()).collect();
diff --git a/substrate/frame/revive/src/wasm/runtime.rs b/substrate/frame/revive/src/wasm/runtime.rs
index 485abbfda225a..245c91278a7f8 100644
--- a/substrate/frame/revive/src/wasm/runtime.rs
+++ b/substrate/frame/revive/src/wasm/runtime.rs
@@ -46,9 +46,7 @@ const MAX_DECODE_NESTING: u32 = 256;
/// Encode a `U256` into a 32 byte buffer.
fn as_bytes(u: U256) -> [u8; 32] {
- let mut bytes = [0u8; 32];
- u.to_little_endian(&mut bytes);
- bytes
+ u.to_little_endian()
}
#[derive(Clone, Copy)]
diff --git a/substrate/frame/sassafras/src/mock.rs b/substrate/frame/sassafras/src/mock.rs
index f145bffa3a05c..d2b329e8a2ba1 100644
--- a/substrate/frame/sassafras/src/mock.rs
+++ b/substrate/frame/sassafras/src/mock.rs
@@ -89,7 +89,7 @@ pub fn new_test_ext_with_pairs(
with_ring_context: bool,
) -> (Vec, sp_io::TestExternalities) {
let pairs = (0..authorities_len)
- .map(|i| AuthorityPair::from_seed(&U256::from(i).into()))
+ .map(|i| AuthorityPair::from_seed(&U256::from(i).to_big_endian()))
.collect::>();
let authorities: Vec<_> = pairs.iter().map(|p| p.public()).collect();
diff --git a/substrate/primitives/runtime/src/testing.rs b/substrate/primitives/runtime/src/testing.rs
index b4aeda5a0e7a3..a4ce4b5fc1a05 100644
--- a/substrate/primitives/runtime/src/testing.rs
+++ b/substrate/primitives/runtime/src/testing.rs
@@ -29,10 +29,7 @@ use crate::{
ApplyExtrinsicResultWithInfo, KeyTypeId,
};
use serde::{de::Error as DeError, Deserialize, Deserializer, Serialize, Serializer};
-use sp_core::{
- crypto::{key_types, ByteArray, CryptoType, Dummy},
- U256,
-};
+use sp_core::crypto::{key_types, ByteArray, CryptoType, Dummy};
pub use sp_core::{sr25519, H256};
use std::{
cell::RefCell,
@@ -79,7 +76,8 @@ impl From for u64 {
impl UintAuthorityId {
/// Convert this authority ID into a public key.
pub fn to_public_key(&self) -> T {
- let bytes: [u8; 32] = U256::from(self.0).into();
+ let mut bytes = [0u8; 32];
+ bytes[0..8].copy_from_slice(&self.0.to_le_bytes());
T::from_slice(&bytes).unwrap()
}
}
From 8eda10a8aac88cd064c08f84d20e06ed0b2b8540 Mon Sep 17 00:00:00 2001
From: Javier Viola <363911+pepoviola@users.noreply.github.com>
Date: Wed, 9 Oct 2024 03:14:09 -0300
Subject: [PATCH 09/10] Disable flaky tests reported in 5972/5973/5974 (#5976)
Disable flaky tests reported in:
#5972
#5973
#5974
---
.gitlab/pipeline/zombienet/polkadot.yml | 6 +++---
.gitlab/pipeline/zombienet/substrate.yml | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/.gitlab/pipeline/zombienet/polkadot.yml b/.gitlab/pipeline/zombienet/polkadot.yml
index 0da95f224cfcb..33191d99941d6 100644
--- a/.gitlab/pipeline/zombienet/polkadot.yml
+++ b/.gitlab/pipeline/zombienet/polkadot.yml
@@ -94,7 +94,7 @@ zombienet-polkadot-functional-0002-parachains-disputes:
--local-dir="${LOCAL_DIR}/functional"
--test="0002-parachains-disputes.zndsl"
-zombienet-polkadot-functional-0003-beefy-and-mmr:
+.zombienet-polkadot-functional-0003-beefy-and-mmr:
extends:
- .zombienet-polkadot-common
script:
@@ -190,7 +190,7 @@ zombienet-polkadot-elastic-scaling-0002-elastic-scaling-doesnt-break-parachains:
--local-dir="${LOCAL_DIR}/elastic_scaling"
--test="0002-elastic-scaling-doesnt-break-parachains.zndsl"
-zombienet-polkadot-functional-0012-spam-statement-distribution-requests:
+.zombienet-polkadot-functional-0012-spam-statement-distribution-requests:
extends:
- .zombienet-polkadot-common
script:
@@ -277,7 +277,7 @@ zombienet-polkadot-smoke-0002-parachains-parachains-upgrade-smoke:
--local-dir="${LOCAL_DIR}/smoke"
--test="0002-parachains-upgrade-smoke-test.zndsl"
-zombienet-polkadot-smoke-0003-deregister-register-validator:
+.zombienet-polkadot-smoke-0003-deregister-register-validator:
extends:
- .zombienet-polkadot-common
script:
diff --git a/.gitlab/pipeline/zombienet/substrate.yml b/.gitlab/pipeline/zombienet/substrate.yml
index d4ffa34f3395e..030a0a3f50a9f 100644
--- a/.gitlab/pipeline/zombienet/substrate.yml
+++ b/.gitlab/pipeline/zombienet/substrate.yml
@@ -70,7 +70,7 @@ zombienet-substrate-0001-basic-warp-sync:
--local-dir="${LOCAL_DIR}/0001-basic-warp-sync"
--test="test-warp-sync.zndsl"
-zombienet-substrate-0002-validators-warp-sync:
+.zombienet-substrate-0002-validators-warp-sync:
extends:
- .zombienet-substrate-warp-sync-common
before_script:
From d63cbdf0f205741cd31cb3ea46befe12f3e92adf Mon Sep 17 00:00:00 2001
From: Maksym H <1177472+mordamax@users.noreply.github.com>
Date: Wed, 9 Oct 2024 12:48:57 +0100
Subject: [PATCH 10/10] Update README.md
---
README.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/README.md b/README.md
index 8016b6b37301a..0fe2618746236 100644
--- a/README.md
+++ b/README.md
@@ -1,4 +1,4 @@
-
+1

