Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Spanner: Flaky integration tests #4166

Closed
chingor13 opened this issue Dec 3, 2018 · 5 comments
Closed

Spanner: Flaky integration tests #4166

chingor13 opened this issue Dec 3, 2018 · 5 comments
Assignees
@nithinsujir
Labels
api: spanner Issues related to the Spanner API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@chingor13
Copy link
Contributor

chingor13 commented Dec 3, 2018 •
edited
Loading

Spanner IT tests are occasionally failing with transaction aborted status

Example:
https://source.cloud.google.com/results/invocations/676bd73d-427e-48e7-b824-62b79e3288e0/targets/github%2Fgoogle-cloud-java%2Fgoogle-cloud-clients%2Fgoogle-cloud-spanner%2Ftarget%2Ffailsafe-reports/tests

Last 9 failures:
@chingor13
Copy link
Contributor Author

Anecdotally, it appears to happen when multiple test suites are running simultaneously against the same project.

@chingor13 chingor13 added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: spanner Issues related to the Spanner API. priority: p2 Moderately-important priority. Fix may not be included in next release. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. and removed priority: p2 Moderately-important priority. Fix may not be included in next release. labels Dec 4, 2018
@chingor13
Copy link
Contributor Author

We need to fix this before GA.

@chingor13
Copy link
Contributor Author

We also see WARNING: Leaked session during the tests as well.

@chingor13 chingor13 mentioned this issue Dec 4, 2018
Merged
@chingor13
Copy link
Contributor Author

The timeouts are being addressed in #4176

@nithinsujir nithinsujir mentioned this issue Dec 5, 2018
Merged
@chingor13
Copy link
Contributor Author

Spanner integration tests are looking fine after upping the timeout and handling the aborted exceptions. The only recent failures are due to maven networking issues.

@olavloite olavloite mentioned this issue Jul 31, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nithinsujir nithinsujir

Labels
api: spanner Issues related to the Spanner API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chingor13 @nithinsujir