Skip to content

Commit

Permalink
[metrics] Remove (now) unnecessary metric size check.
Browse files Browse the repository at this point in the history
  • Loading branch information
manugarg committed Aug 8, 2023
1 parent 33b78ba commit 0a2ab16
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions prober/prober.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ import (
"github.com/cloudprober/cloudprober/targets"
"github.com/cloudprober/cloudprober/targets/endpoint"
"github.com/cloudprober/cloudprober/targets/lameduck"
"github.com/golang/glog"
"google.golang.org/grpc/codes"
"google.golang.org/grpc/status"
)
Expand Down Expand Up @@ -212,11 +211,6 @@ func (pr *Prober) Start(ctx context.Context) {
var em *metrics.EventMetrics
for {
em = <-pr.dataChan
var s = em.String()
if len(s) > logger.MaxLogEntrySize {
glog.Warningf("Metric entry for timestamp %v dropped due to large size: %d", em.Timestamp, len(s))
continue
}

// Replicate the surfacer message to every surfacer we have
// registered. Note that s.Write() is expected to be
Expand Down

0 comments on commit 0a2ab16

Please sign in to comment.