Skip to content

Comments

Update otel config in DDOT installation guide#33934

Merged
buraizu merged 3 commits intomasterfrom
agagniere/update-otel-config
Jan 27, 2026
Merged

Update otel config in DDOT installation guide#33934
buraizu merged 3 commits intomasterfrom
agagniere/update-otel-config

Conversation

@agagniere
Copy link
Contributor

@agagniere agagniere commented Jan 16, 2026

What does this PR do? What is the motivation?

Update the OTel configuration snippets in the DDOT installation tutorial, to reflect the change in default config that has happened: the batch processor was removed in favor of using the exporter's sending queue's batching capabilities

OTAGENT-782

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

A few images, representing the pipelines that include the batch processor, will need to be updated as well

@agagniere agagniere self-assigned this Jan 16, 2026
@agagniere agagniere force-pushed the agagniere/update-otel-config branch from c5a8123 to f78be67 Compare January 16, 2026 16:56
@agagniere agagniere changed the title Update otel config in k8s setup tutorial Update otel config in DDOT installation guide Jan 16, 2026
@agagniere agagniere force-pushed the agagniere/update-otel-config branch from 37177c5 to fac316f Compare January 26, 2026 16:08
@github-actions github-actions bot added the Images Images are added/removed with this PR label Jan 26, 2026
@agagniere agagniere marked this pull request as ready for review January 26, 2026 16:35
@agagniere agagniere requested a review from a team as a code owner January 26, 2026 16:35
@buraizu buraizu self-assigned this Jan 26, 2026
Copy link
Contributor

@buraizu buraizu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, this looks good overall -- but unfortunately, replacing an image with one that has the same file name can cause caching issues. Would you be able to upload the new image with a new name, such as components-3, and likewise update the src attributes of the images?

Feel free to reach out to me internally if you'd like additional context on the caching issues introduced by this (or if you have any other questions/feedback).

@agagniere agagniere force-pushed the agagniere/update-otel-config branch from fac316f to ee2ba2e Compare January 27, 2026 10:51
@agagniere agagniere requested a review from buraizu January 27, 2026 12:42
@buraizu buraizu merged commit 95ac89a into master Jan 27, 2026
21 of 23 checks passed
@buraizu buraizu deleted the agagniere/update-otel-config branch January 27, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images Images are added/removed with this PR opentelemetry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants