Merge pull request #280 from DataDog/ivoanjo/very-proto-files-try-2

Allow verify trace-protobuf to continue on error, try 2
DataDog · Nov 1, 2023 · c48f7be · c48f7be
2 parents 8cbc9db + 4b5573f
commit c48f7be
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/.github/workflows/verify-proto-files.yml b/.github/workflows/verify-proto-files.yml
@@ -1,5 +1,7 @@
 name: 'Verify trace-protobuf'
-on: [push]
+on:
+  pull_request:
+    types: [ opened, synchronize, reopened ]
 
 jobs:
   verify-proto-files:
@@ -24,7 +26,7 @@ jobs:
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           outcome: ${{ steps.agent_payload_proto.outcome }}
-          test-id: agent_payload_proto
+          test-id: Verify trace-protobuf agent_payload_proto
       - name: diff tracer_payload.proto
         working-directory: trace-protobuf/src/pb
         continue-on-error: true
@@ -39,7 +41,7 @@ jobs:
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           outcome: ${{ steps.tracer_payload_proto.outcome }}
-          test-id: tracer_payload_proto
+          test-id: Verify trace-protobuf tracer_payload_proto
       - name: diff stats.proto
         working-directory: trace-protobuf/src/pb
         continue-on-error: true
@@ -54,7 +56,7 @@ jobs:
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           outcome: ${{ steps.stats_proto.outcome }}
-          test-id: stats_proto
+          test-id: Verify trace-protobuf stats_proto
       - name: diff span.proto
         working-directory: trace-protobuf/src/pb
         continue-on-error: true
@@ -69,4 +71,4 @@ jobs:
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           outcome: ${{ steps.span_proto.outcome }}
-          test-id: span_proto
+          test-id: Verify trace-protobuf span_proto