search

newrelic / infra-integrations-sdk

New Relic Infrastructure Integrations SDK
Apache License 2.0
44 stars 23 forks source link

Make timeout configurable (NR-281994) #315

Open kang-makes opened 1 week ago

kang-makes commented 1 week ago

When users increase the interval to (for example) 5 minutes, our samples appear broken because the cache TTL is set by default to 1 minute making RATES and DELTAS to report only 0’s / nulls.

This PR set TTL as a variable and rise the default to 6 minutes (5 minutes plus a sane amount of time in case the integrations takes too long to execute)

sigilioso commented 3 days ago

This PR set TTL as a variable and rise the default to 6 minutes (5 minutes plus a sane amount of time in case the integrations takes too long to execute)

I think it is already planned but this change should be followed by an update in the docs to state what is the maximum supported interval value.