volta-cli / action

GitHub Action to Setup Volta
https://volta.sh
MIT License
93 stars 23 forks source link

Frequent failures #77

Open ef4 opened 3 years ago

ef4 commented 3 years ago

This may not be the fault of this package, but I frequently lose a job to an HTTP 400 while running this action.

https://github.com/embroider-build/embroider/pull/1001/checks?check_run_id=3863504298

It's sporadic and it's not super common, but when you have enough jobs the chance of hitting it on any given workflow gets quite high.

Better logging of which HTTP request is failing might help, and maybe retry is appropriate.

ef4 commented 3 years ago

I figured out how to enable more logging in this action, so that may help me identify which URL is getting the 400.

- uses: volta-cli/action@v1
  env:
    ACTIONS_STEP_DEBUG: true

Once I get some failures with the logging on I can see about retries.

rwjblue commented 2 years ago

https://github.com/volta-cli/action/pull/97 is an initial stab at helping out here, we should now have logging to tell us if the failures are happening in tc.downloadTool or in the actual downloading of node / yarn / etc.

I have a few more ideas to implement also:

rwjblue commented 2 years ago

97 is published as volta-cli/action@v3.0.0-beta.1 if anyone wants to give it a whirl...

ef4 commented 2 years ago

https://github.com/embroider-build/embroider is now updated to v3, so we should hopefully see improvement from #97.

rwjblue commented 2 years ago

https://github.com/volta-cli/action/pull/101 should also help a bit here (released in v3.0.2). I was able to get to the bottom of at least one of the major causes of request failure that we were hitting.

JamesHenry commented 1 year ago

I have been receiving intermittent "could not unpack" errors on the lerna repo for some time now:

https://github.com/lerna/lerna/actions/runs/4500110940/jobs/7918811928?pr=3603

Volta error: Could not unpack Node v18.15.0

I recommend volta to everyone I interact with, it's so great for local usage. If we can somehow just resolve this weak link then it will basically be flawless in my eyes!

I'm going to try switching to the setup-node action for now because apparently it will check for volta.node in package.json these days

marcneander commented 1 year ago

I have been receiving intermittent "could not unpack" errors on the lerna repo for some time now:

https://github.com/lerna/lerna/actions/runs/4500110940/jobs/7918811928?pr=3603

Volta error: Could not unpack Node v18.15.0

I recommend volta to everyone I interact with, it's so great for local usage. If we can somehow just resolve this weak link then it will basically be flawless in my eyes!

I'm going to try switching to the setup-node action for now because apparently it will check for volta.node in package.json these days

We're getting the same issue since a few days ago.

mekwall commented 1 year ago

Volta error: Could not unpack Node v18.15.0

Same here. Any updates on this issue @rwjblue?

alexandr2110pro commented 1 year ago

We too. Screenshot 2023-05-09 at 16 40 29

rwjblue commented 1 year ago

FWIW, the Could not unpack Node issue is more likely to be an issue with volta itself than with this action (since at that point in the runtime, we are calling volta setup directly and the action isn't in the picture...)

NullVoxPopuli commented 1 year ago

I've been avoiding volta for C.I., since actions/setup-node supports reading the config, and explained here: https://github.com/wyvox/action-setup-pnpm/blob/main/action.yml#L130

ef4 commented 1 year ago

The "could not unpack" issue that we continue to see is almost certainly a problem with @actions/tool-cache. It's not clear that this action is really using that cache correctly, as we never see it hit the cache, and overwhelming the network filesystem that backs the cache seems to be the likely cause of the failures we're seeing.

korniychuk commented 10 months ago

It still reproducible in 5%-10% workflow runs. Screenshot 2024-02-06 at 21 26 34@2x