Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Move tracing to leverage opencensus #1352

Closed
3 tasks
mcastelino opened this issue Mar 11, 2019 · 6 comments
Closed
3 tasks

Move tracing to leverage opencensus #1352

mcastelino opened this issue Mar 11, 2019 · 6 comments
Labels
area/tracing Tracing enhancement Improvement to an existing feature high-severity Very important issue

Comments

@mcastelino
Copy link
Contributor

mcastelino commented Mar 11, 2019

Move tracing to leverage opencensus

The current tracing implementation uses opentracing/Jaeger. Moving the implementation to use opencensus and still exporting to Jaeger will allow us to have flexibility to

  • Support alternative backends
  • Eliminate the vsock workaround using a custom exporter
  • Add support for metrics using opencensus if needed
  • Enable tracing even when kata-agent is running as agent/initrd.

Some early PoC code that shows how to create a custom exporter
https://github.com/mcastelino/custom_exporter

Phasing

The work will need to be done in three phases

  • 1. Move from opentracing/Jaeger specific APIs to opencensus with the default Jaeger exporter
    This will still need to current vsock scaffolding but does not change any functionality. So externally nothing changes with this switch.

  • 2. Switch from the default opencensus Jaeger exporter to the custom exporter
    This will remove the need for all the vsock related customizations on the VM rootfs.

  • 3. Add the host side exporter service into the kata-shim (for v2). This will remove the need to run any logic on the host for tracing.

@mcastelino
Copy link
Contributor Author

/cc @egernst @jodh-intel @sboeuf

@egernst egernst added the enhancement Improvement to an existing feature label Mar 11, 2019
@jodh-intel
Copy link
Contributor

@jodh-intel
Copy link
Contributor

Hi @mcastelino - do you think you'll have time to look at this or do we need to find someone else to look at it?

@jodh-intel jodh-intel added the high-severity Very important issue label Jun 27, 2019
@jodh-intel
Copy link
Contributor

Current agent tracing is too awkward to setup so this will help simplify the process.

@jodh-intel jodh-intel added the area/tracing Tracing label Jun 27, 2019
@egernst
Copy link
Member

egernst commented Aug 7, 2019

@grahamwhaley
Copy link
Contributor

Just dropping in a link to https://opencensus.io/ for future reference.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/tracing Tracing enhancement Improvement to an existing feature high-severity Very important issue
Projects
None yet
Development

No branches or pull requests

4 participants