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.

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

build release runtime with production profile #2549

Merged
merged 2 commits into from
Nov 2, 2023
Merged

Conversation

librelois
Copy link
Collaborator

What does it do?

We now generate the weights with the production profile, so we need to build the release runtime with the same profile (see paritytech/polkadot-sdk#2111)

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@librelois librelois added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit labels Nov 2, 2023
@librelois librelois added the not-breaking Does not need to be mentioned in breaking changes label Nov 2, 2023
@librelois librelois merged commit 3548f6a into master Nov 2, 2023
28 of 30 checks passed
@librelois librelois deleted the elois-runtime-prod branch November 2, 2023 13:28
@crystalin
Copy link
Collaborator

@noandrea: make sure we test those new runtimes properly as we have never run "production" runtime in the past

noandrea pushed a commit that referenced this pull request Nov 2, 2023
* build release runtime with production profile

* typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants