Latency regression in latest releases

[dhleong]

Hello! I've been using v3 in CI for quite some time, but when I went to set up a new project I thought I'd dust off the cobwebs and clean up some warnings that github was showing about old versions. I have not tried to bisect or anything, but version 12.5 took 2 minutes to run (!), and most of it appeared to be just hanging after the "post" step of writing to cache. I downgraded to 3.5 (which includes version updates, at least) and it was back to running almost instantaneously. Is this something you're seeing elsewhere as well, or is it something peculiar about my setup?

Thanks for your help and all your work on these actions!

[frenchy64]

I'm also seeing this.

Here's a build that pauses for 2m even after a cache hit: https://github.com/threatgrid/ctia/actions/runs/11637562591/job/32410961520?pr=1453#step:6:20

Possibly related, here's another build that saves a cache that seems to have already been a cache hit: https://github.com/threatgrid/ctia/actions/runs/11598161623/job/32293388110#step:6:24

[frenchy64]

FWIW the symptoms remind me of actions/cache#597

The ocaml action added explicit exit calls after saving the cache:

• Some discussion: cache: too slow cleanup actions/toolkit#1578
• Code: Force post-process exit with process.exit ocaml/setup-ocaml#724

msys2 also has a similar fix, they observed the problem started with the Node 16=>20 upgrade.

[frenchy64]

@dhleong Could you try 2d31e3e via:

DeLaGuardo/setup-clojure@2d31e3ee6005571f8b20ab9c0827f29238f58d88

EDIT: updated the SHA

[DeLaGuardo]

@dhleong Please try the new 12.6 release and let me know if you still expirience this issue. Huge shout out to @frenchy64 for the fix.

[dhleong]

That seems to have fixed it for me! Thank you both 🙏