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

fix(cogify): lower concurrency and set resource limits to memory = 2x cpu #176

Merged
merged 2 commits into from
Sep 18, 2023
Merged

fix(cogify): lower concurrency and set resource limits to memory = 2x cpu #176

merged 2 commits into from
Sep 18, 2023

Conversation

blacha
Copy link
Member

@blacha blacha commented Sep 18, 2023

Motivation

We are still getting pods killed by OOMKilled, possibly caused by too many gdal_translates happening at the same time.

Modification

Remove memory limits and double the requested memory
Lower concurrency of gdal_translates to 2 per pod rather than 4 per pod

@blacha blacha requested review from a team as code owners September 18, 2023 02:11
@blacha blacha added this pull request to the merge queue Sep 18, 2023
Merged via the queue into master with commit f391f68 Sep 18, 2023
1 check passed
@blacha blacha deleted the fix/cogify-memory-limit branch September 18, 2023 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wentao-Kuang Wentao-Kuang Wentao-Kuang approved these changes

@amfage amfage Awaiting requested review from amfage amfage is a code owner automatically assigned from linz/li-topo-data-engineering

@MDavidson17 MDavidson17 Awaiting requested review from MDavidson17 MDavidson17 is a code owner automatically assigned from linz/li-topo-data-engineering

@paulfouquet paulfouquet Awaiting requested review from paulfouquet paulfouquet is a code owner automatically assigned from linz/li-topo-data-engineering

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@blacha @Wentao-Kuang