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.

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

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

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
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 <223565+codeboten@users.noreply.github.com>
codeboten added a commit to codeboten/opentelemetry-collector that referenced this issue Jun 17, 2024
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 <223565+codeboten@users.noreply.github.com>
@codeboten codeboten added the collector-telemetry healthchecker and other telemetry collection issues label Jun 17, 2024
@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
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 <223565+codeboten@users.noreply.github.com>
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
codeboten added a commit that referenced this issue Jul 9, 2024
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 <223565+codeboten@users.noreply.github.com>
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
codeboten added a commit that referenced this issue Jul 31, 2024
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 <223565+codeboten@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this from Blocked to Done in Collector: v1 Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
collector-telemetry healthchecker and other telemetry collection issues
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants