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

Increase page size for azure windows agents? #155

Closed
h-vetinari opened this issue Jun 30, 2021 · 5 comments · Fixed by #156
Closed

Increase page size for azure windows agents? #155

h-vetinari opened this issue Jun 30, 2021 · 5 comments · Fixed by #156

Comments

@h-vetinari
Copy link
Member

After often encountering

OSError: [WinError 1455] The paging file is too small for this operation to complete

in conda-forge/numpy-feedstock#237 & many other pull requests (for numpy), I decided to have a quick look at what's behind that.

Stumbling over this discussion (regarding GH Actions), someone noted:

I’ve investigated this issue and the problem is that Windows decides to create a single page file on the D:\ drive, which has only 14 GB, and by default it allocates only 1/8th of the volume size, or about 1.75 GB. This is a very small page file by modern standards and many applications will fail.

How about adding an option (or step) to the ci-setup that allows setting the page file size? It seems to be possible for GHAs - someone published an action for it (it's MIT licensed so we could even reuse some of that code).

@isuruf
Copy link
Member

isuruf commented Jun 30, 2021

PRs welcome. I don't think we need an option for this. Doing this unconditionally makes sense.

@isuruf
Copy link
Member

isuruf commented Jun 30, 2021

Wait, maybe only if CONDA_BLD_PATH is set to C://bld//

@h-vetinari
Copy link
Member Author

Core of such a PR would IMO be copying SetPageFileSize.ps1 and calling it from run_conda_forge_build_setup_win.bat appropriately (i.e. if-cond & params) - would you want that in recipe/ or recipe/conda_forge_ci_setup or elsewhere?

@isuruf
Copy link
Member

isuruf commented Jun 30, 2021

In recipe is fine.

@h-vetinari
Copy link
Member Author

See #156

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

Successfully merging a pull request may close this issue.

2 participants