ci(bigtable): add fake monitoring server and metric test that uses it#16266
ci(bigtable): add fake monitoring server and metric test that uses it#16266scotthart wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces an Observability Collector Server within the CI infrastructure to support end-to-end integration testing of OpenTelemetry metrics for Google Cloud C++ client libraries, starting with Bigtable. The changes include the server implementation, protobuf definitions for observability verification, and integration tests. My feedback includes a recommendation to refactor duplicated code in the server implementation to improve maintainability.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #16266 +/- ##
=======================================
Coverage 92.27% 92.27%
=======================================
Files 2214 2214
Lines 206505 206537 +32
=======================================
+ Hits 190549 190579 +30
- Misses 15956 15958 +2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
c8590cf to
876f2a8
Compare
No description provided.