kubeflow / mcp-server / 29187740074
41%

DEFAULT BRANCH: main
Ran
Jobs 3
Files 34
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 40.501% (+5.4%) from 35.078%
29187740074

push

github

web-flow 
feat: add OpenTelemetry tracing for tool calls (#21)

* feat: add OpenTelemetry tracing for tool calls

Signed-off-by: priyank <priyank8445@gmail.com>

* fix: address telemetry review feedback

Signed-off-by: priyank <priyank8445@gmail.com>

* fix(telemetry): address review — SpanKind.CLIENT, set_status(ERROR), hoist tracer, noop kwargs

- Use SpanKind.CLIENT for tool spans (K8s API = external service)

- Replace manual record_exception() with set_status(Status(StatusCode.ERROR)) to avoid duplicate events and improve Jaeger trace visibility

- Move get_tracer() to _audit_wrap scope (once per tool, not per call)

- Add **kwargs and set_status() to _NoopTracer/_NoopSpan for API compatibility

Signed-off-by: priyank <priyank8445@gmail.com>

* fix: use standard OTEL_EXPORTER_OTLP_ENDPOINT env var

Replace custom KUBEFLOW_MCP_OTEL_ENDPOINT with the official OpenTelemetry env var across README, CLI help text, and config loader.

Signed-off-by: priyank <priyank8445@gmail.com>

* fix: address review — atexit shutdown, args preview, endpoint normalization, circuit breaker test

- Register atexit.shutdown on TracerProvider to flush in-flight spans

- Add tool.args_preview span attribute with masked kwargs (truncated to 300 chars)

- Auto-append /v1/traces for base URLs to match OTel SDK convention

- Add test for circuit breaker open path (tool.success=False)

- Document kubeflow-mcp-agent as separate Jaeger service

Signed-off-by: priyank <priyank8445@gmail.com>

* feat(telemetry): align OpenTelemetry tracing with MCP semantic conventions

Signed-off-by: priyank <priyank8445@gmail.com>

* fix(telemetry): address review — middleware ContextVars, export timeouts, env var fallback, service rename

Signed-off-by: priyank <priyank8445@gmail.com>

---------

Signed-off-by: priyank <priyank8445@gmail.com>

987 of 2437 relevant lines covered (40.5%)

1.22 hits per line

Coverage Regressions

Lines Coverage ∆ File
68
59.05
 24.46% core/server.py
14
80.23
 1.22% core/config.py
13
82.42
 1.02% cli.py
Jobs
ID Job ID Ran Files Coverage
1 python-3.12 - 29187740074.1 34
40.5
 GitHub Action Run
2 python-3.10 - 29187740074.2 34
40.5
 GitHub Action Run
3 python-3.11 - 29187740074.3 34
40.5
 GitHub Action Run
Source Files on build 29187740074