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

Set the default codespaces container larger #786

Closed
corbob opened this issue Jul 18, 2023 · 0 comments · Fixed by #791
Closed

Set the default codespaces container larger #786

corbob opened this issue Jul 18, 2023 · 0 comments · Fixed by #791
Labels
0 - _Triaging Issue is accepted, but a milestone has yet to be added for the issue BuildAutomation Issues for changes to the build system, without functional changes needed to the project

Comments

@corbob
Copy link
Member

corbob commented Jul 18, 2023

What You Are Seeing?

When trying to preview the docs site in a codespaces container, regularly the preview fails with exit code 143. This is dotnet exit code to indicate it ran out of memory. When this happens, I end up needing to change the codespace to the larger container.

The size of the container can be specified in the .devcontainer settings: https://docs.github.com/en/codespaces/setting-up-your-project-for-codespaces/configuring-dev-containers/setting-a-minimum-specification-for-codespace-machines

What is Expected?

The default container should have enough memory to preview the site.

How Did You Get This To Happen? (Steps to Reproduce)

  1. Open a codespace to review a PR.
  2. Try to run ./preview.sh in the codespace
@corbob corbob added 0 - _Triaging Issue is accepted, but a milestone has yet to be added for the issue BuildAutomation Issues for changes to the build system, without functional changes needed to the project labels Jul 18, 2023
pauby added a commit to pauby/chocolatey-docs that referenced this issue Jul 21, 2023
@pauby
(chocolatey#786) Set required memory for container
d8b0ecc
@pauby pauby mentioned this issue Jul 21, 2023
Merged
7 tasks
pauby added a commit to pauby/chocolatey-docs that referenced this issue Jul 21, 2023
@pauby
(chocolatey#786) Set required memory for container
b082dbd
pauby added a commit to pauby/chocolatey-docs that referenced this issue Jul 21, 2023
@pauby
(chocolatey#786) Set required memory for container
406756a
pauby added a commit to pauby/chocolatey-docs that referenced this issue Jun 1, 2024
@pauby
(chocolatey#786) Set required memory for container
62884bb
pauby added a commit to pauby/chocolatey-docs that referenced this issue Jun 1, 2024
@pauby
(chocolatey#786) Set required memory for container
3458ed6
@gep13 gep13 closed this as completed in #791 Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0 - _Triaging Issue is accepted, but a milestone has yet to be added for the issue BuildAutomation Issues for changes to the build system, without functional changes needed to the project
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

(#786) Set required memory for container pauby/chocolatey-docs
1 participant
@corbob