ci: walrus 1.29 version bump (#2241) #7408
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Code | |
| on: | |
| # Run workflow on every PR. | |
| pull_request: | |
| # Run workflow on the main branch after every merge. | |
| # This is important to fill the GitHub Actions cache in a way that PRs can see it. | |
| push: | |
| branches: | |
| - main | |
| concurrency: | |
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | |
| cancel-in-progress: true | |
| env: | |
| CARGO_TERM_COLOR: always | |
| CLICOLOR_FORCE: 1 | |
| # Incremental compilation is useful as part of an edit-build-test-edit cycle, as it lets the | |
| # compiler avoid recompiling code that hasn't changed. The setting does not improve the current | |
| # compilation but instead saves additional information to speed up future compilations (see | |
| # https://doc.rust-lang.org/cargo/reference/profiles.html#incremental). Thus, this is only useful | |
| # in CI if the result is cached, which we only do on the `main` branch. | |
| CARGO_INCREMENTAL: ${{ github.ref == 'refs/heads/main' && '1' || '0' }} | |
| # Allow more retries for network requests in cargo (downloading crates) and | |
| # rustup (installing toolchains). This should help to reduce flaky CI failures | |
| # from transient network timeouts or other issues. | |
| CARGO_NET_RETRY: 10 | |
| RUSTUP_MAX_RETRIES: 10 | |
| # Don't emit giant backtraces in the CI logs. | |
| RUST_BACKTRACE: short | |
| RUSTDOCFLAGS: -D warnings | |
| SUI_TAG: testnet-v1.50.1 | |
| jobs: | |
| diff: | |
| runs-on: ubuntu-latest | |
| permissions: | |
| contents: read | |
| pull-requests: read | |
| outputs: | |
| isRust: ${{ steps.diff.outputs.isRust }} | |
| isMove: ${{ steps.diff.outputs.isMove }} | |
| isDenyConfig: ${{ steps.diff.outputs.isDenyConfig }} | |
| isTestConfig: ${{ steps.diff.outputs.isTestConfig }} | |
| isOpenapi: ${{ steps.diff.outputs.isOpenapi }} | |
| isRelevantForRustTests: ${{ steps.diff.outputs.isRust == 'true' || steps.diff.outputs.isMove == 'true' || steps.diff.outputs.isTestConfig | |
| == 'true' }} | |
| isTestnetContracts: ${{ steps.diff.outputs.isTestnetContracts }} | |
| isMainnetContracts: ${{ steps.diff.outputs.isMainnetContracts }} | |
| isExampleConfig: ${{ steps.diff.outputs.isExampleConfig }} | |
| isCLI: ${{ steps.diff.outputs.isCLI }} | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Detect Changes | |
| uses: "./.github/actions/diffs" | |
| id: diff | |
| dependencies: | |
| name: Check dependencies | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isRust == 'true' || needs.diff.outputs.isDenyConfig == 'true' }} | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - uses: taiki-e/install-action@970d55e3ce02a46d60ffae7b4fab3dedace6e769 # [email protected] | |
| with: | |
| tool: [email protected] | |
| - name: Check dependencies | |
| run: cargo deny check bans licenses sources | |
| lint: | |
| name: Lint Rust code | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isRust == 'true' }} | |
| runs-on: ubuntu-ghcloud | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Check formatting with rustfmt | |
| run: > | |
| cargo fmt --all -- --check | |
| --config group_imports=StdExternalCrate,imports_granularity=Crate,imports_layout=HorizontalVertical | |
| - name: Lint using clippy (w/o tests) | |
| run: cargo clippy --workspace --all-features --no-deps -- -D warnings | |
| - name: Lint using clippy (w/ tests) | |
| run: cargo clippy --workspace --all-features --tests --no-deps -- -D warnings | |
| - name: Check documentation | |
| run: cargo doc --workspace --no-deps | |
| build: | |
| name: Build Rust code | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isRust == 'true' }} | |
| env: | |
| OPENSSL_STATIC: "1" | |
| OPENSSL_NO_VENDOR: "0" | |
| runs-on: ubuntu-ghcloud | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Install dependencies | |
| run: | | |
| sudo apt-get update | |
| sudo apt-get install -y libssl-dev pkg-config zlib1g-dev libpq-dev build-essential cmake | |
| - name: Build Rust code | |
| run: cargo build --workspace --features "walrus-service/backup" --verbose | |
| test: | |
| name: Test Rust code | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isRelevantForRustTests == 'true' }} | |
| runs-on: ubuntu-ghcloud | |
| env: | |
| RUST_LOG: walrus=info,error,sui_=off,consensus_core=off | |
| OPENSSL_STATIC: "1" | |
| OPENSSL_NO_VENDOR: "0" | |
| steps: | |
| - uses: taiki-e/install-action@970d55e3ce02a46d60ffae7b4fab3dedace6e769 # [email protected] | |
| with: | |
| tool: cargo-nextest | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - uses: Swatinem/rust-cache@9d47c6ad4b02e050fd481d890b2ea34778fd09d6 # [email protected] | |
| with: | |
| save-if: ${{ github.ref == 'refs/heads/main' && 'true' || 'false' }} | |
| - name: Install dependencies | |
| run: | | |
| sudo apt-get update | |
| sudo apt-get install -y libssl-dev pkg-config zlib1g-dev libpq-dev build-essential cmake | |
| - name: Run tests | |
| run: cargo nextest run --workspace --features "walrus-service/backup" --profile ci --run-ignored all | |
| - name: Run doctests | |
| run: cargo test --doc | |
| test-coverage: | |
| name: Run all Rust tests and report coverage | |
| if: contains(github.event.pull_request.labels.*.name, 'report_coverage') | |
| runs-on: ubuntu-ghcloud | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| env: | |
| RUSTC_BOOTSTRAP: 1 | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - uses: ./.github/actions/run_tarpaulin | |
| - name: Add coverage PR comment | |
| uses: marocchino/sticky-pull-request-comment@67d0dec7b07ed060a405f9b2a64b8ab319fdd7db # pin@v2 | |
| if: ${{ github.event_name == 'pull_request' && github.actor != 'dependabot[bot]' }} | |
| with: | |
| path: code-coverage-results.md | |
| simtests: | |
| name: Run all simtests | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isRelevantForRustTests == 'true' }} | |
| runs-on: ubuntu-ghcloud | |
| env: | |
| MSIM_TEST_NUM: 5 | |
| # Turn off Sui logging since it can be very distractive in CI | |
| RUST_LOG: simtest=info,error,sui_=off,consensus_core=off | |
| steps: | |
| - uses: taiki-e/install-action@970d55e3ce02a46d60ffae7b4fab3dedace6e769 # [email protected] | |
| with: | |
| tool: cargo-nextest | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - uses: Swatinem/rust-cache@9d47c6ad4b02e050fd481d890b2ea34778fd09d6 # [email protected] | |
| with: | |
| save-if: ${{ github.ref == 'refs/heads/main' && 'true' || 'false' }} | |
| - run: ./scripts/simtest/install.sh | |
| - name: Run tests | |
| run: cargo simtest simtest --profile simtest | |
| simtests-build: | |
| name: Build all simtests | |
| needs: diff | |
| # if: ${{ needs.diff.outputs.isRelevantForRustTests == 'true' }} | |
| if: false | |
| runs-on: ubuntu-ghcloud | |
| steps: | |
| - uses: taiki-e/install-action@970d55e3ce02a46d60ffae7b4fab3dedace6e769 # [email protected] | |
| with: | |
| tool: cargo-nextest | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - run: ./scripts/simtest/install.sh | |
| - name: Build simtests | |
| run: cargo simtest build | |
| test-move: | |
| name: Test Move code | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isMove == 'true' }} | |
| runs-on: ubuntu-ghcloud | |
| env: | |
| SUI_BIN: "/home/runner/.cargo/bin/sui" | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Restore cached sui binary | |
| id: cache-sui-restore | |
| uses: actions/cache/restore@5a3ec84eff668545956fd18022155c47e93e2684 # pin@v4 | |
| with: | |
| path: ${{ env.SUI_BIN }} | |
| key: ${{ runner.os }}-${{ runner.arch }}-${{ env.SUI_TAG }} | |
| - name: Install correct Rust toolchain | |
| run: rustup update && rustup toolchain install | |
| if: steps.cache-sui-restore.outputs.cache-hit != 'true' | |
| - name: Install sui | |
| if: steps.cache-sui-restore.outputs.cache-hit != 'true' | |
| run: cargo install --locked --git https://github.com/MystenLabs/sui.git --tag $SUI_TAG --debug --features tracing | |
| sui | |
| - name: Run Move tests and check coverage | |
| run: bash ./scripts/move_coverage.sh | |
| - name: Cache sui binary | |
| if: ${{ github.ref == 'refs/heads/main' && steps.cache-sui-restore.outputs.cache-hit != 'true' }} | |
| uses: actions/cache/save@5a3ec84eff668545956fd18022155c47e93e2684 # pin@v4 | |
| with: | |
| path: ${{ env.SUI_BIN }} | |
| key: ${{ steps.cache-sui-restore.outputs.cache-primary-key }} | |
| test-contracts-warning: | |
| name: Check if the PR touches the testnet-contracts directory | |
| runs-on: ubuntu-latest | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isTestnetContracts == 'true' && github.event_name == 'pull_request' && github.actor != 'dependabot[bot]'}} | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Post a warning if the PR touches the testnet-contracts directory | |
| uses: marocchino/sticky-pull-request-comment@67d0dec7b07ed060a405f9b2a64b8ab319fdd7db # pin@v2 | |
| with: | |
| message: > | |
| **Warning:** This PR touches the `testnet-contracts` directory. This should only be | |
| done if the Testnet contracts are updated. | |
| cc @karlwuest @mlegner | |
| mainnet-contracts-warning: | |
| name: Check if the PR touches the mainnet-contracts directory | |
| runs-on: ubuntu-latest | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isMainnetContracts == 'true' && github.event_name == 'pull_request' && github.actor != 'dependabot[bot]'}} | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Post a warning if the PR touches the mainnet-contracts directory | |
| uses: marocchino/sticky-pull-request-comment@67d0dec7b07ed060a405f9b2a64b8ab319fdd7db # pin@v2 | |
| with: | |
| message: > | |
| **Warning:** This PR touches the `mainnet-contracts` directory. This should only be | |
| done if the Mainnet contracts are updated. | |
| cc @karlwuest @mlegner | |
| example-config-warning: | |
| name: Check if the PR touches any of the example config files | |
| runs-on: ubuntu-latest | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isExampleConfig == 'true' && github.event_name == 'pull_request' && github.actor != 'dependabot[bot]'}} | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Post a warning if the PR touches an example config file | |
| uses: marocchino/sticky-pull-request-comment@67d0dec7b07ed060a405f9b2a64b8ab319fdd7db # pin@v2 | |
| with: | |
| message: | | |
| **Warning:** This PR modifies one of the example config files. Please consider the | |
| following: | |
| - Make sure the changes are backwards compatible with the current configuration. | |
| - Make sure any added parameters follow the conventions of the existing parameters; in | |
| particular, durations should take seconds or milliseconds using the naming convention | |
| `_secs` or `_millis`, respectively. | |
| - If there are added optional parameter sections, it should be possible to specify them | |
| partially. A useful pattern there is to implement `Default` for the struct and derive | |
| `#[serde(default)]` on it, see `BlobRecoveryConfig` as an example. | |
| - You may need to update the documentation to reflect the changes. | |
| cli-warning: | |
| name: Check if the PR touches any of the CLI files | |
| runs-on: ubuntu-latest | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isCLI == 'true' && github.event_name == 'pull_request' && github.actor != 'dependabot[bot]'}} | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Post a warning if the PR modifies the Walrus CLI | |
| uses: marocchino/sticky-pull-request-comment@67d0dec7b07ed060a405f9b2a64b8ab319fdd7db # pin@v2 | |
| with: | |
| message: | | |
| **Warning:** This PR modifies the Walrus CLI. Please consider the following: | |
| - Make sure the changes are backwards compatible. Consider deprecating options before | |
| removing them. | |
| - Generally only use `long` CLI options, not `short` ones to avoid conflicts in the | |
| future. | |
| - If you added new options or features, or modified the behavior, please document the | |
| changes in the release notes of the PR and update the documentation in the `docs/book` | |
| directory. | |
| openapi-warning: | |
| name: Check if the PR touches any of the OpenAPI files | |
| runs-on: ubuntu-latest | |
| needs: diff | |
| if: ${{ needs.diff.outputs.isOpenapi == 'true' && github.event_name == 'pull_request' && github.actor != 'dependabot[bot]'}} | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| steps: | |
| - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # pin@v4 | |
| - name: Post a warning if the PR touches an OpenAPI file | |
| uses: marocchino/sticky-pull-request-comment@67d0dec7b07ed060a405f9b2a64b8ab319fdd7db # pin@v2 | |
| with: | |
| message: | | |
| **Warning:** This PR modifies one of the OpenAPI files. Please consider the | |
| following: | |
| - Make sure the API changes are backwards compatible. | |
| - Make sure to follow existing conventions for any added parameters, requests, and | |
| responses. | |
| - Make sure the generated HTML files do not contain errors. | |
| - This probably requires release notes and maybe changes to the documentation. | |
| cc @jpcsmith @mlegner | |
| check-all: | |
| name: Check if all code checks succeeded | |
| if: always() | |
| needs: | |
| - diff | |
| - dependencies | |
| - lint | |
| - build | |
| - test | |
| - simtests-build | |
| - simtests | |
| - test-move | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Decide whether all needed jobs succeeded | |
| uses: re-actors/alls-green@05ac9388f0aebcb5727afa17fcccfecd6f8ec5fe # [email protected] | |
| with: | |
| allowed-skips: ${{ toJSON(needs) }} | |
| jobs: ${{ toJSON(needs) }} |