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.

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

Docker: Use one process to decrease memory usage. #16060

Merged
merged 1 commit into from Sep 28, 2023

Conversation

marcfehling
Copy link
Member

This is another attempt to get the docker workflow fixed. The message we get from github is rather vague:

The hosted runner: GitHub Actions X 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.

I believe the only thing that could potentially fail from the description is that we run out of memory. I suggest to limit the number of processes explicitly to 1 with which we build deal.II in the Dockerfile.

@masterleinad can build the Docker image without problems, see #16047 (comment). It could be the system specs of the github machines that cause this.

Probably a nicer approach is to introduce a new argument for the number of processes to remain flexible, i.e., ARG NPROCS=2, and then set it explicitly to 1 for the focal image. But we could leave that for a follow-up PR. Let's try this out first!

Copy link
Member

@masterleinad masterleinad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try it!

@marcfehling marcfehling merged commit c00406d into dealii:master Sep 28, 2023
3 of 15 checks passed
@marcfehling
Copy link
Member Author

Reducing the number of jobs helped to prevent the focal build from crashing. There was an uploading issue for the jammy build, but I believe it's just a temporary problem. I call this a success!
https://github.com/dealii/dealii/actions/runs/6342478809

Runtime increased substantially as a result from using only one job. I will prepare some follow-up PRs to mitigate the impact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants