Skip to content

[DRAFT] Update Docker Resource Limits for Pathways Main Workload #479

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
Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

[DRAFT] Update Docker Resource Limits for Pathways Main Workload #479

wants to merge 1 commit into from

Conversation

SujeethJinesh
Copy link
Collaborator

Debating on if we should even remove memory limits entirely?

Fixes / Features

  • Update memory limits on the workload

Testing / Documentation

Tested on internal cluster

  • [ y ] Tests pass
  • [ y ] Appropriate changes to documentation are included in the PR

@SujeethJinesh SujeethJinesh force-pushed the sujinesh/upgrade_memory_limits_on_user_job branch 10 times, most recently from 598aaf4 to 85b44d2 Compare May 31, 2025 21:17
@SujeethJinesh SujeethJinesh force-pushed the sujinesh/upgrade_memory_limits_on_user_job branch from 85b44d2 to bf8b65a Compare June 1, 2025 09:09
@SujeethJinesh SujeethJinesh force-pushed the sujinesh/upgrade_memory_limits_on_user_job branch from bf8b65a to 3931c15 Compare June 1, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Obliviour Obliviour Awaiting requested review from Obliviour Obliviour will be requested when the pull request is marked ready for review Obliviour is a code owner

@44past4 44past4 Awaiting requested review from 44past4 44past4 will be requested when the pull request is marked ready for review 44past4 is a code owner

@sharabiani sharabiani Awaiting requested review from sharabiani sharabiani will be requested when the pull request is marked ready for review sharabiani is a code owner

@pawloch00 pawloch00 Awaiting requested review from pawloch00 pawloch00 will be requested when the pull request is marked ready for review pawloch00 is a code owner

@BluValor BluValor Awaiting requested review from BluValor BluValor will be requested when the pull request is marked ready for review BluValor is a code owner

@gcie gcie Awaiting requested review from gcie gcie will be requested when the pull request is marked ready for review gcie is a code owner

@RoshaniN RoshaniN Awaiting requested review from RoshaniN RoshaniN will be requested when the pull request is marked ready for review RoshaniN is a code owner

@scaliby scaliby Awaiting requested review from scaliby scaliby will be requested when the pull request is marked ready for review scaliby is a code owner

@jamOne- jamOne- Awaiting requested review from jamOne- jamOne- will be requested when the pull request is marked ready for review jamOne- is a code owner

@SikaGrr SikaGrr Awaiting requested review from SikaGrr SikaGrr will be requested when the pull request is marked ready for review SikaGrr is a code owner

@FIoannides FIoannides Awaiting requested review from FIoannides FIoannides will be requested when the pull request is marked ready for review FIoannides is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SujeethJinesh