cache: too slow cleanup

[smorimoto]

I'm opening this issue as a blank issue because it's still not clear if this is a bug!

For some reason, calling the cache save function in the post phase seems to cause a lot of blank time after uploading the cache. This did not happen until a few months ago, if I remember correctly. I just want to check if the actions team has a clear idea of what caused this.

[smorimoto]

Reference to a possibly relevant issue: actions/cache#597

[smorimoto]

I worked around this problem with this approach: ocaml/setup-ocaml#724

[Kith330]

818449

[lazka]

We are hitting the same issue at https://github.com/msys2/setup-msys2, 2 minutes hangs after everything is executed and a cache was saved before.

I did some testing and I can confirm that downgrading to node16 from node20 fixes the issue (can anyone confirm this?). So it seems like a bug to me.

I can also confirm that calling process.exit fixes it as well.

[smorimoto]

It's definitely a bug, but it's weird how changing the runtime changes the behaviour. I will look for the cause.

[lazka]

I don't know much about node, but using process.getActiveResourcesInfo() at the end shows that there are three TCPSocketWrap still active, while they aren't with node 16, or if no cache is uploaded.