Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dricross/otel bump v0.115.0 #1485

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Conversation

dricross
Copy link
Contributor

@dricross dricross commented Jan 6, 2025

Description of the issue

We need to track and pull in changes from upstream periodically.

Major changes to lookout for:

  • receiver.CreateSettings renamed to receiver.Settings: Dricross/otel bump v0.115.0 #1485 (comment). Many changes like this
  • pipelines.Config now requires a pipelines.ID rather than component.ID
    • Change the Translator / TranslatorMap generic functions to also accept another type, ID
    • For component translators, this ID is component.ID. For pipeline translators, this ID is pipeline.ID.
    • Component translators are then the concrete type Translator[component.Config, component.ID], which mean “given a config, generate a component.Config which is identified by a component.ID”
    • Pipeline translators are the concrete type Translator[*ComponentTranslators, pipeline.ID], which means “given a config, generate a ComponentTranslator (a loose definition of an otel pipeline) which is identified by a pipeline.ID”
  • Upstream update to TracesConfig adds new field: open-telemetry/opentelemetry-collector@b33913b
    • Generated otel.Config now has a new field under Service.Telemetry.Level which is set to Basic by default
    • Add level: Basic to basically all the expected yaml files
  • Otel config decoder no longer supports "weakly typed" values by default
    • no longer accepts “true” for local_mode setting. Must be true
    • no longer accepts [] for external_labels setting. Must be {}
    • This is due to a upstream change: open-telemetry/opentelemetry-collector@e477c3a
    • Previously, the “weak typing” was enforced so the decoder would know to convert a string to a boolean. With “strong typing” enforced by default, the decoder will no longer accept it
  • New defaults for clientConfig
  • New defaults for server config (similar to clientConfig)
  • debugexporterhas new field: open-telemetry/opentelemetry-collector@f47b9f2
    • Setting to true maintains previous functionality
  • receiver/awscontainerinsightsreceiver/internal/host:

Description of changes

Merge OTel contrib v0.115.0/v1.21.0. Previous version was v0.103.0

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

Requirements

Before commit the code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make lint

@dricross dricross requested a review from a team as a code owner January 6, 2025 15:13
@dricross dricross force-pushed the dricross/otel-bump-v0.115.0 branch from 73b1131 to d61236c Compare January 9, 2025 15:56
@dricross dricross force-pushed the dricross/otel-bump-v0.115.0 branch from d61236c to f2ac15b Compare January 9, 2025 16:06
Copy link
Contributor

This PR was marked stale due to lack of activity.

@github-actions github-actions bot added the Stale label Jan 17, 2025
@github-actions github-actions bot removed the Stale label Jan 28, 2025
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.

1 participant