Skip to content

Conversation

@giautm
Copy link
Contributor

@giautm giautm commented Jun 18, 2025

This PR follows the PR #10757 to allow disabling gRPC DirectPath using the build tag disable_grpc_modules.

This PR is trying to resolve the same issue as mentioned in #12343 for Spanner and BigTable client.

Closes: #12488

@giautm giautm requested review from a team as code owners June 18, 2025 15:38
@giautm giautm changed the title g/envoy feat(spanner): allow disable gRPC PD deps Jun 18, 2025
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label Jun 19, 2025
@rahul2393 rahul2393 changed the title feat(spanner): allow disable gRPC PD deps feat(bigtable/spanner): allow disable gRPC PD deps Jun 19, 2025
@rahul2393
Copy link
Contributor

cc: @mohanli-ml @bhshkh

@rahul2393 rahul2393 requested review from bhshkh and mohanli-ml June 19, 2025 08:38
@rahul2393 rahul2393 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 19, 2025
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 19, 2025
@product-auto-label product-auto-label bot added api: bigtable Issues related to the Bigtable API. and removed api: spanner Issues related to the Spanner API. labels Jun 20, 2025
@giautm
Copy link
Contributor Author

giautm commented Jun 20, 2025

Hey, can we have a review soon?

bhshkh
bhshkh previously approved these changes Jun 23, 2025
@bhshkh bhshkh dismissed their stale review June 23, 2025 20:56

Need some changes

@bhshkh
Copy link
Contributor

bhshkh commented Jun 23, 2025

Please add description mentioning what is being done and why.

Create an issue here https://github.com/googleapis/google-cloud-go/issues to describe what is the problem being solved.

@giautm
Copy link
Contributor Author

giautm commented Jun 24, 2025

@bhshkh Created an issue, and updated the description.

@giautm giautm changed the title feat(bigtable/spanner): allow disable gRPC PD deps feat(bigtable/spanner): allow disable gRPC DirectPath deps Jun 24, 2025
@giautm
Copy link
Contributor Author

giautm commented Jun 26, 2025

@bhshkh ping, please help me to reviews and merge this PR.

@giautm
Copy link
Contributor Author

giautm commented Jul 1, 2025

Hello, please unblock me with this @rahul2393

@bhshkh bhshkh enabled auto-merge (squash) July 2, 2025 05:24
@bhshkh bhshkh merged commit 2205d27 into googleapis:main Jul 2, 2025
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spanner/bigtable: shipping extra 8MB with github.com/envoyproxy/go-control-plane/envoy

4 participants