-
Notifications
You must be signed in to change notification settings - Fork 113
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
Update to polkadot v0.9.39 #4705
Changes from all commits
cccc77e
f74dc18
bb28c30
b770d81
ab2d5c9
eac1549
547cbf8
cc64367
e35ed91
3a0ab6c
bb22687
9b069a8
1033c1c
a2475af
7ab781e
b78efb4
c400c98
b51dd88
40cb698
8872a09
9697a84
caca06c
b5c2462
7f4adef
e811309
ff3989b
3ed4fae
d0a80bf
2762641
6e670eb
946f7d3
28ec59d
dfa917b
e8c4f29
d86aab3
f279314
26e0994
b39f7ed
687e447
d575b9e
5ed8e4d
7588b17
87bc9ab
5ed79e2
7594361
f83212f
3aa2861
6d18b00
602c2ec
677badf
2b7caee
6004cf6
2fbf578
85bf256
276eb02
9f67cc4
5462b1f
3182d28
8300116
5156465
3a79f1e
7642067
4a21288
dbbf2bc
9233342
4c40cf2
5f4b657
d8d852c
c8bbdb2
17e15f5
fda5c05
e3b6dc6
3887173
5a9daad
36f6031
8de82d4
e1fd7e7
b3fb6d6
d930492
914c4d8
354cf7d
5d5c9c1
85fd568
1efdd5e
b3eebca
7fc95a4
779c774
f936481
0825bd1
09fac54
2f82527
d53ddd4
51fb763
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -36,16 +36,15 @@ jobs: | |
|
||
- name: Build | ||
env: | ||
WASM_BUILD_TOOLCHAIN: nightly-2022-05-11 | ||
WASM_BUILD_TOOLCHAIN: nightly-2022-11-15 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Was this toolchain ver. recommended by substrate? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes the nightly version at time the version was released |
||
run: | | ||
cargo +nightly-2022-05-11 clippy --release --features runtime-benchmarks && | ||
cargo +nightly-2022-05-11 test --release --features runtime-benchmarks && | ||
cargo +nightly-2022-05-11 build --release --features runtime-benchmarks | ||
./scripts/cargo-checks-with-benchmarking.sh && | ||
./scripts/cargo-build-with-benchmarking.sh | ||
if: env.GIT_DIFF | ||
|
||
- name: Test Generate Weights | ||
run: | | ||
./scripts/generate-weights.sh 1 1 && git diff | ||
./scripts/generate-weights.sh 2 1 && git diff | ||
if: env.GIT_DIFF | ||
|
||
# Build and test joystream-node with all runtime profiles | ||
|
@@ -78,6 +77,19 @@ jobs: | |
yarn cargo-checks && yarn cargo-build | ||
./target/release/call-sizes | ||
if: env.GIT_DIFF | ||
- name: OnRuntimeUpgrade (try-runtime) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This make the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's a good point. |
||
run: | | ||
cargo +nightly-2022-11-15 build --release --features try-runtime | ||
RUST_LOG=info,runtime=debug ./target/release/joystream-node try-runtime \ | ||
--runtime ./target/release/wbuild/joystream-node-runtime/joystream_node_runtime.compact.compressed.wasm \ | ||
on-runtime-upgrade \ | ||
live \ | ||
--uri wss://rpc.joystream.org:9944/ | ||
if: env.GIT_DIFF | ||
- name: run_migrations test | ||
run: | | ||
RUN_MIGRATION_TESTS=true RUST_LOG=info cargo +nightly-2022-11-15 test --release --features try-runtime | ||
if: env.GIT_DIFF | ||
- name: Staging Runtime | ||
env: | ||
RUNTIME_PROFILE: "STAGING" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you said "only amd64" above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The line is already commented out, but left it for reference if we want to add it back.
It might be better to drop this whole workflow and improve
joystream-node-docker.yml
which deploys two worker machines on AWS (amd64 and arm64) to build natively for each platform arch have the ansible playbook build for all the runtime profiles.