-
Notifications
You must be signed in to change notification settings - Fork 323
Merge devnet-ready into testnet #2831
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
Changes from all commits
f219958
ebee71b
8093069
58047d1
5a8ea76
f32902d
54e2ecd
35d077f
a7f9c72
eac6362
6421a6c
2aacb0f
1ced310
17c1529
522f58c
214be94
b5c3273
3698470
4cad05c
a5c329b
b855af3
3c6c002
6df5589
2662630
508cf8b
74d1cd9
69ad447
442261d
503c31d
4fa605f
dee84e2
ee67422
22cd7d4
15e1db6
557779a
6047135
2ef4e5b
1c10833
b0cad27
265fc76
b179fcc
ef36e94
a8f2bd3
baf979b
1c0572e
4910038
3b3340f
c7b2e95
695b504
3c07c24
7da9766
27b4354
b83efe5
a1378ff
3911188
5ffde0b
2931ff8
a8d37c8
8dac5f7
e7c2a57
f40907f
e2b0c03
d9e5b42
e20b873
0c3eb19
dbfa848
e00a9fb
03e454c
3ebc4b6
6d09b8b
f30674e
03c0d07
bd00dce
4c527f2
a03b14f
c079857
4cec05f
1177c28
c222820
89e9467
9ea5fb6
0fe3718
58879bd
20f9164
cc11248
440e627
f3ee244
1f536d7
fdf06f5
e1541f0
7494ba6
44c418b
916158c
7695b2f
4855bf3
07ffd42
2c8a3fd
f38c0db
23039e0
3e606f0
7994692
b37a197
bb71e41
977d534
3ef6894
7d2c726
096a405
bff5760
612d546
ab38313
7c45c97
d141ec2
8a4bad0
fb5a4cd
0cee4d1
069029f
2303adc
eb25e8f
59b467a
fd5e882
00e256b
3eedbd5
75b7b79
9efcbe7
d74b987
6eb4031
51d5e04
4bf9055
2c73b8f
f2c7bea
aeeefa5
1a5da03
fe25a01
06032d5
265a501
d4a6da2
5751675
d370972
424efa4
539200b
4ec382c
ffe705c
d1ae758
aecf5a1
2c82c55
171b5e5
2841b5e
e9050d4
77e6bfc
14cd19c
bc5c024
503170f
98151ec
593de8a
536c893
c4c77a2
8bc8ef6
566fb8c
d005de3
132b9cc
1ad1688
bb51677
1490758
5cd2a57
6d13cf3
724bb84
156eeca
a63880d
bf519d0
559e62c
f02efd6
e7f1bfd
34e1def
b749a63
ebe1ac6
6d81084
cb26860
a36fc33
992cfc6
1cb69bd
1c305d6
a5202f6
f6c858a
612e09b
08c1a21
7bdc3c0
1d5ccec
6c3c519
0caa50f
24f1d95
e5b15ca
d9bfd08
531e042
22842e4
d340436
bb3591e
3b94406
2519652
6506ac6
14bc6f9
5a06a94
c4b8e77
12d04c0
c7fad8d
6b7222a
6a4206f
a259fed
5f2c452
e8ed66e
0a6ebe6
b75479d
6dc423b
5b74a6d
4dd460c
1bb29ac
3200a6f
d10a8db
19b8435
e932dd4
177ffc7
57b8bc3
5c6de1f
6f8da13
4ae8304
bb676ae
e3a92aa
f72d089
8f0e7d4
e14e463
571848b
e072065
4cb0523
45999f7
d091d3a
621583e
7a8e436
22dd037
0c3d92d
f0fa8ba
7dfd2a4
d084d43
f039a34
54e6d49
4f3d5ff
8ab7f9c
820882a
45dbee4
53eb334
737e207
c192c06
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 |
|---|---|---|
| @@ -0,0 +1,99 @@ | ||
| name: "Subtensor Try Runtime" | ||
| description: "Check Subtensor runtime migrations using try-runtime-cli" | ||
| author: "OpenTensor Foundation" | ||
| branding: | ||
| icon: "shield" | ||
| color: "green" | ||
|
|
||
| inputs: | ||
| runtime-package: | ||
| description: "The runtime package name" | ||
| required: true | ||
| node-uri: | ||
| description: "URI of a node to scrape state" | ||
| required: true | ||
| checks: | ||
| description: "Types of checks to run." | ||
| default: "all" | ||
| blocktime: | ||
| description: "Block time in milliseconds to use for try-runtime checks." | ||
| default: "12000" | ||
| extra-args: | ||
| description: "Extra args to pass to the on-runtime-upgrade subcommand." | ||
| default: "" | ||
| cache-on-failure: | ||
| description: "Whether to save the Rust cache even if the job fails" | ||
| default: "false" | ||
|
|
||
| runs: | ||
| using: "composite" | ||
| steps: | ||
| - name: Download try-runtime-cli v0.10.1 | ||
| run: | | ||
| TRY_RUNTIME_BIN="$RUNNER_TEMP/try-runtime" | ||
| curl --fail --silent --show-error --location \ | ||
| "https://github.com/paritytech/try-runtime-cli/releases/download/v0.10.1/try-runtime-x86_64-unknown-linux-musl" \ | ||
| --output "$TRY_RUNTIME_BIN" | ||
| chmod +x "$TRY_RUNTIME_BIN" | ||
| echo "TRY_RUNTIME_BIN=$TRY_RUNTIME_BIN" >> "$GITHUB_ENV" | ||
| shell: bash | ||
|
|
||
| - name: Install Protoc | ||
| uses: arduino/setup-protoc@v1.3.0 | ||
| with: | ||
| version: "3.6.1" | ||
| repo-token: ${{ github.token }} | ||
|
|
||
| - name: Add wasm32-unknown-unknown target | ||
| run: rustup target add wasm32-unknown-unknown | ||
| shell: bash | ||
|
|
||
| - name: Add rust-src component | ||
| run: rustup component add rust-src | ||
| shell: bash | ||
|
|
||
| - name: Fetch cache | ||
| uses: Swatinem/rust-cache@v2.7.0 | ||
| with: | ||
| shared-key: try-runtime | ||
| cache-on-failure: ${{ inputs.cache-on-failure }} | ||
|
|
||
| - name: Build ${{ inputs.runtime-package }} | ||
| run: cargo build --profile production -p "$RUNTIME_PACKAGE" --features try-runtime -q --locked | ||
| shell: bash | ||
| env: | ||
| RUNTIME_PACKAGE: ${{ inputs.runtime-package }} | ||
|
|
||
| - name: Check migrations | ||
| run: | | ||
| RUNTIME_BLOB_NAME="${RUNTIME_PACKAGE//-/_}.compact.compressed.wasm" | ||
| RUNTIME_BLOB_PATH="./target/production/wbuild/$RUNTIME_PACKAGE/$RUNTIME_BLOB_NAME" | ||
| export RUST_LOG=remote-ext=debug,runtime=debug | ||
|
|
||
| command=( | ||
| "$TRY_RUNTIME_BIN" | ||
| --runtime "$RUNTIME_BLOB_PATH" | ||
| on-runtime-upgrade | ||
| "--checks=$CHECKS" | ||
| --blocktime | ||
| "$BLOCKTIME" | ||
| ) | ||
|
|
||
| if [[ -n "$EXTRA_ARGS" ]]; then | ||
| read -r -a extra_args <<< "$EXTRA_ARGS" | ||
| command+=("${extra_args[@]}") | ||
| fi | ||
|
|
||
| command+=(live --uri "$NODE_URI") | ||
|
|
||
| echo "Running command:" | ||
| printf '%q ' "${command[@]}" | ||
| printf '\n' | ||
| "${command[@]}" | ||
| shell: bash | ||
| env: | ||
| RUNTIME_PACKAGE: ${{ inputs.runtime-package }} | ||
| NODE_URI: ${{ inputs.node-uri }} | ||
| CHECKS: ${{ inputs.checks }} | ||
| BLOCKTIME: ${{ inputs.blocktime }} | ||
| EXTRA_ARGS: ${{ inputs.extra-args }} |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| # Subtensor AI Review — Shared Context | ||
|
|
||
| You are reviewing a pull request to **opentensor/subtensor**, the Substrate-based runtime for the Bittensor blockchain (~$4B market cap). Lives and livelihoods depend on the security and correctness of this code. Be thorough, precise, and uncompromising on safety. | ||
| You are reviewing a pull request to **RaoFoundation/subtensor**, the Substrate-based runtime for the Bittensor blockchain (~$4B market cap). Lives and livelihoods depend on the security and correctness of this code. Be thorough, precise, and uncompromising on safety. | ||
|
Contributor
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. [HIGH] AI review trusted files are modified in this PR This PR changes
Contributor
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. [HIGH] AI review trusted files are modified in this PR The operating instructions for this run explicitly require flagging PR-side changes to |
||
|
|
||
| ## Repository topology | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -29,17 +29,13 @@ jobs: | |
| with: | ||
| toolchain: stable | ||
|
|
||
| - name: Utilize Shared Rust Cache | ||
| uses: Swatinem/rust-cache@v2 | ||
| with: | ||
| key: "try-runtime" | ||
|
|
||
| - name: Run Try Runtime Checks | ||
| uses: "paritytech/try-runtime-gha@v0.1.0" | ||
| uses: ./.github/actions/try-runtime | ||
|
Contributor
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. [HIGH] Local action is loaded from the PR checkout on pull_request This
Contributor
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. [HIGH] Local action is loaded from the PR checkout on pull_request This
Contributor
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. [HIGH] Local action is loaded from the PR checkout on pull_request This |
||
| with: | ||
| runtime-package: "node-subtensor-runtime" | ||
|
Contributor
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. [HIGH] Local action is loaded from the PR checkout on pull_request This workflow checks out the PR head and then runs
Contributor
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. [HIGH] Local action is loaded from the PR checkout on pull_request This workflow runs on |
||
| node-uri: "wss://dev.chain.opentensor.ai:443" | ||
| checks: "all" | ||
| blocktime: "12000" | ||
| extra-args: "--disable-spec-version-check --no-weight-warnings" | ||
|
|
||
| check-testnet: | ||
|
|
@@ -60,17 +56,13 @@ jobs: | |
| with: | ||
| toolchain: stable | ||
|
|
||
| - name: Utilize Shared Rust Cache | ||
| uses: Swatinem/rust-cache@v2 | ||
| with: | ||
| key: "try-runtime" | ||
|
|
||
| - name: Run Try Runtime Checks | ||
| uses: "paritytech/try-runtime-gha@v0.1.0" | ||
| uses: ./.github/actions/try-runtime | ||
| with: | ||
| runtime-package: "node-subtensor-runtime" | ||
| node-uri: "wss://archive.dev.opentensor.ai:8443" | ||
| checks: "all" | ||
| blocktime: "12000" | ||
| extra-args: "--disable-spec-version-check --no-weight-warnings" | ||
|
|
||
| check-finney: | ||
|
|
@@ -91,16 +83,12 @@ jobs: | |
| with: | ||
| toolchain: stable | ||
|
|
||
| - name: Utilize Shared Rust Cache | ||
| uses: Swatinem/rust-cache@v2 | ||
| with: | ||
| key: try-runtime | ||
| cache-on-failure: true | ||
|
|
||
| - name: Run Try Runtime Checks | ||
| uses: "paritytech/try-runtime-gha@v0.1.0" | ||
| uses: ./.github/actions/try-runtime | ||
| with: | ||
| runtime-package: "node-subtensor-runtime" | ||
| node-uri: "wss://archive.dev.opentensor.ai:443" | ||
| checks: "all" | ||
| blocktime: "12000" | ||
| extra-args: "--disable-spec-version-check --no-weight-warnings" | ||
| cache-on-failure: "true" | ||
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.
[HIGH] AI review trusted files are modified in this PR
This PR changes
.github/ai-review/*files relative to the trusted base (README.md,auditor.md,common.md,post_review.py,prefetch.sh, andskeptic.md). The operating instructions for this review explicitly require flagging PR-side changes to.github/ai-review/*as HIGH or CRITICAL. Even if these lines are intended as an org rename, changes to review prompts and posting/prefetch scripts should be separated and reviewed against the trusted copies before a deployment fast-forward consumes them.