FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix SigNoz cache span filtering by ejsmith · Pull Request #2479 · exceptionless/Exceptionless · GitHub

Fix SigNoz cache span filtering - #2479

Draft
ejsmith wants to merge 3 commits into
mainfrom
issue/fix-signoz-cache-span-filter
Draft

Fix SigNoz cache span filtering#2479
ejsmith wants to merge 3 commits into
mainfrom
issue/fix-signoz-cache-span-filter

Conversation

ejsmith commented Aug 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Drop successful Redis spans regardless of the exporting service name.
  • Match both db.system and the newer db.system.name semantic-convention attribute.
  • Preserve Redis spans whose status is STATUS_CODE_ERROR and retain the existing Message: EntityChanged filter.

Root cause

The cache-span filter was scoped to two fixed service.name values. A deployment renamed a service, so its successful Redis spans stopped matching the filter and were exported to SigNoz.

Validation

  • git diff --check
  • Rendered signoz/k8s-infra chart 0.16.0 with helm template and confirmed filter/drop_cache_spans is present in the node-agent traces pipeline after chart-injected processors and before batch.

Breaking changes

None.

ejsmith force-pushed the issue/fix-signoz-cache-span-filter branch from 197ce3a to 28a2bc8 Compare August 17, 2026 15:41

Copy link
Copy Markdown

Package Line Rate Branch Rate Complexity Health
Exceptionless.Insulation 37% 35% 286
Exceptionless.Core 75% 67% 10184
Exceptionless.Web 85% 69% 7903
Exceptionless.AppHost 38% 41% 147
Summary 78% (25156 / 32154) 67% (11809 / 17554) 18520

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL