Skip to content

[CB] Remove OpenTelemetry#45984

Merged
Cyrilvallez merged 3 commits into
mainfrom
cb-remove-opentelemetry
May 19, 2026
Merged

[CB] Remove OpenTelemetry#45984
Cyrilvallez merged 3 commits into
mainfrom
cb-remove-opentelemetry

Conversation

@remi-or

@remi-or remi-or commented May 15, 2026

Copy link
Copy Markdown
Collaborator

This PR remove the code related to OpenTelemetry in continuous batching because:

  1. it has not been updated since the rework of CB started (~September 2025)
  2. it is not used by serve
  3. it is not used in CB

If telemetry is needed, it will be added back, after manager rework is done and TP is finalized, both of which will change the internal workings quite a bit.

Requires #45821 before merging

@remi-or remi-or requested a review from ArthurZucker May 15, 2026 01:57
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@remi-or remi-or force-pushed the cb-remove-opentelemetry branch from 739784d to 54d9c1e Compare May 18, 2026 07:05
@github-actions

Copy link
Copy Markdown
Contributor

View the CircleCI Test Summary for this PR:

https://huggingface.co/spaces/transformers-community/circle-ci-viz?pr=45984&sha=298503

@Cyrilvallez Cyrilvallez merged commit 3034a13 into main May 19, 2026
92 of 95 checks passed
@Cyrilvallez Cyrilvallez deleted the cb-remove-opentelemetry branch May 19, 2026 04:24
yuchenxie4645 pushed a commit to yuchenxie4645/transformers that referenced this pull request May 28, 2026
kashif pushed a commit to kashif/transformers that referenced this pull request Jun 1, 2026
khushali9 pushed a commit to khushali9/transformers that referenced this pull request Jun 8, 2026
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.

5 participants