ci: Add utility to generate the feature "powerset" list

The list isn't technically a powerset because we limit the "depth". We're doing this instead of using `cargo hack` because `cargo hack` does not allow splitting the tasks across different workflow runners via e.g. a matrix. This PR also updates the `feature_powerset.yml` workflow to use our custom powerset script.
roadster-rs · Oct 19, 2024 · a7ce914 · a7ce914
1 parent 2d8d31f
commit a7ce914
Show file tree

Hide file tree

Showing 15 changed files with 541 additions and 104 deletions.
diff --git a/.github/workflows/feature_powerset.yml b/.github/workflows/feature_powerset.yml
@@ -43,65 +43,97 @@ jobs:
         if: ${{ github.event_name == 'pull_request' && github.event.label.name == 'powerset_check' }}
         run: echo "should_run=true" >> "$GITHUB_OUTPUT"
 
-  powerset_test:
-    name: Powerset Tests
+  generate_powerset:
     needs: check_trigger
     if: ${{ needs.check_trigger.outputs.should_run1 == 'true' || needs.check_trigger.outputs.should_run2 == 'true' || github.event_name == 'workflow_dispatch' }}
     runs-on: ubuntu-latest
+    outputs:
+      data: ${{ steps.build_data.outputs.data }}
     steps:
       - uses: actions/checkout@v4
-      - uses: rui314/setup-mold@v1
-      - uses: Swatinem/rust-cache@v2
-      - uses: taiki-e/install-action@cargo-hack
-      - uses: taiki-e/install-action@nextest
-      # protoc is needed to build examples that have grpc enabled
-      - uses: taiki-e/install-action@protoc
-      - name: Test
-        run: cargo hack nextest run --no-fail-fast --feature-powerset --depth 3 --skip default --group-features jwt-ietf,jwt --group-features jwt-openid,jwt --group-features open-api,http --group-features email-smtp,email --group-features email-sendgrid,email --clean-per-run --log-group github-actions --exclude-no-default-features --exclude-all-features
-      - name: Check disk usage
-        run: df -h
+      - name: Random seed
+        id: random_seed
+        run: |
+          if [ -n "${{ github.event.number }}" ]; then
+              echo "seed=-r ${{ github.event.number }}" >> "$GITHUB_OUTPUT"
+          else
+              echo "seed=" >> "$GITHUB_OUTPUT"
+          fi
+      - name: Build powerset data
+        id: build_data
+        run: |
+          cd private/powerset_matrix
+          echo "data=$(cargo run -- -s 50 -f json -c 50 ${{ steps.random_seed.outputs.seed }})" >> "$GITHUB_OUTPUT"
 
-  powerset_doc_test:
-    name: Powerset Doc tests
-    needs: check_trigger
-    if: ${{ needs.check_trigger.outputs.should_run1 == 'true' || needs.check_trigger.outputs.should_run2 == 'true' || github.event_name == 'workflow_dispatch' }}
+  powerset_test:
+    name: Powerset Tests
+    needs: generate_powerset
     runs-on: ubuntu-latest
+    strategy:
+      max-parallel: 10
+      matrix:
+        index: ${{ fromJson(needs.generate_powerset.outputs.data).indexes }}
+    env:
+      features: ${{ join(fromJson(needs.generate_powerset.outputs.data).powersets[ matrix.index ], ' ') }}
     steps:
       - uses: actions/checkout@v4
       - uses: rui314/setup-mold@v1
-      - uses: Swatinem/rust-cache@v2
-      - uses: taiki-e/install-action@cargo-hack
-      # protoc is needed to build examples that have grpc enabled
-      - uses: taiki-e/install-action@protoc
-      - name: Doc test
-        run: cargo hack test --doc --no-fail-fast --feature-powerset --depth 3 --skip default --group-features jwt-ietf,jwt --group-features jwt-openid,jwt --group-features open-api,http --group-features email-smtp,email --group-features email-sendgrid,email --clean-per-run --log-group github-actions --exclude-no-default-features --exclude-all-features
+      - uses: taiki-e/install-action@nextest
+      - name: Test
+        run: |
+          features=($features)
+          for feature_list in "${features[@]}"; do
+            echo "::group::cargo nextest run --no-fail-fast --features $feature_list"
+            cargo nextest run --no-fail-fast --features "$feature_list"
+            echo "::endgroup::"
+            echo "::group::cargo test --doc --no-fail-fast --features $feature_list"
+            cargo test --doc --no-fail-fast --features "$feature_list"
+            cargo clean -p roadster
+            echo "::endgroup::"
+          done
 
   powerset_check:
     name: Powerset Check
-    needs: check_trigger
-    if: ${{ needs.check_trigger.outputs.should_run1 == 'true' || needs.check_trigger.outputs.should_run2 == 'true' || github.event_name == 'workflow_dispatch' }}
+    needs: generate_powerset
     runs-on: ubuntu-latest
+    strategy:
+      max-parallel: 10
+      matrix:
+        index: ${{ fromJson(needs.generate_powerset.outputs.data).indexes }}
+    env:
+      features: ${{ join(fromJson(needs.generate_powerset.outputs.data).powersets[ matrix.index ], ' ') }}
     steps:
       - uses: actions/checkout@v4
       - uses: rui314/setup-mold@v1
-      - uses: Swatinem/rust-cache@v2
-      - uses: taiki-e/install-action@cargo-hack
-      # protoc is needed to build examples that have grpc enabled
-      - uses: taiki-e/install-action@protoc
-      - name: Check
-        run: cargo hack check --feature-powerset --depth 3 --no-dev-deps --skip default --group-features jwt-ietf,jwt --group-features jwt-openid,jwt --group-features open-api,http --group-features email-smtp,email --group-features email-sendgrid,email --clean-per-run --log-group github-actions --exclude-no-default-features --exclude-all-features
+      - name: Test
+        run: |
+          features=($features)
+          for feature_list in "${features[@]}"; do
+            echo "::group::cargo check --features $feature_list"
+            cargo check --features "$feature_list"
+            cargo clean -p roadster
+            echo "::endgroup::"
+          done
 
   powerset_clippy:
     name: Powerset Clippy
-    needs: check_trigger
-    if: ${{ needs.check_trigger.outputs.should_run1 == 'true' || needs.check_trigger.outputs.should_run2 == 'true' || github.event_name == 'workflow_dispatch' }}
+    needs: generate_powerset
     runs-on: ubuntu-latest
+    strategy:
+      max-parallel: 10
+      matrix:
+        index: ${{ fromJson(needs.generate_powerset.outputs.data).indexes }}
+    env:
+      features: ${{ join(fromJson(needs.generate_powerset.outputs.data).powersets[ matrix.index ], ' ') }}
     steps:
       - uses: actions/checkout@v4
       - uses: rui314/setup-mold@v1
-      - uses: Swatinem/rust-cache@v2
-      - uses: taiki-e/install-action@cargo-hack
-      # protoc is needed to build examples that have grpc enabled
-      - uses: taiki-e/install-action@protoc
-      - name: Clippy
-        run: cargo hack clippy --all-targets --feature-powerset --depth 3 --skip default --group-features jwt-ietf,jwt --group-features jwt-openid,jwt --group-features open-api,http --group-features email-smtp,email --group-features email-sendgrid,email --clean-per-run --log-group github-actions --exclude-no-default-features --exclude-all-features -- -D warnings
+      - name: Test
+        run: |
+          features=($features)
+          for feature_list in "${features[@]}"; do
+            echo "::group::cargo clippy --features $feature_list"
+            cargo clippy --features "$feature_list"
+            cargo clean -p roadster
+            echo "::endgroup::"
+          done
diff --git a/Cargo.toml b/Cargo.toml
@@ -98,12 +98,12 @@ rstest = { workspace = true, optional = true }
 # Others
 anyhow = { workspace = true }
 serde = { workspace = true }
-serde_derive = "1.0.185"
+serde_derive = { workspace = true }
+serde_json = { workspace = true }
 serde_with = { version = "3.0.0", features = ["macros", "chrono_0_4"] }
-strum = "0.26.0"
-strum_macros = "0.26.0"
-itertools = "0.13.0"
-serde_json = "1.0.96"
+strum = { workspace = true }
+strum_macros = { workspace = true }
+itertools = { workspace = true }
 toml = "0.8.0"
 url = { version = "2.5.0", features = ["serde"] }
 uuid = { version = "1.1.2", features = ["v4", "serde"] }
@@ -113,10 +113,10 @@ chrono = { version = "0.4.34", features = ["serde"] }
 byte-unit = { version = "5.0.0", features = ["serde"] }
 convert_case = "0.6.0"
 const_format = "0.2.30"
-typed-builder = "0.20.0"
+typed-builder = { workspace = true }
 num-traits = "0.2.14"
 validator = { version = "0.18.0", features = ["derive"] }
-thiserror = "1.0.38"
+thiserror = { workspace = true }
 # Add latest version of `time` to resolve a build error on nightly
 # https://github.com/time-rs/time/issues/681
 time = "0.3.36"
@@ -131,7 +131,7 @@ mockall_double = "0.3.1"
 rstest = { workspace = true }
 
 [workspace]
-members = [".", "examples/*", "book/examples/*"]
+members = [".", "examples/*", "book/examples/*", "private/*"]
 
 [workspace.dependencies]
 # Tracing
@@ -179,10 +179,19 @@ tokio = { version = "1.34.0", features = ["full"] }
 tokio-util = { version = "0.7.10" }
 anyhow = "1.0.86"
 serde = { version = "1.0.185", features = ["derive"] }
+serde_derive = "1.0.185"
+serde_json = "1.0.96"
+strum = "0.26.0"
+strum_macros = "0.26.0"
 cfg-if = "1.0.0"
 vergen = { version = "9.0.0" }
 vergen-gitcl = { version = "1.0.0" }
 reqwest = "0.12.8"
+itertools = "0.13.0"
+cargo-manifest = "0.15.0"
+typed-builder = "0.20.0"
+rand = "0.8.5"
+thiserror = "1.0.49"
 
 [package.metadata.docs.rs]
 # Have docs.rs pass `--all-features` to ensure all features have their documentation built.