Skip to content

fix: each metric should have an expire time#11838

Merged
Revolyssup merged 3 commits into
apache:masterfrom
Revolyssup:revolyssup/expiry-in-metric
Dec 17, 2024
Merged

fix: each metric should have an expire time#11838
Revolyssup merged 3 commits into
apache:masterfrom
Revolyssup:revolyssup/expiry-in-metric

Conversation

@Revolyssup

Copy link
Copy Markdown
Contributor

Description

Fixes # (issue)
The previous PR that implemented prometheus metrics expiry didn't meet the expectation and we need to ensure each metric has configurable expiry.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working labels Dec 17, 2024
@@ -0,0 +1,132 @@
#

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrong typo file name expiry

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@Revolyssup
Revolyssup merged commit de7bd0b into apache:master Dec 17, 2024
@Revolyssup
Revolyssup deleted the revolyssup/expiry-in-metric branch December 17, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants