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

[ci] Weights PR for master and client on cumulus #1553

Merged
merged 3 commits into from
Aug 22, 2022
Merged

[ci] Weights PR for master and client on cumulus #1553

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
93ae56c
[ci] Weights PR for master and client on cumulus
alvicsam Aug 17, 2022
159eca1
add 4th pr creation to benchmarks-statemint
alvicsam Aug 17, 2022
baf175e
rename benchmarks-statemint to benchmarks-assets
alvicsam Aug 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 20 additions & 5 deletions .gitlab-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,7 @@ benchmarks-build:
- git commit -m "[benchmarks] pr with weights"
- git push origin $BRANCHNAME

benchmarks-statemint:
benchmarks-assets:
stage: benchmarks-run
before_script:
- *rust-info-script
Expand All @@ -326,9 +326,24 @@ benchmarks-statemint:
- ./scripts/benchmarks-ci.sh assets westmint ./artifacts
- export BRANCHNAME="weights-statemint-${CI_COMMIT_BRANCH}"
- *git-commit-push
# create PR
# create PR to release-parachains-v* branch
- curl -u ${GITHUB_USER}:${GITHUB_TOKEN}
-d '{"title":"[benchmarks] Update weights for statemine/t","body":"This PR is generated automatically by CI.","head":"'${BRANCHNAME}'","base":"'${CI_COMMIT_BRANCH}'"}'
Copy link
Contributor

Choose a reason for hiding this comment

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

Are other runtimes done in another location ?
We need weights for westmint, rococo-parachain, etc...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, the name of the job is confusing after benchmarks-collectives was added. benchmarks-statemint is actually run for statemine, statemint and westmint. Do you have a suggestion for renaming?

Copy link
Contributor

Choose a reason for hiding this comment

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

Sure, in the new structure, all of those are under the assets category.
See https://github.com/paritytech/cumulus/tree/master/parachains/runtimes

-X POST https://api.github.com/repos/paritytech/${CI_PROJECT_NAME}/pulls
# create PR to master
- curl -u ${GITHUB_USER}:${GITHUB_TOKEN}
-d '{"title":"[benchmarks] Update weights for statemine/t","body":"This PR is generated automatically by CI.","head":"'${BRANCHNAME}'","base":"master"}'
-X POST https://api.github.com/repos/paritytech/${CI_PROJECT_NAME}/pulls
# create PR to a branch with version number (e.g. v0.9.270) and release-v* (e.g. release-v0.9.270)
# transform release-parachains-v9270 to v0.9.270
- export BASEBRANCH=$(echo ${CI_COMMIT_BRANCH} | cut -d "-" -f 3 | sed -e "s/\(.\)\(.\)\(...\)/\10.\2.\3/")
# create PR to v* branch
- curl -u ${GITHUB_USER}:${GITHUB_TOKEN}
-d '{"title":"[benchmarks] Update weights for statemine/t","body":"This PR is generated automatically by CI.","head":"'${BRANCHNAME}'","base":"'${BASEBRANCH}'"}'
-X POST https://api.github.com/repos/paritytech/${CI_PROJECT_NAME}/pulls
# create PR to release-v* branch
- curl -u ${GITHUB_USER}:${GITHUB_TOKEN}
-d '{"title":"[benchmarks] Update weights for statemine/t","body":"This PR is generated automatically by CI.","head":"'$BRANCHNAME'","base":"'${CI_COMMIT_BRANCH}'"}'
-d '{"title":"[benchmarks] Update weights for statemine/t","body":"This PR is generated automatically by CI.","head":"'${BRANCHNAME}'","base":"'release-${BASEBRANCH}'"}'
-X POST https://api.github.com/repos/paritytech/${CI_PROJECT_NAME}/pulls
after_script:
- rm -rf .git/config
Expand All @@ -355,13 +370,13 @@ benchmarks-collectives:
tags:
- weights

publish-benchmarks-statemint-s3: &publish-benchmarks
publish-benchmarks-assets-s3: &publish-benchmarks
stage: publish
<<: *kubernetes-env
image: paritytech/awscli:latest
<<: *benchmarks-refs
needs:
- job: benchmarks-statemint
- job: benchmarks-assets
artifacts: true
variables:
GIT_STRATEGY: none
Expand Down