tree 35cd8c86809f38b8f69cd091cf43ac5be83c8d0c
parent eee5386be31560f3c875645222cb791d36237364
author Olav Loite <koloite@gmail.com> 1588314873 +0200
committer Knut Olav Løite <koloite@gmail.com> 1588613229 +0000

spanner: clean up test instances

The TestIntegration_StartBackupOperation (often) fails with a
RESOURCE_EXHAUSTED error caused by the backend having too many live
instances. This seems to be caused by the fact that the test is in a
different package than the other integration tests for Spanner, which
again means that the integration tests are executed in parallel with
each other. As each integration test run initializes its own Spanner
instance, this can cause RESOURCE_EXHAUSTED errors on the backend as
the test project does not support multiple Spanner instances.

This change also adds some additional cleanup code for test instances
and backups to make sure that everything is cleaned up, also when
unexpected errors occur.

Fixes #1957 and #1958

Change-Id: I61622c161261e808c178f5c1d99ba382b9d882e6
Reviewed-on: https://code-review.googlesource.com/c/gocloud/+/55590
Reviewed-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Hengfeng Li <hengfeng@google.com>
Reviewed-by: Cody Oss <codyoss@google.com>