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

Add build options to the srtool build step #4956

Merged
merged 24 commits into from
Aug 26, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
df3922d
add runtimes build options to srtool build
EgorPopelyaev Jun 27, 2024
f611426
add srtool build with options
EgorPopelyaev Jul 5, 2024
a45ad83
add additional step to set build options
EgorPopelyaev Jul 5, 2024
8b4b2f2
add qoutes
EgorPopelyaev Jul 5, 2024
defe4a4
change scope of BUILD_OPTIONS
EgorPopelyaev Jul 5, 2024
8b08e5f
chaneg build optas and add a job to build runtimes with logging activ…
EgorPopelyaev Jul 8, 2024
79d3474
add on-chain-release-build feature to where it is missing
EgorPopelyaev Jul 9, 2024
aca9c75
Merge branch 'master' into ep-add-build-opts-to-release
EgorPopelyaev Jul 9, 2024
213574f
add upload runtiems without logging
EgorPopelyaev Jul 9, 2024
1ccbc2a
change call of the env
EgorPopelyaev Jul 9, 2024
6ad1e29
hardcode suffix as reusable workflow does not parse env
EgorPopelyaev Jul 9, 2024
8698c39
adjust path to release-notes-context.json
EgorPopelyaev Jul 10, 2024
9c02c32
Merge branch 'master' into ep-add-build-opts-to-release
bkchr Jul 17, 2024
9d236b7
Merge branch 'master' of https://github.com/paritytech/polkadot-sdk i…
EgorPopelyaev Jul 31, 2024
75720f7
Merge branch 'ep-add-build-opts-to-release' of https://github.com/Ego…
EgorPopelyaev Jul 31, 2024
cff9c73
delete `disable-logging` and adjust flow to build runtimes only one time
EgorPopelyaev Jul 31, 2024
063cba0
Merge branch 'master' into ep-add-build-opts-to-release
EgorPopelyaev Aug 6, 2024
26da67b
Merge branch 'master' of https://github.com/paritytech/polkadot-sdk i…
EgorPopelyaev Aug 12, 2024
9942d15
Merge branch 'ep-add-build-opts-to-release' of https://github.com/Ego…
EgorPopelyaev Aug 12, 2024
df7bf5a
Merge branch 'master' into ep-add-build-opts-to-release
EgorPopelyaev Aug 12, 2024
d7cf048
Merge branch 'master' of https://github.com/paritytech/polkadot-sdk i…
EgorPopelyaev Aug 22, 2024
6b9d737
Add PrDoc (auto generated)
EgorPopelyaev Aug 22, 2024
de219c9
Merge branch 'master' of https://github.com/paritytech/polkadot-sdk i…
EgorPopelyaev Aug 26, 2024
e48b99d
update prdoc
EgorPopelyaev Aug 26, 2024
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
1 change: 1 addition & 0 deletions .github/workflows/release-30_publish_release_draft.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ jobs:
uses: "./.github/workflows/release-srtool.yml"
with:
excluded_runtimes: "substrate-test bp cumulus-test kitchensink minimal-template parachain-template penpal polkadot-test seedling shell frame-try sp solochain-template"
build_opts: "--features on-chain-release-build"

build-binaries:
runs-on: ubuntu-latest
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/release-srtool.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ on:
inputs:
excluded_runtimes:
type: string
build_opts:
type: string
outputs:
published_runtimes:
value: ${{ jobs.find-runtimes.outputs.runtime }}
Expand Down Expand Up @@ -74,6 +76,8 @@ jobs:
- name: Srtool build
id: srtool_build
uses: chevdor/srtool-actions@v0.9.2
env:
BUILD_OPTS: ${{ inputs.build_opts }}
with:
chain: ${{ matrix.chain }}
runtime_dir: ${{ matrix.runtime_dir }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -258,4 +258,4 @@ metadata-hash = ["substrate-wasm-builder/metadata-hash"]
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["metadata-hash", "sp-api/disable-logging"]
on-chain-release-build = ["metadata-hash"]
Original file line number Diff line number Diff line change
Expand Up @@ -262,4 +262,4 @@ metadata-hash = ["substrate-wasm-builder/metadata-hash"]
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["metadata-hash", "sp-api/disable-logging"]
on-chain-release-build = ["metadata-hash"]
Original file line number Diff line number Diff line change
Expand Up @@ -305,4 +305,4 @@ fast-runtime = []
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["sp-api/disable-logging"]
on-chain-release-build = []
Original file line number Diff line number Diff line change
Expand Up @@ -292,6 +292,6 @@ try-runtime = [
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["sp-api/disable-logging"]
on-chain-release-build = []

fast-runtime = []
Original file line number Diff line number Diff line change
Expand Up @@ -243,4 +243,4 @@ std = [
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["sp-api/disable-logging"]
on-chain-release-build = []
Original file line number Diff line number Diff line change
Expand Up @@ -203,4 +203,4 @@ try-runtime = [
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["sp-api/disable-logging"]
on-chain-release-build = []
Original file line number Diff line number Diff line change
Expand Up @@ -209,4 +209,4 @@ metadata-hash = ["substrate-wasm-builder/metadata-hash"]
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["metadata-hash", "sp-api/disable-logging"]
on-chain-release-build = ["metadata-hash"]
Original file line number Diff line number Diff line change
Expand Up @@ -206,4 +206,4 @@ metadata-hash = ["substrate-wasm-builder/metadata-hash"]
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["metadata-hash", "sp-api/disable-logging"]
on-chain-release-build = ["metadata-hash"]
Original file line number Diff line number Diff line change
Expand Up @@ -136,4 +136,4 @@ try-runtime = [
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["sp-api/disable-logging"]
on-chain-release-build = []
5 changes: 5 additions & 0 deletions cumulus/parachains/runtimes/people/people-rococo/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -191,3 +191,8 @@ try-runtime = [
"polkadot-runtime-common/try-runtime",
"sp-runtime/try-runtime",
]

# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = []
5 changes: 5 additions & 0 deletions cumulus/parachains/runtimes/people/people-westend/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -191,3 +191,8 @@ try-runtime = [
"polkadot-runtime-common/try-runtime",
"sp-runtime/try-runtime",
]

# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = []
Original file line number Diff line number Diff line change
Expand Up @@ -134,3 +134,8 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
]

# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = []
2 changes: 1 addition & 1 deletion polkadot/runtime/rococo/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -339,4 +339,4 @@ runtime-metrics = [
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["metadata-hash", "sp-api/disable-logging"]
on-chain-release-build = ["metadata-hash"]
2 changes: 1 addition & 1 deletion polkadot/runtime/westend/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -358,4 +358,4 @@ runtime-metrics = [
# A feature that should be enabled when the runtime should be built for on-chain
# deployment. This will disable stuff that shouldn't be part of the on-chain wasm
# to make it smaller, like logging for example.
on-chain-release-build = ["metadata-hash", "sp-api/disable-logging"]
on-chain-release-build = ["metadata-hash"]
39 changes: 39 additions & 0 deletions prdoc/pr_4956.prdoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
title: Add build options to the srtool build step and delete `disanle-logging` feature

doc:
- audience: Runtime Dev
description: |
This PR adds possibility to set BUILD_OPTIONS to the "Srtool Build\" step in the release pipeline while building runtimes.
And deletes the `disable-logging` feature from test runtimes to be able to build those with an activated logging.



crates:
- name: people-rococo-runtime
bump: patch
- name: people-westend-runtime
bump: patch
- name: rococo-parachain-runtime
bump: patch
- name: asset-hub-rococo-runtime
bump: patch
- name: asset-hub-westend-runtime
bump: patch
- name: bridge-hub-rococo-runtime
bump: patch
- name: bridge-hub-westend-runtime
bump: patch
- name: collectives-westend-runtime
bump: patch
- name: contracts-rococo-runtime
bump: patch
- name: coretime-rococo-runtime
bump: patch
- name: coretime-westend-runtime
bump: patch
- name: glutton-westend-runtime
bump: patch
- name: rococo-runtime
bump: patch
- name: westend-runtime
bump: patch
Loading