Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changed lambda expiration to quarterly #1802

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

pmillett
Copy link
Contributor

@pmillett pmillett commented Dec 3, 2024

What is the problem the customer is trying to solve?

There are many functions that invoke infrequently. They expect the associated entities to stick around in New Relic for at least as long as APM entities
, which is quarterly.

Why is it important for users that we fix that problem?

Lambda function entities that get deleted after 24 hours
lose tags and other entity-related data. It makes alerting on these entities difficult/impossible.

What is the objective they are pursuing?

Make infrequently invoked functions more useable in New Relic.

How does the ideal solution look for them?

Change the entityExpirationTime for Lambda functions to the same as APM entities (quarterly).

Related to this issue
#1792

Checklist

  • I've read the guidelines and understand the acceptance criteria.
  • The value of the attribute marked as identifier will be unique and valid.
  • I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

@CLAassistant
Copy link

CLAassistant commented Dec 3, 2024

CLA assistant check
All committers have signed the CLA.

@vlopeziglesias vlopeziglesias force-pushed the lambda-expirationExtension branch from e0bb242 to e40453f Compare December 11, 2024 18:27
@vlopeziglesias vlopeziglesias force-pushed the lambda-expirationExtension branch from e40453f to a689035 Compare December 12, 2024 10:04
@vlopeziglesias vlopeziglesias force-pushed the lambda-expirationExtension branch from a689035 to 5cd48bf Compare December 12, 2024 10:28
@vlopeziglesias vlopeziglesias merged commit b287e63 into newrelic:main Dec 12, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants