Repository navigation
Conversation
Propose running the Docker and Podman compute drivers as external processes launched by the gateway, then moving the desktop drivers to ai-openshell/desktop. Refs #5200 Signed-off-by: Evan Lezar <elezar@nvidia.com>
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
@matthewgrossman @elezar #4848 would be part of this too right? |
The move to a separate repo would demonstrate how a driver / project in another repo would consume the conformance tests. Since we have #5138 to add a single binary, this will be consumed by the |
Signed-off-by: Evan Lezar <elezar@nvidia.com>
|
Yes. I've updated the RFC (11198f9) to cover this. B3 now describes conformance as the |
|
We've already proposed moving desktop driver to their own repo as part of RFC 0019. Can we roll this into RFC 0019 or a ticket? Feel free to push a new desktop specific plan to the RFC if you'd like though I would just put this into a ticket or series of tickets since we don't need much of a decision here. If I understand correctly the only net new thing we are asking for is a small service launcher pattern. |
I may have been erring on the side of getting feedback for this, and was concerned that an RFC was too heavyweight for this. In terms of "net new things", I don't quite agree that it's only a small service launcher pattern. The decisions that need review may not be called out clearly enough in the RFC and I will fix that. These are:
I can start on the more mechanical changes as issues and get working on them -- blocking on the decisions that are required -- and update the existing RFC with a desktop-specific plan instead of a new RFC. Update: I have updated #5200 with a set of issues for the more mechanical steps. These should cover "Secion A" from this RFC. I will migrate the other content to the existing RFC. |
Summary
Propose RFC 0022: move the desktop compute drivers (Docker, Podman, and later the microVM driver) to the
ai-openshell/desktoprepository. The move first requires running Docker and Podman as external driver processes launched by the gateway, which is Part A of the RFC; Part B covers the move and the core,desktop, andpackagesrelease flow.Related Issue
Refs #5200. That issue stays open as the tracking issue for the whole migration; this PR only adds the RFC.
Builds on the repository layout proposed in RFC 0019 (#4426, #4425).
Changes
rfc/0022-desktop-driver-migration/README.md.probeandpreflightcommands, and a small driver SDK carved out ofopenshell-core.ai-openshell/desktop, the git-tag SDK dependency, conformance, and apackagesbundle release.Release Notes
Testing
markdown:lintincluding Mermaid validation,markdown:format, andlicense:check)RFC-only change; Rust, SDK, and e2e suites are not applicable.
Checklist