elastic / apm

Elastic Application Performance Monitoring - resources and general issue tracking for Elastic APM.
https://www.elastic.co/apm
Apache License 2.0
384 stars 114 forks source link

Minimum delay for retries when central config cannot be retrieved #667

Closed jaggederest closed 2 years ago

jaggederest commented 2 years ago

In the Ruby Agent we had a report of unbounded memory growth when the central configuration server (APM Server) is offline and behind a reverse proxy which does not return empty payloads.

This behavior would be avoided if there were a minimum retry time and rate, per the spec change.

As these changes are optional and suggested, I opted for the shorter template without a meta issue.

/schedule 2022-09-21

apmmachine commented 2 years ago

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

#### Build stats * Start Time: 2022-09-19T04:24:00.116+0000 * Duration: 3 min 12 sec

jaggederest commented 2 years ago

Also, I seem to remember us discussing whether this needs to use the full PR process - if that's the case should I reopen a new PR for this branch using the template?

felixbarny commented 2 years ago

No, just replace the PR description with the checkboxes of the full process.