Skip to content

Let OpenBot's own 4 CPU Podman machine through the engine check before setup - #1347

Merged
davidmckayv merged 1 commit into
release/1.0from
fix/engine-precheck-floor
Nov 9, 2026
Merged

davidmckayv merged 1 commit into
release/1.0from
fix/engine-precheck-floor

Conversation

@davidmckayv

Copy link
Copy Markdown
Contributor

#1343 added an engine check before the CopilotKit account step that asks for 6 CPUs and 8 GB. OpenBot creates its Podman machine with 4 CPUs and 6144 MiB (create_machine(4, 6144, 60) in main.rs), so every Podman install, the Windows default and a Mac without Docker Desktop, would stop there, though the back end's own check passes on that machine (fresh Windows installs on it passed today).

The check now stops only an engine that can never pass the back end's check: fewer than 4 CPUs, under 4 GiB of memory, or under 40 GiB of free disk. Anything larger is left to the back end at Start, which now names the short resource (#1343).

Test: a_small_or_full_engine_is_caught_before_local_setup adds OpenBot's 4 CPU / 6 GiB Podman machine and a 4 CPU / 8 GB Docker Desktop (both pass) and a 3 CPU engine (stopped). It failed before this change and passes after.

…e setup

The check added for a full Docker disk also asked for 6 CPUs and 8 GB, which every Podman machine
OpenBot creates (4 CPUs, 6 GiB) fails, though the back end's own check passes on it. It now stops
only an engine that can never pass: fewer than 4 CPUs, under 4 GiB of memory, or under 40 GiB of
free disk.
@davidmckayv
davidmckayv merged commit e57e3f5 into release/1.0 Nov 9, 2026
37 of 41 checks passed
@davidmckayv
davidmckayv deleted the fix/engine-precheck-floor branch November 9, 2026 06:18
davidmckayv added a commit that referenced this pull request Nov 9, 2026
Adds #1341, #1343 and #1347 to the 1.0 notes, worded from the merged code.
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.

1 participant