[mergify]

Motivation/summary

commit attempts a fix on TestApprovedMetrics by filtering out server aggregates to focus on agent metrics.

Checklist

• Not a user facing change

For functional changes, consider:

• Is it observable through the addition of either logging or metrics?
• Is its use being published in telemetry to enable product improvement?
• Have system tests been added to avoid regression?

How to test these changes

Related issues

fixes: #21608

---

This is an automatic backport of pull request #22390 done by [Mergify](https://mergify.com).

[github-actions]

🤖 GitHub comments

Just comment with:

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

[Copilot]

🟢 Approval recommended

There are no active review issues for the backport targeting flaky metric approval checks.

No open issues 🎉

What changed in this PR

Backports #22390 to 9.5 to address flaky TestApprovedMetrics results reported in #21608, keeping approval checks focused on agent metrics rather than server-generated aggregates.

Changes:

• Excludes metrics with metricset.interval from the approval query, covering all server aggregate types.
• Removes a stray trailing backtick from the metricset intake fixture.

File	Description
testdata/​intake-v2/​metricsets.ndjson	Removes an invalid trailing character from the final metricset record.
systemtest/​metrics_test.go	Broadens aggregate filtering and updates the explanatory comment.

_{Review effort: Balanced}

[infra-vault-gh-plugin-prod]

💚 Build Succeeded

• Buildkite Build
• Commit: ad90eec

cc @aelnahas

[github-actions]

Dependency review finished without posting an analysis comment.

If you expected a review, push a new commit to this pull request (or close and reopen it) to try again.

Workflow run: https://github.com/elastic/apm-server/actions/runs/42227617001

[mergify]

✅ Merged into 9.5 as 2f4c868 at 16:43 UTC

10 min in the queue, 8 min of it CI.

So far

• ✅ Entered the queue Fri 6 Nov, 16:32 · rule default · triggered by rule squash and merge backport PRs after CI passes
• ✅ Passed · draft merge queue: merged #22396: [9.5] (backport #22390) fix flaky test TestApprovedMetrics #22404
• ✅ Merged 16:43 · 2f4c868 · squash

9 merge conditions met

• check-success=CLA
• check-success=lint
• check-success=system-test
• check-success=test (macos-latest)
• github-review-approved · required by GitHub repository ruleset rule 9.*, GitHub repository ruleset rule [org] Require a PR for Renovate and GitHub repository ruleset rule [org] Require a PR
• any of:
  • check-success=test (windows-2025)
  • check-success=test (windows-latest)
• any of · required by GitHub repository ruleset rule 9.*:
  • check-success = lint
  • check-neutral = lint
  • check-skipped = lint
• any of · required by GitHub repository ruleset rule 9.*:
  • check-success = buildkite/docs-build-pr
  • check-neutral = buildkite/docs-build-pr
  • check-skipped = buildkite/docs-build-pr
• any of · required by GitHub repository ruleset rule 9.*:
  • check-success = CLA
  • check-neutral = CLA
  • check-skipped = CLA

_{Times in UTC · dashboard · how the merge queue works}