Skip to content

Commit

Permalink
chore(logcache fetcher): Don't print full logcache REST API results
Browse files Browse the repository at this point in the history
# Issue

Some `Info` log statements were filling up our log files.

# Fix

Don't print all envelopes, just how many were received.
  • Loading branch information
silvestre authored Nov 20, 2024
1 parent 94c2dd3 commit 1893054
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/autoscaler/eventgenerator/metric/fetcher.go
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ func (l *logCacheFetcher) getMetricsRestAPI(appId string, metricType string, sta
if err != nil {
return []models.AppInstanceMetric{}, fmt.Errorf("fail to Read %s metric from %s GoLogCache client: %w", logcache_v1.EnvelopeType_GAUGE, appId, err)
}
l.logger.Info("received-rest-api-result", lager.Data{"envelopes": envelopes})
l.logger.Info("received-rest-api-result", lager.Data{"numEnvelopes": len(envelopes)})

metrics := l.envelopeProcessor.GetGaugeMetrics(envelopes, time.Now().UnixNano())

Expand Down

0 comments on commit 1893054

Please sign in to comment.