diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2025-04-22 17:01:31 +0200 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2025-04-23 09:01:11 +0000 |
| commit | 6eb12242337d83872574660e27cfe2e8db463e48 (patch) | |
| tree | 6ac911128f0e94e5784c068cbca88b6b21917fcb | |
| parent | 57d54c0889fd10499d368f0132c75567b0a28f57 (diff) | |
pkg/manager: log no longer needed reproductions
We already mention the scheduled and started reproductions, but if the
reproduction was not started intentionally, that makes the logs look
very confusing.
Print a log message if NeedRepro() returned false when we were to
actually start the reproduction.
| -rw-r--r-- | pkg/manager/repro.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/pkg/manager/repro.go b/pkg/manager/repro.go index c9eb2f9a9..f9fb2d0e2 100644 --- a/pkg/manager/repro.go +++ b/pkg/manager/repro.go @@ -187,6 +187,7 @@ func (r *ReproLoop) Loop(ctx context.Context) { crash := r.popCrash() for { if crash != nil && !r.mgr.NeedRepro(crash) { + log.Logf(1, "reproduction of %q aborted: it's no longer needed", crash.FullTitle()) crash = nil // Now we might not need that many VMs. r.mu.Lock() |