fix: quote the runroot path in the generated storage.conf (TOML) #10

Merged
mlvtito merged 1 commit from fix/storage-conf-toml into main 2026-11-15 21:37:11 +00:00
Collaborator

Follow-up to #7 (second dispatch): podman now fails to decode the generated storage.conf:

Failed to obtain podman configuration: decode configuration "/etc/containers/storage.conf": toml: line 3 (last key "storage.runroot"): expected value but found '/' instead

The runroot value was a bare path — TOML requires quoted strings. One-character fix: runroot = "/tmp/containers-runroot". The generated file was validated with a real TOML parser (tomllib) before pushing this time. Silver lining: the previous error is gone, so the vfs config IS being picked up.

Wiki updates on merge

Nothing to update in the Wiki.

Follow-up to #7 (second dispatch): podman now fails to decode the generated storage.conf: ``` Failed to obtain podman configuration: decode configuration "/etc/containers/storage.conf": toml: line 3 (last key "storage.runroot"): expected value but found '/' instead ``` The `runroot` value was a bare path — TOML requires quoted strings. One-character fix: `runroot = "/tmp/containers-runroot"`. The generated file was validated with a real TOML parser (tomllib) before pushing this time. Silver lining: the previous error is gone, so the vfs config IS being picked up. ## Wiki updates on merge Nothing to update in the Wiki.
TOML requires quoted strings; the bare /tmp path made podman fail the
config decode ("expected value but found '/'"). Validated with a real
TOML parser this time.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
serious-devs/ai-factory!10
No description provided.