Skip to content

feat: migrate observability to Micrometer - #134

Merged
starboyate merged 4 commits into
mainfrom
migrate-to-micrometer
Aug 12, 2026
Merged

starboyate merged 4 commits into
mainfrom
migrate-to-micrometer

Conversation

@me0106

@me0106 me0106 commented Aug 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Closes #124

  • Replace the old OpenTelemetry tracing/decorator path with Micrometer Observation-based instrumentation across extraction, retrieval, rerank, graph, tier, and strategy components.
  • Add Micrometer observability plugin/starter wiring for observation handlers, business metrics recording, and Spring Boot auto-configuration.
  • Rework retrieval debug trace collection to use request-scoped recorders fed by observation contexts, with bounded API trace output and inline documentation.
  • Update memind-server wiring/configuration for Micrometer observability and OpenTelemetry export through Spring Boot integration.

Review Corrections

This update addresses the follow-up observability review findings:

  • Removed duplicate Spring Boot ObservationHandler registration; handler beans are now discovered and registered by Spring Boot.
  • Aligned timeout fallback status across the API result, retrieval trace, and metrics as DEGRADED.
  • Created a fresh mutable Observation.Context for every Insight generator subscription.
  • Disabled Memind observation creation when memind.server.observability.enabled=false.
  • Corrected the result-merge deduped_count metric.
  • Added observation coverage for custom Reranker implementations without double-observing LlmReranker.
  • Marked query-expansion and rerank fallback paths as degraded.
  • Allowed matching GlobalObservationConvention beans to override component defaults.
  • Removed the unconditional Spring Boot debug: true setting.

Test Plan

  • mvnd test
  • 36 reactor modules succeeded
  • 0 test failures

@starboyate
starboyate merged commit 0e1096f into main Aug 12, 2026
1 check passed
@me0106
me0106 deleted the migrate-to-micrometer branch August 13, 2026 08:40
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.

[Feature]: replace Reactor OpenTelemetry instrumentation with Micrometer observability

2 participants