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

[CI] Build wasm blob with srtool and include prop hashes and blobs in release notes #1506

Merged
merged 19 commits into from Jul 31, 2020
Merged

Conversation

s3krit
Copy link
Contributor

@s3krit s3krit commented Jul 30, 2020

Previously, building the WASM blobs was done on the cleanroom server, manually initiated by me. Since we have reproducible builds of the WASM blobs using srtool now, this isn't necessary.

This PR includes the building of those blobs in Github Actions - using the Github-provided runners, this only takes 30 minutes for a clean build (around 10 minutes after that depending on caching, which is implemented) - which is faster than the Gitlab pipeline takes from start to finish, so this doesn't actually delay anything.

One important change is that now, the creation of the draft-release on tagging will only happen after the WASM blobs are built - which means there will be that 10-35 minute gap between pushing a tag and the draft release being created (and the #polkadot matrix channel being notified).

An example of the format of the release can be found here:

https://github.com/s3krit/polkadot/releases/tag/v0.8.19

@s3krit s3krit added 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. labels Jul 30, 2020
@gavofyork gavofyork merged commit 75acb33 into paritytech:master Jul 31, 2020
@s3krit s3krit deleted the mp-srtool branch July 31, 2020 13:44
ordian added a commit that referenced this pull request Jul 31, 2020
* master:
  guide: collator networking & subsystems (#1452)
  Guide: add a diagram for Inclusion Pipeline & Approval Subsystem (#1457)
  [CI] Build wasm blob with srtool and include prop hashes and blobs in release notes (#1506)
matrix:
runtime: ['polkadot', 'kusama']
container:
image: chevdor/srtool:nightly-2020-07-20
Copy link
Contributor

Choose a reason for hiding this comment

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

afaiu according to our Docker images maintenance policy only official images or images based on them should be used.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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.

4 participants