Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove opencensus bridge #10414

Closed
codeboten opened this issue Jun 17, 2024 · 0 comments · Fixed by #10406
Closed

Remove opencensus bridge #10414

codeboten opened this issue Jun 17, 2024 · 0 comments · Fixed by #10406
Labels
collector-telemetry healthchecker and other telemetry collection issues
Milestone
Self observability

Comments

@codeboten
Copy link
Contributor

This will enable us to close the loop on many goleak exceptions that are in place today. Additionally it will allow us to move the meter provider configuration code to use the otel-go config package.

Removing the bridge requires the completion of the migration away from opencensus in the contrib repo.
codeboten added a commit to codeboten/opentelemetry-collector that referenced this issue Jun 17, 2024
@codeboten
[service] add internal factory method for meter provider
6de9b11 
This will be used when moving the meter provider configuration to use the otel-go config package.

Part of open-telemetry#10414

Signed-off-by: Alex Boten <[email protected]>
codeboten added a commit to codeboten/opentelemetry-collector that referenced this issue Jun 17, 2024
@codeboten
[service] add internal factory method for meter provider
b0e100d 
This will be used when moving the meter provider configuration to use the otel-go config package.

Part of open-telemetry#10414

Signed-off-by: Alex Boten <[email protected]>
@codeboten codeboten mentioned this issue Jun 17, 2024
Merged
@codeboten codeboten added the collector-telemetry healthchecker and other telemetry collection issues label Jun 17, 2024
@codeboten codeboten mentioned this issue Jun 17, 2024
Merged
@codeboten codeboten mentioned this issue Jul 4, 2024
Merged
@codeboten codeboten moved this from Todo to Blocked in Collector: v1 Jul 4, 2024
codeboten added a commit that referenced this issue Jul 5, 2024
@codeboten @mx-psi
[service] add internal factory method for meter provider (#10415)
6fca9ee 
This will be used when moving the meter provider configuration to use
the otel-go config package.

Part of #10414

Skipping changelog as there's no usage of this method yet. It will be
done in a separate PR

---------

Signed-off-by: Alex Boten <[email protected]>
Co-authored-by: Pablo Baeyens <[email protected]>
codeboten added a commit that referenced this issue Jul 9, 2024
@codeboten @mx-psi
[service] add service.disableOpenCensusBridge gate (#10542)
650ac0b 
This feature gate allows end users to re-enable the opencensus bridge if
there's a need for it. This preceeds #10406 and will be taken out of
draft once
open-telemetry/opentelemetry-collector-contrib#29867
is completed.

Related to
#10414

---------

Signed-off-by: Alex Boten <[email protected]>
Co-authored-by: Pablo Baeyens <[email protected]>
codeboten added a commit that referenced this issue Jul 31, 2024
@codeboten
remove opencensus bridge support (#10406)
eebb96b 
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
Closes #8945

This is blocked until
open-telemetry/opentelemetry-collector-contrib#29867
is completed.

---------

Signed-off-by: Alex Boten <[email protected]>
@github-project-automation github-project-automation bot moved this from Blocked to Done in Collector: v1 Jul 31, 2024
@whiskeysierra whiskeysierra mentioned this issue Sep 2, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
collector-telemetry healthchecker and other telemetry collection issues
Projects
Collector: v1
Archived in project
Milestone
Self observability
Development

Successfully merging a pull request may close this issue.

remove opencensus bridge support codeboten/opentelemetry-collector
2 participants
@codeboten @mx-psi