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

Rethink how GCS+gRPC is built #14256

Open
dbolduc opened this issue May 17, 2024 · 0 comments
Open

Rethink how GCS+gRPC is built #14256

dbolduc opened this issue May 17, 2024 · 0 comments
Labels
api: storage Issues related to the Cloud Storage API. type: cleanup An internal cleanup or hygiene concern.

Comments

@dbolduc
Copy link
Member

dbolduc commented May 17, 2024

We are taking a hard dependency on opentelemetry-cpp and a soft dependency on gRPC's OTel plugin, even though both are required for metrics. It seems like we would want either both deps to be optional, or neither to be optional.

CMake and Bazel are also out of step.

We need to make progress though, so let's come back to this decision later.

@dbolduc dbolduc added api: storage Issues related to the Cloud Storage API. type: cleanup An internal cleanup or hygiene concern. labels May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the Cloud Storage API. type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Development

No branches or pull requests

1 participant