Only log the entire summary proto when confirm is false.
Currently we are spamming our logs with the full content of every dashboard's summary every update loop.
@@ -97,8 +97,8 @@ func Update(ctx context.Context, client gcs.Client, configPath gcs.Path, concurr
errCh <- errors.New(dash.Name)
continue
}
- log.WithField("summary", sum).Info("summarized")
if !confirm {
+ log.WithField("summary", sum).Info("Summarized")
continue
}
summaryPath, err := configPath.ResolveReference(&url.URL{Path: path.Join(summaryPathPrefix, summaryPath(dash.Name))})
@@ -112,6 +112,7 @@ func Update(ctx context.Context, client gcs.Client, configPath gcs.Path, concurr
errCh <- errors.New(dash.Name)
continue
}
+ log.Info("Wrote dashboard summary")
errCh <- nil
}
wg.Done()
