Fix CI rules not filtering integration test files properly

[lengrongfu]

fixes #1639

[lengrongfu]

What is the wrong job in this pipeline?

[yihuaf]

What is the wrong job in this pipeline?

It timed out after 15 min. I kick it again and see what happens.

[yihuaf]

@lengrongfu I believe your change is good additions, but doesn't fix the underlying issue. I just checked this change with another PR I have and it doesn't trigger with integration test file changes. I believe the correct fix is in the review. Please make the change and I can merge :)

[codecov-commenter]

Codecov Report

Merging #1643 (4bd1f57) into main (4303e4d) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1643      +/-   ##
==========================================
- Coverage   68.96%   68.95%   -0.01%     
==========================================
  Files         120      120              
  Lines       13156    13156              
==========================================
- Hits         9073     9072       -1     
- Misses       4083     4084       +1     

[yihuaf]

I apologize. I think I screwed up with directly merging the suggestions. The commits are not signed off. Feel free to override my commit with the suggested change and force push again...

[lengrongfu]

I will run effective tests to make sure this problem is fixed before going from draft to official pr.

[yihuaf]

I pass the test first, then review pr.

Can you elaborate what you mean?

[yihuaf]

I will run effective tests to make sure this problem is fixed before going from draft to official pr.

I mean sure :) But I already tested for you. You can just include my suggested change (squash the commits) and then I can approve.

[yihuaf]

LGTM. Thank you.