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

[CI] Fix listing pallets for benchmarks jobs #3570

Merged
merged 2 commits into from
Aug 4, 2021
Merged

Conversation

s3krit
Copy link
Contributor

@s3krit s3krit commented Aug 4, 2021

Due to paritytech/substrate#9373 we now have a --list flag for the benchmarks command, and the previous method no longer works, yielding an empty list.

@s3krit s3krit added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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. labels Aug 4, 2021
@s3krit s3krit requested review from chevdor and TriplEight August 4, 2021 12:05
--extrinsic "*" \
--repeat 0 | \
sed -r -e 's/Pallet: "([a-z_:]+)".*/\1/' | \
--chain "${runtime}-dev" --list |\
Copy link
Contributor

Choose a reason for hiding this comment

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

You may want to keep the previous indentation to easily distinguish the cargo from the following piped calls.

@s3krit s3krit merged commit 00232dc into master Aug 4, 2021
@s3krit s3krit deleted the mp-fix-weights-job branch August 4, 2021 13:29
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. A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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.

2 participants