Back

ollama #14106

add ability to turn on debug request logging

by drifkin · Feb 06, 2026 at 19:33 UTC · scan-989dd359fbfeb59b

View on Github · Download JSON
High Risk (55%)

Get this automatically on every PR

Install the Axiomo GitHub App to get Signals as check runs and PR comments on every pull request.

Install App

Risk Assessment

Risk level: High (55%)

Risk Drivers

  • touches_configuration: Modifies configuration code
  • cross_concern_spread: Touches 3 functional domains: api, configuration, testing
  • api_surface_change: API surface changed in 1 file(s)

Intent

4/4 criteria met

Implement debug logging for inference requests.

Non-Goals

  • - Provide production-level logging features
  • - Optimize performance for logging
  • - Introduce logging for other request types

Acceptance Criteria

  • ✓
    Debug logging is enabled when OLLAMA_DEBUG_LOG_REQUESTS is set

    envconfig/config.go adds DebugLogRequests config option

  • ✓
    Logs are stored in a temporary folder

    server/inference_request_log.go handles logging setup and file creation

  • ✓
    A shell script is created for request replay

    server/inference_request_log.go includes cURL script generation

  • ✓
    Includes tests for the new logging feature

    server/routes_request_log_test.go contains relevant tests

Confidence: 90.0% Source: pr description AI: openai

Contributors

drifkin PR Author 1 commit + Trusted
Account Age: 5878 days
Prior PRs: 47
Merged: 42

Trusted contributor with 42 merged PRs. has 267 followers. unfamiliar with 2 files.

Evidence

Evidence Completeness: 0.0%
Missing: ci_passing, tests_passing, lint_passing, security_scan_clean, coverage_maintained, build_successful

Supply Chain

None Risk
Modifies dependencies
Modifies lockfile
Modifies CI config
Modifies build scripts

Focus Files

Focus on 3 critical file(s)

envconfig/config.go +43

Modifies configuration code; Source code

critical
server/inference_request_log.go +144

144 lines changed; New file; Source code

critical
server/routes_request_log_test.go +128

128 lines changed; New file; Source code

critical
server/routes.go +16

Source code

medium

Triage

51

minutes to review

high

effort level

none

staleness risk

Allocate focused review time

Recommendation

NEEDS DISCUSSION 39.0% readiness

Insufficient evidence (CI/tests) to evaluate

Next Steps

Question

Why is ci_passing missing? Consider adding this check.

Question

Why is tests_passing missing? Consider adding this check.

Concern envconfig/config.go

Critical file: Modifies configuration code; Source code

Concern server/inference_request_log.go

Critical file: 144 lines changed; New file; Source code

Concern server/routes_request_log_test.go

Critical file: 128 lines changed; New file; Source code