[GitHub] [pulsar] nicoloboschi commented on a diff in pull request #17551: [ci] Skip run flaky tests suite for cpp-only or doc-only changes

[GitBox <gi...@apache.org>]

nicoloboschi commented on code in PR #17551:
URL: https://github.com/apache/pulsar/pull/17551#discussion_r966151871


##########
.github/workflows/pulsar-ci-flaky.yaml:
##########
@@ -22,9 +22,27 @@ on:
   pull_request:
     branches:
       - master
+    paths-ignore:
+      # Doc
+      - 'site2/**'
+      - 'deployment/**'
+      - '.asf.yaml'
+      - '*.md'
+      - '**/*.md'
+      # CPP
+      - 'pulsar-client-cpp/**'
   push:
     branches:
       - branch-*
+    paths-ignore:
+      # Doc
+      - 'site2/**'
+      - 'deployment/**'
+      - '.asf.yaml'
+      - '*.md'
+      - '**/*.md'
+      # CPP
+      - 'pulsar-client-cpp/**'

Review Comment:
   the [doc](https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions) says 
   >When all the path names match patterns in paths-ignore, the workflow will not run. If any path names do not match patterns in paths-ignore, even if some path names match the patterns, the workflow will run.
   
   what kind of risks you think we run ? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org