Skip to content

Allow configurable sizeLimit for shared memory (emptyDir) Volumes#492

Merged
shengnuo merged 2 commits intoNVIDIA:mainfrom
shengnuo:empty-dir-size-limit
May 13, 2025
Merged

Allow configurable sizeLimit for shared memory (emptyDir) Volumes#492
shengnuo merged 2 commits intoNVIDIA:mainfrom
shengnuo:empty-dir-size-limit

Conversation

@shengnuo
Copy link
Copy Markdown
Collaborator

@shengnuo shengnuo commented May 13, 2025

This PR allows the size of emptyDir volumes to be limited

  • For NIMs, the size can be specified in .spec.storage.sharedMemorySizeLimit for NIMServices; and in .spec.services[].spec.storage.sharedMemorySizeLimit for NIMPipelines
  • For Customizer training jobs, the limit can be specified in .spec.trainingConfig.sharedMemorySizeLimit

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 13, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@shengnuo shengnuo changed the title Empty dir size limit Allow configurable sizeLimit for emptyDir Volumes May 13, 2025
@shengnuo shengnuo force-pushed the empty-dir-size-limit branch from 9c0e136 to d6652e8 Compare May 13, 2025 17:07
Comment thread api/apps/v1alpha1/nimservice_types.go Outdated
Comment thread api/apps/v1alpha1/nemo_customizer_types.go Outdated
Comment thread api/apps/v1alpha1/nemo_customizer_types.go Outdated
shengnuo added 2 commits May 13, 2025 18:40
Signed-off-by: Sheng Lin <shelin@nvidia.com>
Signed-off-by: Sheng Lin <shelin@nvidia.com>
@shengnuo shengnuo force-pushed the empty-dir-size-limit branch from d6652e8 to 2d6ae9d Compare May 13, 2025 22:41
@shivamerla
Copy link
Copy Markdown
Collaborator

@shengnuo please cherry-pick to release branch.

@shengnuo
Copy link
Copy Markdown
Collaborator Author

#493 PR to release-2.0 branch

@shengnuo shengnuo changed the title Allow configurable sizeLimit for emptyDir Volumes Allow configurable sizeLimit for shared memory (emptyDir) Volumes May 13, 2025
@shengnuo shengnuo merged commit 5c31e6f into NVIDIA:main May 13, 2025
9 checks passed
Copy link
Copy Markdown
Collaborator

@mkhaas mkhaas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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 this pull request may close these issues.

3 participants