apm-server icon indicating copy to clipboard operation
apm-server copied to clipboard

tbs: Clean up metric registration in storage manager

Open isaacaflores2 opened this issue 10 months ago • 2 comments

Motivation/summary

I noticed I forgot to clean up the metric registration. It is no longer required since storage metrics are now synchronous.

How to test these changes

  1. Unit Test
  2. Calling the stats endpoint localhost:5066/stats

Related issues

Part of https://github.com/elastic/apm-server/issues/17104

isaacaflores2 avatar Jun 18 '25 16:06 isaacaflores2

:robot: GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

github-actions[bot] avatar Jun 18 '25 16:06 github-actions[bot]

This pull request does not have a backport label. Could you fix it @isaacaflores2? 🙏 To fixup this pull request, you need to add the backport labels for the needed branches, such as:

  • backport-7.17 is the label to automatically backport to the 7.17 branch.
  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit.
  • backport-9./d is the label to automatically backport to the 9./d branch. /d is the digit.
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

mergify[bot] avatar Jun 18 '25 16:06 mergify[bot]

@mergifyio backport 9.0

github-actions[bot] avatar Jun 19 '25 15:06 github-actions[bot]

backport 9.0

✅ Backports have been created

mergify[bot] avatar Jun 19 '25 15:06 mergify[bot]