Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

Update opentelemetry-collector-contrib #1815

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alejandrodnm
Copy link
Contributor

@alejandrodnm alejandrodnm commented Dec 23, 2022

Description

This version includes the changes to the Jaeger translator that support
multi ChildOf references to the links.

open-telemetry/opentelemetry-collector-contrib#14463

We used to do this before translating to OTEL and after translating from
OTEL, in order to keep this information and pass the Jaeger
certification tests.

There are some breaking changes from the opentelemetry-collector that were also addressed.

This PR updates to opentelemetry-collector 1.0-RC2, we might want to wait for the official release of 1.0 to update and merge.

Merge requirements

Please take into account the following non-code changes that you may need to make with your PR:

  • CHANGELOG entry for user-facing changes
  • Updated the relevant documentation

@alejandrodnm alejandrodnm force-pushed the adn/jaeger-reftypes branch 3 times, most recently from aecf5ad to b4d2036 Compare December 23, 2022 15:02
@alejandrodnm alejandrodnm force-pushed the adn/jaeger-reftypes branch 3 times, most recently from b4d2036 to d93fde7 Compare January 11, 2023 16:32
This version includes the changes to the Jaeger translator that support
multi ChildOf references to the links.

open-telemetry/opentelemetry-collector-contrib#14463

We used to do this before translating to OTEL and after translating from
OTEL, in order to keep this information and pass the Jaeger
certification tests.
@alejandrodnm alejandrodnm marked this pull request as ready for review January 11, 2023 16:43
@alejandrodnm alejandrodnm requested review from a team as code owners January 11, 2023 16:43
@alejandrodnm alejandrodnm requested review from a team and removed request for a team January 11, 2023 16:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant