Skip to content

Commit

Permalink
[improve][ci][branch-3.0] Upgrade actions in pulsar-ci and pulsar-ci-…
Browse files Browse the repository at this point in the history
…flaky, port owasp cache change

(cherry picked from commit db40c8f)
  • Loading branch information
lhotari authored and nikhil-ctds committed May 15, 2024
1 parent 462f806 commit ce93e19
Show file tree
Hide file tree
Showing 5 changed files with 90 additions and 68 deletions.
8 changes: 4 additions & 4 deletions .github/actions/upload-coverage/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ runs:
- name: "Upload to Codecov (attempt #1)"
id: codecov-upload-1
if: steps.repo-check.outputs.passed == 'true'
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
continue-on-error: true
with:
flags: ${{ inputs.flags }}
Expand All @@ -64,7 +64,7 @@ runs:
- name: "Upload to Codecov (attempt #2)"
id: codecov-upload-2
if: steps.codecov-upload-1.outcome == 'failure'
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
continue-on-error: true
with:
flags: ${{ inputs.flags }}
Expand All @@ -77,7 +77,7 @@ runs:
- name: "Upload to Codecov (attempt #3)"
id: codecov-upload-3
if: steps.codecov-upload-2.outcome == 'failure'
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
# fail on last attempt
continue-on-error: false
with:
Expand All @@ -97,4 +97,4 @@ runs:
[Code coverage report](https://app.codecov.io/github/$GITHUB_REPOSITORY/commit/${head_sha}/tree)
EOF
EOF
4 changes: 3 additions & 1 deletion .github/changes-filter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,12 @@ docs:
- 'deployment/**'
- 'wiki/**'
- 'pip/**'
java_non_tests:
- '**/src/main/java/**/*.java'
tests:
- added|modified: '**/src/test/java/**/*.java'
need_owasp:
- 'pom.xml'
- '**/pom.xml'
- 'src/owasp-dependency-check-false-positives.xml'
- 'src/owasp-dependency-check-suppressions.xml'
- 'src/owasp-dependency-check-suppressions.xml'
6 changes: 3 additions & 3 deletions .github/workflows/ci-go-functions.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
docs_only: ${{ steps.check_changes.outputs.docs_only }}
steps:
- name: checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Detect changed files
id: changes
Expand Down Expand Up @@ -80,13 +80,13 @@ jobs:

steps:
- name: Check out code into the Go module directory
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm

- name: Set up Go
uses: actions/setup-go@v2
uses: actions/setup-go@v5
with:
go-version: ${{ matrix.go-version }}
id: go
Expand Down
14 changes: 7 additions & 7 deletions .github/workflows/pulsar-ci-flaky.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ jobs:
collect_coverage: ${{ steps.check_coverage.outputs.collect_coverage }}
steps:
- name: checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Detect changed files
id: changes
Expand Down Expand Up @@ -101,7 +101,7 @@ jobs:
if: ${{ needs.preconditions.outputs.docs_only != 'true' }}
steps:
- name: checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm
Expand All @@ -115,7 +115,7 @@ jobs:
limit-access-to-actor: true

- name: Cache local Maven repository
uses: actions/cache@v3
uses: actions/cache@v4
timeout-minutes: 5
with:
path: |
Expand All @@ -126,7 +126,7 @@ jobs:
${{ runner.os }}-m2-dependencies-core-modules-
- name: Set up JDK 17
uses: actions/setup-java@v3
uses: actions/setup-java@v4
with:
distribution: 'temurin'
java-version: 17
Expand Down Expand Up @@ -156,7 +156,7 @@ jobs:
- name: Upload Jacoco report files to build artifacts
if: ${{ needs.preconditions.outputs.collect_coverage == 'true' }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: Jacoco-coverage-report-flaky
path: target/jacoco_test_coverage_report_flaky.zip
Expand All @@ -176,15 +176,15 @@ jobs:
annotate_only: 'true'

- name: Upload Surefire reports
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
with:
name: Unit-BROKER_FLAKY-surefire-reports
path: surefire-reports
retention-days: 7

- name: Upload possible heap dump
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ always() }}
with:
name: Unit-BROKER_FLAKY-heapdump
Expand Down
Loading

0 comments on commit ce93e19

Please sign in to comment.