feat(storage): support DirectPath over Interconnect in GCS gRPC - #16408
feat(storage): support DirectPath over Interconnect in GCS gRPC#16408kalragauri wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces the DirectPathXdsOverInterconnectOption and the corresponding environment variable GOOGLE_CLOUD_ENABLE_DIRECT_PATH_XDS_OVER_INTERCONNECT to allow configuring DirectPath over Interconnect. When enabled, the client targets google-c2p:///storage-direct.googleapis.com?force-xds with standard TLS. Comprehensive unit tests have been added to cover various scenarios and overrides. Feedback is provided regarding a style guide violation where auto is used for a boolean type instead of an explicit bool const declaration.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #16408 +/- ##
==========================================
+ Coverage 92.25% 92.26% +0.01%
==========================================
Files 2246 2246
Lines 212291 212416 +125
==========================================
+ Hits 195841 195995 +154
+ Misses 16450 16421 -29 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This PR adds support for DirectPath over Interconnect in the Cloud Storage gRPC client library.
Summary of Changes
storage_experimental::DirectPathXdsOverInterconnectOptionand support for theGOOGLE_CLOUD_ENABLE_DIRECT_PATH_XDS_OVER_INTERCONNECTenvironment variable.storage_internal::DefaultOptionsGrpcto resolve target endpointgoogle-c2p:///storage-direct.googleapis.com?force-xdswith authoritystorage.googleapis.comwhen enabled, bypassing GCE environment checks.