Conversation
|
@Levarix would you be willing to test once the binaries are built? |
Looks like new artifacts were built from this PR. Get them here!Artifact URLs |
(cherry picked from commit e0ac507)
|
@akrantz01 I've tested this, and now we have another huge problem. I applied a new version of telegraf in a production environment and telegraf processes consumes a lot of memory and don't purge it. Memory usage is still increasing until instance stopped work. It happens only with telegraf with couchbase plugin
|
|
@Levarix can you please open an issue for this? @akrantz01 the problem might be in func (cb *Couchbase) queryDetailedBucketStats(server, bucket string, bucketStats *BucketStats) error {
// Set up an HTTP request to get the complete set of bucket stats.
req, err := http.NewRequest("GET", server+"/pools/default/buckets/"+bucket+"/stats?", nil)
if err != nil {
return err
}
r, err := client.Do(req)
if err != nil {
return err
}
defer r.Body.Close()
return json.NewDecoder(r.Body).Decode(bucketStats)
}as the final |


Required for all PRs:
resolves #9416
Fixed an out-of-bounds panic that occurred when one of the bucket stats fields was missing in the response from Couchbase. A check to see if the array is
nilwas added to all bucket stats fields.