This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new fcf3b4bde1 GH-47751: [CI] Fix check for job to ignore on reporting
(#47755)
fcf3b4bde1 is described below
commit fcf3b4bde10bf710ec14bb9d5ef5e2fc6668b494
Author: Raúl Cumplido <[email protected]>
AuthorDate: Thu Oct 9 06:14:19 2025 +0200
GH-47751: [CI] Fix check for job to ignore on reporting (#47755)
### Rationale for this change
It failed notifying on Zulip and the email, see:
https://lists.apache.org/thread/12glcwk1jb8lb5x6lw3t2vr6zn9kp66v
We basically were ignoring all jobs except the report-ci :disappointed:
### What changes are included in this PR?
The check was wrong, I clearly did not test this properly after the latest
change, sorry about that.
### Are these changes tested?
I validated locally, without the change:
```
$ archery ci report-chat --dry-run --ignore report-ci --repository
apache/arrow 18329667061
*Extra CI GitHub report for
<https://github.com/apache/arrow/actions/runs/18329667061|C++ Extra>*
:tada: *1 successful jobs*
```
It shows successful because it has now finished, this is not the case when
is running on CI, but only a single job which is not expected, it should show 5!
with the change:
```
archery ci report-chat --dry-run --ignore report-ci --repository
apache/arrow 18329667061
*Extra CI GitHub report for
<https://github.com/apache/arrow/actions/runs/18329667061|C++ Extra>*
:tada: *5 successful jobs*
```
### Are there any user-facing changes?
No
* GitHub Issue: #47751
Authored-by: Raúl Cumplido <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
---
dev/archery/archery/ci/core.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dev/archery/archery/ci/core.py b/dev/archery/archery/ci/core.py
index 45364ffb0d..42a8092e09 100644
--- a/dev/archery/archery/ci/core.py
+++ b/dev/archery/archery/ci/core.py
@@ -70,7 +70,7 @@ class Workflow:
# from the reusable workflow this will be report-ci.
# The real job_data['name'] is the display name, like
# "report-extra-cpp / report-ci".
- if self.ignore_job in job_data.get('name'):
+ if self.ignore_job not in job_data.get('name'):
job = Job(job_data)
jobs.append(job)
return jobs
