-
Notifications
You must be signed in to change notification settings - Fork 515
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 dependency on github.com/census-instrumentation/opencensus-proto #1014
Comments
We cannot remove it until OpenCensus is deleted in Envoy core extensions. How does this cause issue for genproto? It's just another proto? |
@kyessenov it is related to the issue linked above. TLDR is we did a module split to limit the dependencies pulled into some projects that depend on genproto. But this can cause an ambiguous import if you have a really old ref to genproto these days -- such as demonstrated in this open census project.
Is this planned work, and is there an issue we can follow? |
envoyproxy/envoy#9958 is the issue tracking the replacement. |
This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions. |
can we add no stalebot to this. This remains a problem for for our Cloud SDK. |
Open Census is now deprecated and the project mentioned archive has been archived for over a year. I have seen some dependency issues popping up in the go/proto/grpc space and have traced it back to this dependency having a really old refs on google.golang.org/genproto in this repo. Is there a plan to remove this dependency from the tree? As another issue pointed out, which I have linked below, this can also pose a security risk to depend upon projects that are not getting updated.
Previous issue that was autoclosed: #952
The text was updated successfully, but these errors were encountered: