Skip to content

Commit a8cb0ce

Browse files
Copilotjsturtevant
andcommitted
Move fuzzing failure notification from reusable workflow to scheduled workflow
- Removed notification step and issues permission from dep_fuzzing.yml (reusable workflow) - Added separate notify-failure job to Fuzzing.yml (scheduled workflow) - This ensures issues are only created for scheduled cron job failures, not PR runs Co-authored-by: jsturtevant <[email protected]>
1 parent b1ffb6a commit a8cb0ce

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

.github/workflows/Fuzzing.yml

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,17 @@ jobs:
1616
with:
1717
targets: '["fuzz_host_print", "fuzz_guest_call", "fuzz_host_call"]' # Pass as a JSON array
1818
max_total_time: 18000 # 5 hours in seconds
19-
secrets: inherit
19+
secrets: inherit
20+
21+
notify-failure:
22+
runs-on: ubuntu-latest
23+
needs: fuzzing
24+
if: failure() && needs.fuzzing.result == 'failure'
25+
steps:
26+
- name: Checkout code
27+
uses: actions/checkout@v5
28+
29+
- name: Notify Fuzzing Failure
30+
run: ./dev/notify-fuzzing-failure.sh "fuzz_host_print,fuzz_guest_call,fuzz_host_call"
31+
env:
32+
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/dep_fuzzing.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ on:
2020
permissions:
2121
id-token: write
2222
contents: read
23-
issues: write
2423

2524
jobs:
2625
fuzz:
@@ -61,9 +60,3 @@ jobs:
6160
with:
6261
name: fuzz-crash-artifacts
6362
path: fuzz/artifacts/
64-
65-
- name: Notify Fuzzing Failure
66-
if: failure() # Only run this step if the fuzzing job failed
67-
run: ./dev/notify-fuzzing-failure.sh "${{ matrix.target }}"
68-
env:
69-
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

0 commit comments

Comments
 (0)