Skip to content

server: do not pass log file to children - #29212

Merged
ngxson merged 1 commit into
ggml-org:masterfrom
dfriehs:router-log-clobber
Sep 22, 2026
Merged

ngxson merged 1 commit into
ggml-org:masterfrom
dfriehs:router-log-clobber

Conversation

@dfriehs

@dfriehs dfriehs commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Overview

Child processes clobbered the router log if the router was started with --log-file.

Requirements

@dfriehs
dfriehs requested a review from a team as a code owner September 21, 2026 10:12
@ngxson
ngxson merged commit bbf99b1 into ggml-org:master Sep 22, 2026
1 check passed
@kyano

kyano commented Sep 23, 2026

Copy link
Copy Markdown

This PR breaks “per-model logging”.

[*]
log-colors = off
log-prefix = true
log-timestamps = true
; no `log-file` here

[modelA]
log-file = /log/model-a.log

[modelB]
log-file = /log/model-b.log

Maybe this way is not usual/encouraged?

@ngxson

ngxson commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

@kyano that should be fixed in #29334

LadislavSopko pushed a commit to 0ics-srls/llama.cpp that referenced this pull request Oct 5, 2026
frostyautumnleaf pushed a commit to frostyautumnleaf/llama.cpp that referenced this pull request Oct 5, 2026
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 7, 2026
@dfriehs
dfriehs deleted the router-log-clobber branch October 8, 2026 04:35
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 8, 2026
Sign up for free to 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.

3 participants