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

resourcemanager: move off the dependency of opencensus-go #10240

Closed
anderson-comvex opened this issue May 21, 2024 · 1 comment
Closed

resourcemanager: move off the dependency of opencensus-go #10240

anderson-comvex opened this issue May 21, 2024 · 1 comment
Labels
api: cloudresourcemanager Issues related to the Resource Manager API. triage me I really want to be triaged.

Comments

@anderson-comvex
Copy link

anderson-comvex commented May 21, 2024

Is your feature request related to a problem? Please describe.

It would prevent adding technical debt and temporary workarounds such as goroutine leak since the fix has been applied already on opentelemetry.

Describe the solution you'd like

Remove the outdated dependency of OpenCensus that have been sunset and move to OpenTelemetry. You can Read the full blog post to learn more.

Describe alternatives you've considered

  • Forking the repository and maintaining its own fork.
  • Added the temporary workaround

Additional context

There are several issues that have been fixed already on the OpenTelemetry library already that requires workarounds on OpenCensus, the ones that we need is specifically small but others could benefit from the maintained library instead.

@anderson-comvex anderson-comvex added the triage me I really want to be triaged. label May 21, 2024
@product-auto-label product-auto-label bot added the api: cloudresourcemanager Issues related to the Resource Manager API. label May 21, 2024
@anderson-comvex anderson-comvex changed the title resourcemanager: move off opencensus-go resourcemanager: move off the dependency of opencensus-go May 21, 2024
@codyoss
Copy link
Member

codyoss commented May 21, 2024

Our plan to remove the dependency is outlined here: https://github.com/googleapis/google-cloud-go/blob/main/debug.md#opencensus Let me know if you have any further questions not covered there.

@codyoss codyoss closed this as completed May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: cloudresourcemanager Issues related to the Resource Manager API. triage me I really want to be triaged.
Projects
None yet
Development

No branches or pull requests

2 participants