Skip to content

Implement basic DRA support for NIMService#521

Merged
varunrsekar merged 6 commits intoNVIDIA:mainfrom
varunrsekar:basic-dra
Jun 10, 2025
Merged

Implement basic DRA support for NIMService#521
varunrsekar merged 6 commits intoNVIDIA:mainfrom
varunrsekar:basic-dra

Conversation

@varunrsekar
Copy link
Copy Markdown
Collaborator

@varunrsekar varunrsekar commented Jun 3, 2025

This PR implements basic support for requesting DRA resources w/ NIMService.

TODO:

  • Status API
  • API validations

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Jun 3, 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.

Comment thread api/apps/v1alpha1/common_types.go Outdated
Signed-off-by: Varun Ramachandra Sekar <vsekar@nvidia.com>
Signed-off-by: Varun Ramachandra Sekar <vsekar@nvidia.com>
Signed-off-by: Varun Ramachandra Sekar <vsekar@nvidia.com>
Signed-off-by: Varun Ramachandra Sekar <vsekar@nvidia.com>
Copy link
Copy Markdown
Collaborator

@shivamerla shivamerla left a comment

Choose a reason for hiding this comment

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

LGTM! status updates are pending

Signed-off-by: Varun Ramachandra Sekar <vsekar@nvidia.com>
@varunrsekar
Copy link
Copy Markdown
Collaborator Author

LGTM! status updates are pending

Thanks @shivamerla . I'll take up the status implementation in a follow-up.

shivamerla
shivamerla previously approved these changes Jun 9, 2025
@varunrsekar varunrsekar changed the title DRAFT: Implement basic DRA support for NIMService Implement basic DRA support for NIMService Jun 9, 2025
Signed-off-by: Varun Ramachandra Sekar <vsekar@nvidia.com>
@varunrsekar varunrsekar merged commit 33fb25b into NVIDIA:main Jun 10, 2025
9 checks passed
@varunrsekar varunrsekar deleted the basic-dra branch June 10, 2025 00:17
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.

2 participants