opentelemetry-collector icon indicating copy to clipboard operation
opentelemetry-collector copied to clipboard

[extension/ballast] Remove the deprecated memory ballast extension

Open TylerHelmuth opened this issue 1 year ago • 2 comments

Description

This PR removes the deprecated memory ballast extension and all the logic in place in memorylimiter and service that was using it.

Link to tracking issue

Related to https://github.com/open-telemetry/opentelemetry-collector/issues/8343. I don't want to close it until the helm chart is updated.

Testing

Unit tests

TylerHelmuth avatar Jul 19 '24 21:07 TylerHelmuth

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.23%. Comparing base (48b11ba) to head (420efda). Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10671      +/-   ##
==========================================
+ Coverage   92.22%   92.23%   +0.01%     
==========================================
  Files         409      406       -3     
  Lines       19134    19101      -33     
==========================================
- Hits        17646    17618      -28     
+ Misses       1127     1123       -4     
+ Partials      361      360       -1     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jul 19 '24 22:07 codecov[bot]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

github-actions[bot] avatar Aug 07 '24 03:08 github-actions[bot]

So far I've only seen https://github.com/open-telemetry/opentelemetry-collector/issues/10995

TylerHelmuth avatar Aug 29 '24 16:08 TylerHelmuth