Skip to content

Commit 642a09b

Browse files
authored
Fix fuzzing workflow to build guest artifacts before fuzzing (#1153)
When the CI jobs were refactored to use reusable workflows, the dep_fuzzing.yml was changed to download guest artifacts instead of building them directly. However, the standalone Fuzzing.yml scheduled workflow was not updated to call dep_build_guests.yml first, causing all weekly fuzzing runs to fail with 'Artifact not found for name: rust-guests-release'. This adds the missing build-guests job as a dependency of the fuzzing job, matching the pattern used in ValidatePullRequest.yml. Signed-off-by: James Sturtevant <[email protected]>
1 parent b879cd7 commit 642a09b

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

.github/workflows/Fuzzing.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,15 @@ permissions:
99
contents: read
1010

1111
jobs:
12+
# Build guests first - fuzzing needs the release guest artifacts
13+
build-guests:
14+
uses: ./.github/workflows/dep_build_guests.yml
15+
secrets: inherit
16+
with:
17+
config: release
18+
1219
fuzzing:
20+
needs: build-guests
1321
uses: ./.github/workflows/dep_fuzzing.yml
1422
with:
1523
targets: '["fuzz_host_print", "fuzz_guest_call", "fuzz_host_call", "fuzz_guest_estimate_trace_event", "fuzz_guest_trace"]' # Pass as a JSON array
@@ -18,8 +26,8 @@ jobs:
1826

1927
notify-failure:
2028
runs-on: ubuntu-latest
21-
needs: fuzzing
22-
if: always() && needs.fuzzing.result == 'failure'
29+
needs: [build-guests, fuzzing]
30+
if: always() && (needs.build-guests.result == 'failure' || needs.fuzzing.result == 'failure')
2331
permissions:
2432
issues: write
2533
steps:

0 commit comments

Comments
 (0)