Skip to content

Telemetry-step-03: add full support for distributed tracing #1185

Nicolas Leclercq requested to merge trace_context_propagation#1185 into main

This is a quite big MR. The added code must be seen as a single functional block. Splitting it into smaller ones makes no sense and would not be satisfying in terms of code coverage. This MR will be presented/commented/reviewed in a dedicated meeting.

This is part of #1185.

  • Rebase, fix conflicts
  • make it compile and passing test
  • rebase
  • add catch2 test
    • using ostream exporter and check that kernel traces were outputted
    • add DS with tracing and check
  • Issue for kernel_traces_enabled doing nothing. kernel tracing is already used in pytango.
  • upstream issue for opentelemetry-cpp that ostream exporter with batch processing produces garbage logs for strings, https://github.com/open-telemetry/opentelemetry-cpp/issues/2651
  • Resolve WIP commit for CI
  • Get it compiled on windows
  • Cleanup commits and fix conflicts
  • Feedback from Anton if it does not crash anymore

Preliminary documentation can be found at tango-doc#403.

Edited by Thomas Braun

Merge request reports