aswinshakil commented on code in PR #5318:
URL: https://github.com/apache/ozone/pull/5318#discussion_r1332260890
##########
.github/workflows/ci.yml:
##########
@@ -306,6 +309,9 @@ jobs:
sudo mkdir .aws && sudo chmod 777 .aws && sudo chown 1000 .aws
popd
./hadoop-ozone/dev-support/checks/kubernetes.sh
+ - name: Summary of failures
+ run: cat target/${{ github.job }}/summary.txt
+ if: ${{ !cancelled() }}
Review Comment:
I have a quick question, why is it that for integration test we use
```
- name: Summary of failures
run: cat target/${{ github.job }}/summary.txt
if: always()
```
Shouldn't that also be only if the job is not canceled?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
