Skip to content

Add chasm Archetype tag to dlq metric writer#10004

Open
awln-temporal wants to merge 4 commits intomainfrom
awln-dlq-archetype-tag
Open

Add chasm Archetype tag to dlq metric writer#10004
awln-temporal wants to merge 4 commits intomainfrom
awln-dlq-archetype-tag

Conversation

@awln-temporal
Copy link
Copy Markdown
Contributor

@awln-temporal awln-temporal commented Apr 21, 2026

What changed?

Add chasm Archetype tag to dlq metrics.

Why?

Need a way to filter CHASM tasks by Archetype, through use of indexed labels in Loki. Instead of filtering queries like {k8s_component="history"} |~ "ChasmPure" |~ "Scheduler" |~ "error|fail" | json, we can filter using {k8s_component="history",archetype="Scheduler"} for better performance.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

@awln-temporal awln-temporal requested review from a team as code owners April 21, 2026 16:23
@awln-temporal awln-temporal force-pushed the awln-dlq-archetype-tag branch from c2580f4 to 1b5ae3e Compare April 22, 2026 15:47
@awln-temporal awln-temporal force-pushed the awln-dlq-archetype-tag branch from 1b5ae3e to 14230d4 Compare April 22, 2026 17:54
@awln-temporal awln-temporal requested a review from yycptt April 22, 2026 22:53
Comment on lines 127 to 128
metrics.TaskCategoryTag(task.GetCategory().Name()),
metrics.NamespaceStateTag(namespaceState),
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.

I think we can remove those two in next release after this change is rolledout and alert definition get updated. TaskTypeTag gives both the queue and the active/standby information.

@awln-temporal awln-temporal enabled auto-merge (squash) April 23, 2026 20:11
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