old py, java, spotbugs plugins fix (#29008)

.github/workflows/beam_PerformanceTests_WordCountIT_PythonVersions.yml

@@ -100,4 +100,10 @@ jobs:
  arguments: |
  --info \
  -Ptest=apache_beam/examples/wordcount_it_test.py::WordCountIT::test_wordcount_it \
- "-Ptest-pipeline-options=${{ env.beam_PerformanceTests_WordCountIT_PythonVersions_test_arguments_1 }}"
+ "-Ptest-pipeline-options=${{ env.beam_PerformanceTests_WordCountIT_PythonVersions_test_arguments_1 }}"
+ - name: Archive code coverage results
+ uses: actions/upload-artifact@v3
+ if: always()
+ with:
+ name: python-code-coverage-report
+ path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Java_Examples_Dataflow_V2.yml

@@ -84,4 +84,6 @@ jobs:
  uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
  files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_Java_Examples_Dataflow_V2_Java.yml

@@ -98,4 +98,6 @@ jobs:
  uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
  files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_Java_PVR_Flink_Streaming.yml

@@ -73,3 +73,16 @@ jobs:
  uses: ./.github/actions/gradle-command-self-hosted-action
  with:
  gradle-command: runners:flink:1.15:job-server:validatesPortableRunnerStreaming
+ - name: Archive JUnit Test Results
+ uses: actions/upload-artifact@v3
+ if: failure()
+ with:
+ name: JUnit Test Results
+ path: "**/build/reports/tests/"
+ - name: Publish JUnit Test Results
+ uses: EnricoMi/publish-unit-test-result-action@v2
+ if: always()
+ with:
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_Java_PVR_Samza.yml

@@ -85,4 +85,6 @@ jobs:
  uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
  files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_Java_PVR_Spark3_Streaming.yml

@@ -83,4 +83,6 @@ jobs:
  uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
- files: '**/build/test-results/**/*.xml'
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_Java_PVR_Spark_Batch.yml

@@ -96,4 +96,4 @@ jobs:
  uses: actions/upload-artifact@v3
  with:
  name: SpotBugs Results
- path: "**/build/reports/spotbugs/*.html"
+ path: "**/build/reports/spotbugs/*.html"

.github/workflows/beam_PostCommit_PortableJar_Flink.yml

@@ -81,6 +81,7 @@ jobs:
  -PpythonVersion=3.8 \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_PortableJar_Spark.yml

@@ -81,6 +81,7 @@ jobs:
  -PpythonVersion=3.8 \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python.yml

@@ -96,6 +96,7 @@ jobs:
  CLOUDSDK_CONFIG: ${{ env.KUBELET_GCLOUD_CONFIG_PATH}}
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Arm.yml

@@ -112,6 +112,7 @@ jobs:
  USER: github-actions
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Examples_Dataflow.yml

@@ -81,6 +81,7 @@ jobs:
  -PpythonVersion=3.11 \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Examples_Direct.yml

@@ -88,6 +88,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Examples_Flink.yml

@@ -88,6 +88,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Examples_Spark.yml

@@ -88,6 +88,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_MongoDBIO_IT.yml

@@ -80,6 +80,7 @@ jobs:
  -PpythonVersion=3.11 \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_ValidatesContainer_Dataflow.yml

@@ -90,6 +90,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_ValidatesContainer_Dataflow_With_RC.yml

@@ -91,6 +91,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_ValidatesRunner_Dataflow.yml

@@ -96,6 +96,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_ValidatesRunner_Flink.yml

@@ -90,6 +90,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_ValidatesRunner_Samza.yml

@@ -88,6 +88,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_ValidatesRunner_Spark.yml

@@ -88,6 +88,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Xlang_Gcp_Dataflow.yml

@@ -82,5 +82,5 @@ jobs:
  uses: actions/upload-artifact@v3
  if: always()
  with:
- name: archiveJunit
+ name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Python_Xlang_Gcp_Direct.yml

@@ -81,5 +81,5 @@ jobs:
  uses: actions/upload-artifact@v3
  if: always()
  with:
- name: archiveJunit
+ name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_Sickbay_Python.yml

@@ -91,6 +91,7 @@ jobs:
  -PpythonVersion=${{ matrix.python_version }} \
  - name: Archive code coverage results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_TransformService_Direct.yml

@@ -92,5 +92,5 @@ jobs:
  uses: actions/upload-artifact@v3
  if: always()
  with:
- name: archiveJunit
+ name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_XVR_Direct.yml

@@ -94,9 +94,16 @@ jobs:
  arguments: |
  -PpythonVersion=${{ matrix.python_version }} \
  -PskipNonPythonTask=false \
- - name: Archive code coverage results
+ - name: Archive JUnit Test Results
  uses: actions/upload-artifact@v3
+ if: failure()
+ with:
+ name: JUnit Test Results
+ path: "**/build/reports/tests/"
+ - name: Publish JUnit Test Results
+ uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
- name: archiveJunit
- path: "**/build/test-results/**/*.xml"
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_XVR_Flink.yml

@@ -95,9 +95,16 @@ jobs:
  arguments: |
  -PpythonVersion=${{ matrix.python_version }} \
  -PskipNonPythonTask=false \
- - name: Archive code coverage results
+ - name: Archive JUnit Test Results
  uses: actions/upload-artifact@v3
+ if: failure()
+ with:
+ name: JUnit Test Results
+ path: "**/build/reports/tests/"
+ - name: Publish JUnit Test Results
+ uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
- name: archiveJunit
- path: "**/build/test-results/**/*.xml"
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_XVR_JavaUsingPython_Dataflow.yml

@@ -82,9 +82,16 @@ jobs:
  gradle-command: :runners:google-cloud-dataflow-java:validatesCrossLanguageRunnerJavaUsingPython
  arguments: |
  -PpythonVersion=${{ matrix.python_version }} \
- - name: Archive code coverage results
+ - name: Archive JUnit Test Results
  uses: actions/upload-artifact@v3
+ if: failure()
+ with:
+ name: JUnit Test Results
+ path: "**/build/reports/tests/"
+ - name: Publish JUnit Test Results
+ uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
- name: archiveJunit
- path: "**/build/test-results/**/*.xml"
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_XVR_PythonUsingJavaSQL_Dataflow.yml

@@ -83,5 +83,5 @@ jobs:
  uses: actions/upload-artifact@v3
  if: always()
  with:
- name: archiveJunit
+ name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_XVR_PythonUsingJava_Dataflow.yml

@@ -86,5 +86,5 @@ jobs:
  uses: actions/upload-artifact@v3
  if: always()
  with:
- name: archiveJunit
+ name: python-code-coverage-report
  path: "**/pytest*.xml"

.github/workflows/beam_PostCommit_XVR_Samza.yml

@@ -94,9 +94,16 @@ jobs:
  arguments: |
  -PpythonVersion=${{ matrix.python_version }} \
  -PskipNonPythonTask=false \
- - name: Archive code coverage results
+ - name: Archive JUnit Test Results
  uses: actions/upload-artifact@v3
+ if: failure()
+ with:
+ name: JUnit Test Results
+ path: "**/build/reports/tests/"
+ - name: Publish JUnit Test Results
+ uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
- name: archiveJunit
- path: "**/build/test-results/**/*.xml"
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PostCommit_XVR_Spark3.yml

@@ -94,9 +94,16 @@ jobs:
  arguments: |
  -PpythonVersion=${{ matrix.python_version }} \
  -PskipNonPythonTask=false \
- - name: Archive code coverage results
+ - name: Archive JUnit Test Results
  uses: actions/upload-artifact@v3
+ if: failure()
+ with:
+ name: JUnit Test Results
+ path: "**/build/reports/tests/"
+ - name: Publish JUnit Test Results
+ uses: EnricoMi/publish-unit-test-result-action@v2
  if: always()
  with:
- name: archiveJunit
- path: "**/build/test-results/**/*.xml"
+ commit: '${{ env.prsha || env.GITHUB_SHA }}'
+ comment_mode: ${{ github.event_name == 'issue_comment' && 'always' || 'off' }}
+ files: '**/build/test-results/**/*.xml'

.github/workflows/beam_PreCommit_Java.yml

@@ -192,6 +192,13 @@ jobs:
  files: '**/build/test-results/**/*.xml'
  - name: Archive SpotBugs Results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: SpotBugs Results
- path: "**/build/reports/spotbugs/*.html"
+ path: '**/build/reports/spotbugs/*.html'
+ - name: Publish SpotBugs Results
+ uses: jwgmeligmeyling/spotbugs-github-action@v1.2
+ if: always()
+ with:
+ name: Publish SpotBugs
+ path: '**/build/reports/spotbugs/*.html'

.github/workflows/beam_PreCommit_Java_Amazon-Web-Services2_IO_Direct.yml

@@ -128,6 +128,13 @@ jobs:
  files: '**/build/test-results/**/*.xml'
  - name: Archive SpotBugs Results
  uses: actions/upload-artifact@v3
+ if: always()
  with:
  name: SpotBugs Results
- path: "**/build/reports/spotbugs/*.html"
+ path: '**/build/reports/spotbugs/*.html'
+ - name: Publish SpotBugs Results
+ uses: jwgmeligmeyling/spotbugs-github-action@v1.2
+ if: always()
+ with:
+ name: Publish SpotBugs
+ path: '**/build/reports/spotbugs/*.html'