raulcd commented on code in PR #47546:
URL: https://github.com/apache/arrow/pull/47546#discussion_r2336987697
##########
.github/workflows/cpp_extra.yml:
##########
@@ -278,3 +278,36 @@ jobs:
cmake --build cpp/examples/minimal_build.build
cd cpp/examples/minimal_build
../minimal_build.build/arrow-example
+
+ report-extra-cpp:
+ runs-on: ubuntu-latest
+ needs:
+ - docker
+ - jni-macos
+ # We don't have the job id as part of the context neither the job name.
+ # The GitHub API exposes numeric id or job name but not the github.job
(report-extra-cpp).
+ # We match github.job to the name so we can pass it via context in order
to be ignored on the report.
+ # The job is still running.
+ name: ${{ github.job }}
+ if: always()
Review Comment:
I will update once we have tested properly.
The always() is required so it runs once the needs have finished even if
they fail.
```suggestion
if: github.event_name == 'schedule' && always()
```
##########
.github/workflows/cpp_extra.yml:
##########
@@ -278,3 +278,36 @@ jobs:
cmake --build cpp/examples/minimal_build.build
cd cpp/examples/minimal_build
../minimal_build.build/arrow-example
+
+ report-extra-cpp:
+ runs-on: ubuntu-latest
+ needs:
+ - docker
+ - jni-macos
+ # We don't have the job id as part of the context neither the job name.
+ # The GitHub API exposes numeric id or job name but not the github.job
(report-extra-cpp).
+ # We match github.job to the name so we can pass it via context in order
to be ignored on the report.
+ # The job is still running.
+ name: ${{ github.job }}
+ if: always()
+ steps:
+ - name: Checkout Arrow
+ uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 #
v5.0.0
+ with:
+ fetch-depth: 0
+ - name: Setup Python
+ uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c #
v6.0.0
+ with:
+ python-version: 3
+ - name: Setup Archery
+ run: python3 -m pip install -e dev/archery[crossbow]
+ - name: Notify
+ env:
+ GH_TOKEN: ${{ github.token }}
+ run: |
+ archery arrowci report-chat \
+ --ignore ${{ github.job }} \
+ --repository ${{ github.repository }} \
+ --dry-run \
+ --webhook ${{ secrets.ARROW_ZULIP_WEBHOOK }} \
Review Comment:
I will ask infra to add this token
##########
.github/workflows/cpp_extra.yml:
##########
@@ -278,3 +278,36 @@ jobs:
cmake --build cpp/examples/minimal_build.build
cd cpp/examples/minimal_build
../minimal_build.build/arrow-example
+
+ report-extra-cpp:
+ runs-on: ubuntu-latest
+ needs:
+ - docker
+ - jni-macos
+ # We don't have the job id as part of the context neither the job name.
+ # The GitHub API exposes numeric id or job name but not the github.job
(report-extra-cpp).
+ # We match github.job to the name so we can pass it via context in order
to be ignored on the report.
+ # The job is still running.
+ name: ${{ github.job }}
+ if: always()
+ steps:
+ - name: Checkout Arrow
+ uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 #
v5.0.0
+ with:
+ fetch-depth: 0
+ - name: Setup Python
+ uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c #
v6.0.0
+ with:
+ python-version: 3
+ - name: Setup Archery
+ run: python3 -m pip install -e dev/archery[crossbow]
+ - name: Notify
+ env:
+ GH_TOKEN: ${{ github.token }}
+ run: |
+ archery arrowci report-chat \
+ --ignore ${{ github.job }} \
+ --repository ${{ github.repository }} \
+ --dry-run \
Review Comment:
I'll remove the `--dry-run` before merging so we send messages
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
