googleapis / googleapis/google-cloud-cpp
GCS+gRPC integration fails
- Dominant language
- C++
- Stars
- 659
- Forks
- 463
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 89
Description
This failure indicates the test used more memory than expected:
```
google/cloud/storage/tests/object_read_large_integration_test.cc:87: Failure
Expected: (current_rss) <= (tolerance), actual: 492998656 vs 492855296
```
* GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/fe4be0b0-03fc-47e1-82ab-d05968016842;tab=detail?project=cloud-cpp-testing-resources
* Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/6505814e75b598e4e2f492f9bbe2c827942143bf/fedora-latest-bazel-asan/log-fe4be0b0-03fc-47e1-82ab-d05968016842.txt
It could be because the test needs adjustment for gRPC, or because there is a real leak somewhere. I suspect the first, since this is a flake.
Contributor guide
Research direction
Start with google/cloud/storage/tests/object_read_large_integration_test.cc at line 87 and reproduce the Fedora Bazel ASAN failure using the linked raw log. Determine whether the gRPC integration needs a tolerance adjustment or whether the excess memory indicates a leak; done means the cause is addressed and the integration test passes reliably.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, google-cloud, grpc
- Domain
- cloud, testing-qa
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100