Skip to content

Add search_probe label and tests for metrics-exporter#101

Merged
rlmanrique merged 2 commits into
mainfrom
metrics_exporter/search_probe
May 29, 2026
Merged

Add search_probe label and tests for metrics-exporter#101
rlmanrique merged 2 commits into
mainfrom
metrics_exporter/search_probe

Conversation

@rlmanrique

@rlmanrique rlmanrique commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Exposes searchProbe from benchmark results as the search_probe Prometheus label on all exporter metrics.
  • Adds unit tests (JSON parsing, label defaults, latest-file selection, metric reset) and a component test that verifies directory polling publishes the newest results to /metrics.
  • Runs go test ./... for metrics-exporter in CI.

Test plan

  • cd metrics-exporter && go test ./...
  • CI green on this PR

rlmanrique and others added 2 commits May 29, 2026 10:57
Refactor metric registration and polling for testability, and run metrics-exporter tests in CI.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rlmanrique
rlmanrique requested a review from a team as a code owner May 29, 2026 09:03

@orca-security-eu orca-security-eu Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@rlmanrique
rlmanrique merged commit 0017d46 into main May 29, 2026
6 checks passed
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.

1 participant