cifuzz: upload artifacts only when the "run fuzzers" step fails (#3733)

Closes #3711
google · Apr 30, 2020 · 8ba4f3a · 8ba4f3a
1 parent aec1054
commit 8ba4f3a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/docs/getting-started/continuous_integration.md b/docs/getting-started/continuous_integration.md
@@ -59,6 +59,7 @@ jobs:
    runs-on: ubuntu-latest
    steps:
    - name: Build Fuzzers
+     id: build
      uses: google/oss-fuzz/infra/cifuzz/actions/build_fuzzers@master
      with:
        oss-fuzz-project-name: 'example'
@@ -71,7 +72,7 @@ jobs:
        dry-run: false
    - name: Upload Crash
      uses: actions/upload-artifact@v1
-     if: failure()
+     if: failure() && steps.build.outcome == 'success'
      with:
        name: artifacts
        path: ./out/artifacts

diff --git a/infra/cifuzz/example_main.yml b/infra/cifuzz/example_main.yml
@@ -5,6 +5,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Build Fuzzers
+      id: build
       uses: google/oss-fuzz/infra/cifuzz/actions/build_fuzzers@master
       with:
         oss-fuzz-project-name: 'example'
@@ -17,7 +18,7 @@ jobs:
         dry-run: false
     - name: Upload Crash
       uses: actions/upload-artifact@v1
-      if: failure()
+      if: failure() && steps.build.outcome == 'success'
       with:
         name: artifacts
         path: ./out/artifacts