Skip to content
This repository was archived by the owner on Oct 6, 2025. It is now read-only.

Conversation

p1-0tr
Copy link
Member

@p1-0tr p1-0tr commented Jul 16, 2025

In some cases we may end up misidentifying the underlying engine type. Improve the logic for doing that to make it impossible to identify DD on windows or macos as plain moby.

In some cases we may end up misidentifying the underlying engine type.
Improve the logic for doing that to make it impossible to identify DD on
windows or macos as plain moby.

Signed-off-by: Piotr Stankiewicz <[email protected]>
@p1-0tr p1-0tr marked this pull request as ready for review July 17, 2025 10:09
@p1-0tr p1-0tr requested review from xenoscopic and doringeman July 22, 2025 07:17
kind = types.ModelRunnerEngineKindMoby
}

if modelRunnerHost != "" && kind == types.ModelRunnerEngineKindMoby {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This makes MODEL_RUNNER_HOST unusable on macOS and Windows.

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, I'll have to have another look into this.

@doringeman
Copy link
Collaborator

Improve the logic for doing that to make it impossible to identify DD on windows or macos as plain moby.

How's that possible at the moment?
If isDesktopContext(ctx, cli) is false because it fails to reach the engine, right?

@p1-0tr
Copy link
Member Author

p1-0tr commented Jul 22, 2025

How's that possible at the moment? If isDesktopContext(ctx, cli) is false because it fails to reach the engine, right?

Yep. It can happen if you try to interact with DMR while DD is resetting/starting (basically it's a race where during isDesktopContext the engine is not running yet, starts a tad later, but we've decided it is plain moby at that point).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants