impl(bigtable): update observability integration test to test all metric types - #16363
impl(bigtable): update observability integration test to test all metric types#16363scotthart wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the observability integration tests to support both static and dynamic channel pools, redirects Cloud Monitoring metric exports to a local OpenTelemetry collector, and introduces a new test case VerifyOutstandingRpcsMetric for dynamic pools. It also refactors the metric verification logic using custom GTest matchers for improved readability. The feedback suggests replacing absl::string_view with std::string_view in two locations to comply with the repository style guide.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #16363 +/- ##
==========================================
- Coverage 92.23% 92.23% -0.01%
==========================================
Files 2237 2237
Lines 210251 210269 +18
==========================================
+ Hits 193928 193934 +6
- Misses 16323 16335 +12 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
No description provided.