Skip to content

[OTAGENT-510] remove deprecated otel routing processor#40231

Merged
dd-mergequeue[bot] merged 3 commits into
mainfrom
yang.song/OTAGENT-510-2
Aug 26, 2025
Merged

[OTAGENT-510] remove deprecated otel routing processor#40231
dd-mergequeue[bot] merged 3 commits into
mainfrom
yang.song/OTAGENT-510-2

Conversation

@songy23
Copy link
Copy Markdown
Member

@songy23 songy23 commented Aug 25, 2025

What does this PR do?

remove deprecated otel routing processor

Motivation

The routing processor has been deprecated and removed in upstream: open-telemetry/opentelemetry-collector-contrib#42154

@songy23 songy23 added this to the 7.71.0 milestone Aug 25, 2025
@songy23 songy23 requested a review from a team as a code owner August 25, 2025 22:09
@songy23 songy23 added the qa/done QA done before merge and regressions are covered by tests label Aug 25, 2025
@songy23 songy23 requested a review from a team as a code owner August 25, 2025 22:09
@songy23 songy23 requested a review from mackjmr August 25, 2025 22:09
@github-actions github-actions Bot added the medium review PR review might take time label Aug 25, 2025
@cit-pr-commenter
Copy link
Copy Markdown

Go Package Import Differences

Baseline: b6d5666
Comparison: 5e5576b

binaryosarchchange
otel-agentlinuxamd64
+0, -1
-github.com/open-telemetry/opentelemetry-collector-contrib/processor/routingprocessor
otel-agentlinuxarm64
+0, -1
-github.com/open-telemetry/opentelemetry-collector-contrib/processor/routingprocessor

@agent-platform-auto-pr
Copy link
Copy Markdown
Contributor

Static quality checks

✅ Please find below the results from static quality gates
Comparison made with ancestor 63ea6fe

Successful checks

Info

Quality gate Delta On disk size (MiB) Delta On wire size (MiB)
agent_deb_amd64 $${-0.89}$$ $${700.24}$$ < $${704.84}$$ $${-0.15}$$ $${177.12}$$ < $${178.58}$$
agent_deb_amd64_fips $${-0.89}$$ $${687.53}$$ < $${703.09}$$ $${-0.13}$$ $${174.01}$$ < $${178.12}$$
agent_heroku_amd64 $${-0.22}$$ $${350.3}$$ < $${355.37}$$ $${-0.12}$$ $${93.96}$$ < $${95.72}$$
agent_msi $${-1.6}$$ $${972.75}$$ < $${980.61}$$ $${-0.28}$$ $${150.11}$$ < $${152.67}$$
agent_rpm_amd64 $${-0.89}$$ $${700.23}$$ < $${704.83}$$ $${-0.42}$$ $${179.02}$$ < $${180.22}$$
agent_rpm_amd64_fips $${-0.89}$$ $${687.52}$$ < $${703.08}$$ $${-0.04}$$ $${175.67}$$ < $${179.85}$$
agent_rpm_arm64 $${-0.9}$$ $${689.16}$$ < $${694.74}$$ $${-0.13}$$ $${161.63}$$ < $${163.96}$$
agent_rpm_arm64_fips $${-0.91}$$ $${677.26}$$ < $${693.05}$$ $${-0.07}$$ $${158.43}$$ < $${163.0}$$
agent_suse_amd64 $${-0.89}$$ $${700.23}$$ < $${704.83}$$ $${-0.42}$$ $${179.02}$$ < $${180.22}$$
agent_suse_amd64_fips $${-0.89}$$ $${687.52}$$ < $${703.08}$$ $${-0.04}$$ $${175.67}$$ < $${179.85}$$
agent_suse_arm64 $${-0.9}$$ $${689.16}$$ < $${694.74}$$ $${-0.13}$$ $${161.63}$$ < $${163.96}$$
agent_suse_arm64_fips $${-0.91}$$ $${677.26}$$ < $${693.05}$$ $${-0.07}$$ $${158.43}$$ < $${163.0}$$
docker_agent_amd64 $${-0.89}$$ $${782.65}$$ < $${788.65}$$ $${-0.22}$$ $${269.85}$$ < $${272.01}$$
docker_agent_arm64 $${-0.9}$$ $${794.91}$$ < $${802.0}$$ $${-0.22}$$ $${256.63}$$ < $${259.7}$$
docker_agent_jmx_amd64 $${-0.89}$$ $${973.52}$$ < $${979.84}$$ $${-0.21}$$ $${338.48}$$ < $${340.95}$$
docker_agent_jmx_arm64 $${-0.9}$$ $${974.38}$$ < $${981.8}$$ $${-0.19}$$ $${321.24}$$ < $${324.65}$$
docker_cluster_agent_amd64 $${+0.01}$$ $${211.9}$$ < $${214.5}$$ $${-0}$$ $${72.05}$$ < $${73.51}$$
docker_cluster_agent_arm64 $${+0.06}$$ $${227.93}$$ < $${230.33}$$ $${+0.01}$$ $${68.35}$$ < $${69.77}$$
docker_cws_instrumentation_amd64 $${0}$$ $${7.07}$$ < $${7.12}$$ $${-0}$$ $${2.95}$$ < $${3.29}$$
docker_cws_instrumentation_arm64 $${0}$$ $${6.69}$$ < $${6.92}$$ $${+0}$$ $${2.7}$$ < $${3.07}$$
docker_dogstatsd_amd64 $${-0.11}$$ $${38.48}$$ < $${39.57}$$ $${-0.03}$$ $${14.89}$$ < $${15.76}$$
docker_dogstatsd_arm64 $${-0.14}$$ $${37.16}$$ < $${38.2}$$ $${-0.02}$$ $${14.34}$$ < $${14.83}$$
dogstatsd_deb_amd64 $${-0.12}$$ $${29.71}$$ < $${31.4}$$ $${-0.02}$$ $${7.85}$$ < $${8.95}$$
dogstatsd_deb_arm64 $${-0.11}$$ $${28.3}$$ < $${29.97}$$ $${-0.02}$$ $${6.81}$$ < $${7.89}$$
dogstatsd_rpm_amd64 $${-0.12}$$ $${29.71}$$ < $${31.4}$$ $${-0.02}$$ $${7.86}$$ < $${8.96}$$
dogstatsd_suse_amd64 $${-0.12}$$ $${29.71}$$ < $${31.4}$$ $${-0.02}$$ $${7.86}$$ < $${8.96}$$
iot_agent_deb_amd64 $${-0.11}$$ $${54.13}$$ < $${54.55}$$ $${-0.02}$$ $${13.65}$$ < $${14.45}$$
iot_agent_deb_arm64 $${-0.11}$$ $${51.45}$$ < $${51.9}$$ $${-0.02}$$ $${11.81}$$ < $${12.63}$$
iot_agent_deb_armhf $${-0.11}$$ $${51.0}$$ < $${51.42}$$ $${-0.01}$$ $${11.88}$$ < $${12.74}$$
iot_agent_rpm_amd64 $${-0.11}$$ $${54.13}$$ < $${54.55}$$ $${-0.02}$$ $${13.67}$$ < $${14.47}$$
iot_agent_suse_amd64 $${-0.11}$$ $${54.13}$$ < $${54.55}$$ $${-0.02}$$ $${13.67}$$ < $${14.47}$$

@songy23 songy23 removed the request for review from mackjmr August 26, 2025 13:31
@songy23 songy23 added the ask-review Ask required teams to review this PR label Aug 26, 2025
@songy23
Copy link
Copy Markdown
Member Author

songy23 commented Aug 26, 2025

/merge

@dd-devflow-routing-codex
Copy link
Copy Markdown

dd-devflow-routing-codex Bot commented Aug 26, 2025

View all feedbacks in Devflow UI.

2025-08-26 15:02:17 UTC ℹ️ Start processing command /merge


2025-08-26 15:02:36 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2025-08-26 17:02:16 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 37m (p90).


2025-08-26 17:40:08 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue Bot merged commit 0173a97 into main Aug 26, 2025
303 checks passed
@dd-mergequeue dd-mergequeue Bot deleted the yang.song/OTAGENT-510-2 branch August 26, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ask-review Ask required teams to review this PR medium review PR review might take time qa/done QA done before merge and regressions are covered by tests team/opentelemetry-agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants