Don't create duplicate issues on workflow failures (#8441)

Co-authored-by: opentelemetrybot <[email protected]>

Author: trask

.github/workflows/build-daily-no-build-cache.yml

@@ -27,9 +27,11 @@ jobs:
   # markdown-link-check and misspell-check are not included here because they don't use gradle cache
   # anyway and so are already covered by the normal daily build
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - common
       - test-latest-deps
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: needs.common.result == 'success' && needs.test-latest-deps.result == 'success'

.github/workflows/build-daily.yml

@@ -36,12 +36,19 @@ jobs:
   misspell-check:
     uses: ./.github/workflows/reusable-misspell-check.yml
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - common
       - test-latest-deps
       - muzzle
       - markdown-link-check
       - misspell-check
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: |
+        needs.common.result == 'success' &&
+        needs.test-latest-deps.result == 'success' &&
+        needs.muzzle.result == 'success' &&
+        needs.markdown-link-check.result == 'success' &&
+        needs.misspell-check.result == 'success'

.github/workflows/codeql-daily.yml

@@ -35,9 +35,10 @@ jobs:
       - name: Perform CodeQL analysis
         uses: github/codeql-action/analyze@v2
 
-  open-issue-on-failure:
-    # open an issue on failure because it can be easy to miss CI failure notifications
+  workflow-notification:
     needs:
       - analyze
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: needs.analyze.result == 'success'

.github/workflows/overhead-benchmark-daily.yml

@@ -45,8 +45,10 @@ jobs:
           committer_name: opentelemetrybot
           committer_email: [email protected]
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - run-overhead-tests
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: needs.run-overhead-tests.result == 'success'

.github/workflows/publish-smoke-test-fake-backend-images.yml

@@ -68,9 +68,12 @@ jobs:
         with:
           arguments: ":smoke-tests:images:fake-backend:dockerPush -PextraTag=${{ env.TAG }}"
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - publishLinux
       - publishWindows
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: needs.publishLinux.result == 'success' && needs.publishWindows.result == 'success'
+

.github/workflows/publish-smoke-test-grpc-images.yml

@@ -16,8 +16,10 @@ jobs:
       project: ":smoke-tests:images:grpc"
       publish: true
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - publish
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: ${{ needs.publish.result == 'success' }}

.github/workflows/publish-smoke-test-play-images.yml

@@ -20,8 +20,10 @@ jobs:
       skip-java-17: true
       skip-java-19: true
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - publish
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: ${{ needs.publish.result == 'success' }}

.github/workflows/publish-smoke-test-quarkus-images.yml

@@ -18,8 +18,10 @@ jobs:
       # Quarkus 2.0+ does not support Java 8
       skip-java-8: true
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - publish
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: ${{ needs.publish.result == 'success' }}

.github/workflows/publish-smoke-test-security-manager-images.yml

@@ -16,8 +16,10 @@ jobs:
       project: ":smoke-tests:images:security-manager"
       publish: true
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - publish
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: ${{ needs.publish.result == 'success' }}

.github/workflows/publish-smoke-test-servlet-images.yml

@@ -76,8 +76,10 @@ jobs:
         if: matrix.os == 'windows-latest'
         run: ./gradlew :smoke-tests:images:servlet:buildWindowsTestImages pushMatrix -PextraTag=${{ needs.prepare.outputs.tag }} -PsmokeTestServer=${{ matrix.smoke-test-server }}
 
-  open-issue-on-failure:
+  workflow-notification:
     needs:
       - publish
-    if: failure() && github.run_attempt == 1
-    uses: ./.github/workflows/reusable-open-issue-on-failure.yml
+    if: always()
+    uses: ./.github/workflows/reusable-workflow-notification.yml
+    with:
+      success: ${{ needs.publish.result == 'success' }}