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

remove opencensus bridge support

Open codeboten opened this issue 1 year ago • 1 comments
trafficstars

This removes the configuration of the OpenCensus bridge from the Collector. This means that any metric still relying on it will no longer be emitting metrics.

Closes #10414

This is blocked until https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/29867 is completed.

codeboten avatar Jun 13 '24 22:06 codeboten

Codecov Report

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

Project coverage is 92.08%. Comparing base (6d32c09) to head (8099032).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #10406   +/-   ##
=======================================
  Coverage   92.07%   92.08%           
=======================================
  Files         405      405           
  Lines       18862    18856    -6     
=======================================
- Hits        17367    17363    -4     
+ Misses       1135     1134    -1     
+ Partials      360      359    -1     

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

codecov[bot] avatar Jun 13 '24 22:06 codecov[bot]

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

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

not stale, still planning on moving this forward, just finishing up the blocking PR

codeboten avatar Jul 03 '24 15:07 codeboten

Do we still have components using opencensus? I see two of them are mine, but I can commit to getting them migrated before the new release in two weeks.

There are 4 as listed in https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/29867

codeboten avatar Jul 03 '24 15:07 codeboten

After discussing this further, i'll add a feature gate before this change.

codeboten avatar Jul 03 '24 15:07 codeboten

This will stay in draft until https://github.com/open-telemetry/opentelemetry-collector/pull/10542 has been merged and released.

codeboten avatar Jul 04 '24 19:07 codeboten