ClusterCockpit/cc-metric-collector
1
0
Fork 0
mirror of https://github.com/ClusterCockpit/cc-metric-collector.git synced 2024-12-25 23:19:06 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix: When sending metrics failed the batch size could be exceeded

Browse Source
This commit is contained in:
Holger Obermaier 2022-06-20 18:06:27 +02:00
parent b732b2d739
commit 580d21d8bb
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
sinks/influxSink.go
View File

@ -142,17 +142,20 @@ func (s *InfluxSink) Flush() error {
// Send metrics from batch slice // Send metrics from batch slice
err := s.writeApi.WritePoint(context.Background(), s.batch...) err := s.writeApi.WritePoint(context.Background(), s.batch...)
if err != nil {
cclog.ComponentError(s.name, "flush failed:", err.Error())
return err
}
// Clear batch slice // Clear batch slice
// (when sending the metrics failed, metrics get dropped and are lost)
for i := range s.batch { for i := range s.batch {
s.batch[i] = nil s.batch[i] = nil
} }
s.batch = s.batch[:0] s.batch = s.batch[:0]
// Report errors for sending metrics
if err != nil {
cclog.ComponentError(s.name, "flush failed:", err.Error())
return err
}
return nil return nil
} }