diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2025-12-29 17:50:00 +0100 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2025-12-29 19:18:08 +0000 |
| commit | cfe5b65b5829e573b6c6fc73936be97921b69efb (patch) | |
| tree | 8a988a0d4232c640db56fa06ffa9c04c309571b1 /executor/executor.cc | |
| parent | 941d9afc6cb86dd16abf3fcaff41adb8c713febe (diff) | |
syz-cluster/pkg/db: fix bunch of bugs in emulator management
There is a bunch of bugs now:
1. The emulator binary is killed when the first test finished,
before subsequent tests start.
2. The child emulator binary (the actual emulator "emulator_main") is leaked.
These subprocesses are never killed and live past tests.
(that's why we get away with the first problem)
3. Errors are not handled well if emulator setup fails.
We leave spannerHost empty and subsequent tests ignore the original error
(since only the first test executes setupSpannerOnce).
4. NewTransientDB duplicates a bunch of work that needs to happen only once
(in particular os.Setenv("SPANNER_EMULATOR_HOST")).
Fix all of that.
Support spanner emulator distributed as part of google-cloud-sdk while we are here,
so that tests can be run locally.
Diffstat (limited to 'executor/executor.cc')
0 files changed, 0 insertions, 0 deletions
