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

lower job memory reservations to accomodate AL2023 AMI #1900

Merged
merged 3 commits into from
Oct 26, 2023
Merged

Conversation

asjohnston-asf
Copy link
Member

@asjohnston-asf asjohnston-asf commented Oct 26, 2023

The ECS Agent running on the AL2023 AMI reserves slightly more memory than on the AL2 AMI. This leaves only 31,517 MB available for jobs on an r6id.xlarge; 126,722 available on an r6id.4xlarge, ect.

This PR lowers the memory reserved for jobs to 7,875 MB/vcpu so they'll fit on any of the r6id family.

Screenshot from 2023-10-25 17-16-57

TODO

  • double-check 10m rtc memory reservation

@asjohnston-asf asjohnston-asf requested a review from a team as a code owner October 26, 2023 01:49
@asjohnston-asf asjohnston-asf added the bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version label Oct 26, 2023
@asjohnston-asf
Copy link
Member Author

FYI, for comparison, I confirmed 31,603 MB of memory is available to jobs when running either the AL2 AMI or the NGAP AMI on an r6idn.xlarge.

@jhkennedy
Copy link
Contributor

We should run the autoRIFT golden test to ensure optical jobs still have enough memory. And maybe check the GUNWs as well (we should add a golden test for them)

@asjohnston-asf asjohnston-asf merged commit d8e167c into develop Oct 26, 2023
12 checks passed
@asjohnston-asf asjohnston-asf deleted the al2023 branch October 26, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants