ci: apply cache size limit and print ccache statistics in "ARM64 Android APK" task #25530

pull sogoagain wants to merge 1 commits into bitcoin:master from sogoagain:2022-07-ccache-android-ci changing 1 files +5 −2

sogoagain commented at 3:43 PM on July 2, 2022: contributor

Fixes #25475

Modified ci/test/06_script_a.sh file to apply cache size limit and print ccahce statistics in "ARM64 Android APK" task.

Please feel free to give me any feedback. Thanks.
DrahtBot added the label Tests on Jul 2, 2022
fanquake requested review from hebasto on Jul 3, 2022
MarcoFalke commented at 8:00 AM on July 4, 2022: member

Please squash your commits according to https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#squashing-commits
hebasto approved
hebasto commented at 12:39 PM on July 4, 2022: member

Approach ACK 7899294d2ec9053f61fb89f015c676be91007bf2, https://api.cirrus-ci.com/v1/task/6391990253256704/logs/ci.log looks good.

Of course, #25530 (comment) should be addressed before merging.
[ci] apply cache size limit and print ccache statistics in "ARM64 Android APK" 31346a3196
sogoagain force-pushed on Jul 4, 2022
hebasto commented at 1:27 PM on July 4, 2022: member

Looking at the very low cache hit rate, I'm wondering whether we should use the NDK_CCACHE environment variable as it has documented here?
hebasto approved
hebasto commented at 1:30 PM on July 4, 2022: member

ACK 31346a31962ab06d49cb45aee8acd92d8806538b, my previous comment can be considered as a suggestion for a follow up.
MarcoFalke merged this on Jul 4, 2022
MarcoFalke closed this on Jul 4, 2022
sogoagain commented at 2:26 PM on July 4, 2022: contributor

@hebasto Yes. The cache hit ratio is definitely low. I will look into the issue starting with NDK_CCACHE environment variable. Thank you.
sogoagain deleted the branch on Jul 4, 2022
sidhujag referenced this in commit 86f85be987 on Jul 4, 2022
DrahtBot locked this on Jul 4, 2023

Contributors

Labels

Linked (view graph)

#25475 ci: Handle `ccache` in the proper way in "ARM64 Android APK" task