tests: remove int64 field test#177
Conversation
tseaver
left a comment
There was a problem hiding this comment.
Usually, rather than removing a flaky system test, we work to un-flake it (e.g., via retries / backoff). Is there an issue open for the flakiness? Does it fail under CI, or just locally (I can see that the image file might not be present on a developer's system).
There is a GCP limitation - we can't create more than 6 images per hour per GCP project. Integration tests use the same project across all languages so the usage might be pretty high. |
|
@georgiyekkert To my knowledge, integration tests do not share projects across languages (because many APIs have rate limits / quotas which would trigger such issues). However, in a period of rapid development, there might be multiple PRs within a single language active, causing system tests to fail due to such limits. That comment aside, the PR should be OK to go forward if the test is truly not needed. |
the test potentially brings flakiness