Skip to content

Commit

Permalink
[fix][ci] Owasp check: do not use 'install verify' to avoid build fai…
Browse files Browse the repository at this point in the history
…lures (apache#15119)
  • Loading branch information
nicoloboschi authored Apr 12, 2022
1 parent af7d276 commit 4bee192
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/ci-owasp-dep-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ jobs:
- '**/pom.xml'
- 'src/owasp-dependency-check-false-positives.xml'
- 'src/owasp-dependency-check-suppressions.xml'
- '.github/workflows/ci-owasp-dep-check.yaml'
- name: Cache local Maven repository
if: ${{ steps.changes.outputs.poms == 'true' }}
Expand Down Expand Up @@ -87,9 +88,9 @@ jobs:
df -h
# Projects dependent on flume, hdfs, hbase, and presto currently excluded from the scan.
- name: run "clean install verify" to trigger dependency check
- name: run "clean verify" to trigger dependency check
if: ${{ steps.changes.outputs.poms == 'true' }}
run: mvn -q -B -ntp clean install verify -PskipDocker,owasp-dependency-check -DskipTests -pl '!pulsar-sql,!distribution/io,!distribution/offloaders,!tiered-storage/file-system,!pulsar-io/flume,!pulsar-io/hbase,!pulsar-io/hdfs2,!pulsar-io/hdfs3,!pulsar-io/docs'
run: mvn -q -B -ntp clean verify -PskipDocker,owasp-dependency-check -DskipTests -pl '!pulsar-sql,!distribution/io,!distribution/offloaders,!tiered-storage/file-system,!pulsar-io/flume,!pulsar-io/hbase,!pulsar-io/hdfs2,!pulsar-io/hdfs3,!pulsar-io/docs'

- name: Upload report
uses: actions/upload-artifact@v2
Expand Down

0 comments on commit 4bee192

Please sign in to comment.