stacklok / toolhive / 21868499263 / 1
62%
main: 62%

DEFAULT BRANCH: main
Ran
Files 519
Run time 12s
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: 61.383% (-0.03%) from 61.412%
21868499263.1

push

github

web-flow 
Add UseLegacyAttributes flag to telemetry config (#3729)

* Add UseLegacyAttributes flag to telemetry config

Add a new UseLegacyAttributes boolean field to the telemetry Config
struct, defaulting to true for backward compatibility. This flag will
control whether legacy (pre-MCP OTEL semconv) attribute names are
emitted alongside the new standard names. The default will change to
false in a future release.

Plumbs the flag through:
- CLI (--otel-use-legacy-attributes)
- Config file (use-legacy-attributes)
- Operator spectoconfig
- Runner config builder
- VirtualMCPServer CRD (useLegacyAttributes, default: true)

The flag is not yet consumed by the telemetry middleware; a follow-up
PR will add the conditional attribute emission logic.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* generate docs

Signed-off-by: Chris Burns <29541485+ChrisJBurns@users.noreply.github.com>

* Remove omitempty from UseLegacyAttributes bool field

Go's omitempty on a bool treats false as the zero value and silently
drops it during JSON/YAML serialization. This meant that explicitly
setting useLegacyAttributes: false in config or CRD would be lost on
round-trip, reverting to the default true.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Signed-off-by: Chris Burns <29541485+ChrisJBurns@users.noreply.github.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

42419 of 69105 relevant lines covered (61.38%)

76.59 hits per line

Source Files on job 21868499263.1