Skip to content

feat: add rfc for an external compute driver - #1344

Closed
cheese-head wants to merge 1 commit into
NVIDIA:mainfrom
cheese-head:feature/external-compute-driver
Closed

feat: add rfc for an external compute driver#1344
cheese-head wants to merge 1 commit into
NVIDIA:mainfrom
cheese-head:feature/external-compute-driver

Conversation

@cheese-head

Copy link
Copy Markdown
Contributor

Summary

Related Issue

Changes

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Patrick Riel <priel@nvidia.com>
@copy-pr-bot

Copy link
Copy Markdown

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.

@github-actions

Copy link
Copy Markdown

Thank you for your interest in contributing to OpenShell, @cheese-head.

This project uses a vouch system for first-time contributors. Before submitting a pull request, you need to be vouched by a maintainer.

To get vouched:

  1. Open a Vouch Request discussion.
  2. Describe what you want to change and why.
  3. Write in your own words — do not have an AI generate the request.
  4. A maintainer will comment /vouch if approved.
  5. Once vouched, open a new PR (preferred) or reopen this one after a few minutes.

See CONTRIBUTING.md for details.

@drewdrew reopened this May 13, 2026
@drewdrew added the rfc label May 14, 2026
@drewdrew moved this from Todo to In progress in OpenShell RoadmapMay 15, 2026
string driver_version = 2;
string default_image = 3;
bool supports_gpu = 4;
uint32 gpu_count = 5;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As a general question: Are these fields not too prescriptive? Please see #1360 where we propose a richer API which also allows some prefiltering by the gateway but a final decision by the driver.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@cheese-head@elezar@drew@johntmyers