Skip to content

Conversation

@sjvans
Copy link
Contributor

@sjvans sjvans commented Oct 10, 2025

adds unofficial "hacks" for running with xotel agent for testing purposes:

  • default to SERVER spans instead of INTERNAL spans to circumvent no sampling decision by xotel (but then rewrite to INTERNAL before export)
  • remove parentSpanId from http spans if there is no traceparent header
  • export filter for @opentelemetry/instrumentation-hdb spans

activate via cds.requires.telemetry._with_xotel = true

@sjvans sjvans changed the title fix: default to SpanKind.SERVER fix: default to SpanKind.SERVER + cds.requires.telemetry.resource Oct 10, 2025
@sjvans sjvans changed the title fix: default to SpanKind.SERVER + cds.requires.telemetry.resource feat: running together with CALM Oct 14, 2025
@sjvans sjvans changed the base branch from 1 to main October 27, 2025 13:45
@sjvans sjvans changed the title feat: running together with CALM fix: running together with xotel agent Nov 13, 2025
@sjvans sjvans changed the title fix: running together with xotel agent test: running together with xotel agent Nov 13, 2025
@sjvans sjvans marked this pull request as ready for review November 13, 2025 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants