Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hyper CI build is running out of disk space #11377

Closed
dfandrich opened this issue Jun 22, 2023 · 2 comments
Closed

Hyper CI build is running out of disk space #11377

dfandrich opened this issue Jun 22, 2023 · 2 comments
Labels
CI Continuous Integration tests

Comments

@dfandrich
Copy link
Contributor

The Hyper CI build on GHA has been failing regularly lately, and the latest build provides a pretty good clue as to why:

hyper
The hosted runner: GitHub Actions 91 lost communication with the server. Anything in your workflow that terminates the runner process, starves it for CPU/Memory, or blocks its network access can cause this error.

hyper
You are running out of disk space. The runner will stop working when the machine runs out of disk space. Free space left: 0 MB

The build fails in the run tests step, well after Hyper and curl have been built, so it's probably sufficient to delete the Hyper sources after it's built to recover some space.
@dfandrich dfandrich added tests CI Continuous Integration labels Jun 22, 2023
dfandrich added a commit that referenced this issue Jun 24, 2023
@dfandrich
CI: remove Hyper source & deps after build to free space
f9bc3ef 
These fill up the available disk space enough that the subsequent test
run fails, and aren't needed after the build completes.

Fixes #11377
@dfandrich
Copy link
Contributor Author

I tried a build with code to clean up the Hyper sources and rust dependencies that it downloads before running the tests, but it didn't help with the disk space warnings. It may be that that the Hyper build has some super verbose logging enabled that rapidly fills up the disk, or it may be something completely different.

@icing
Copy link
Contributor

icing commented Jun 26, 2023 •
edited
Loading

I see an infinite loop with hyper in tests, just filling up the trace log. Investigating.

See #11385 for a fix of this. I introduced a infinite loop with debug log statements in my previous hyper PR. Sorry about that. Don't know why it was not caught on the original PR.

icing added a commit to icing/curl that referenced this issue Jun 26, 2023
@icing
Hyper, fix EOF handling on input
bdbbb0c 
- refs curl#11377, where we ran out of disc space due to an infinite loop
  with debug logging
@icing icing mentioned this issue Jun 26, 2023
Closed
@bagder bagder closed this as completed in def99e0 Jul 8, 2023
bch pushed a commit to bch/curl that referenced this issue Jul 19, 2023
@icing @bch
hyper: fix EOF handling on input
6268ef8 
We ran out of disc space due to an infinite loop with debug logging

Fixes curl#11377
Closes curl#11385
Reported-by: Dan Fandrich
ptitSeb pushed a commit to wasix-org/curl that referenced this issue Sep 25, 2023
@icing @ptitSeb
hyper: fix EOF handling on input
5fe5993 
We ran out of disc space due to an infinite loop with debug logging

Fixes curl#11377
Closes curl#11385
Reported-by: Dan Fandrich
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CI Continuous Integration tests
Milestone
No milestone
Development

No branches or pull requests
2 participants
@icing @dfandrich