Skip to content

[9.1] fix: Use float mapping for fractional service/agent monitoring fields - #8797

Open
chrisburr wants to merge 1 commit into
DIRACGrid:integrationfrom
chrisburr:fix/monitoring-float-mapping
Open

chrisburr wants to merge 1 commit into
DIRACGrid:integrationfrom
chrisburr:fix/monitoring-float-mapping

Conversation

@chrisburr

Copy link
Copy Markdown
Member

ResponseTime, CpuPercentage and CycleDuration are floats (seconds and percent) but were mapped as long, so OpenSearch truncated them in the indexed values. Aggregations (avg, percentiles) therefore returned 0 for all sub-second response times, while _source kept the correct values.

The new mapping only applies to newly created (weekly) indices; existing indices need a reindex to recover correct aggregations.

BEGINRELEASENOTES

*MonitoringSystem
FIX: Use float mapping for ResponseTime, CpuPercentage and CycleDuration in ServiceMonitoring and AgentMonitoring so that aggregations are no longer truncated (existing indices need to be reindexed)

ENDRELEASENOTES

ResponseTime, CpuPercentage and CycleDuration are floats (seconds and
percent) but were mapped as long, so OpenSearch truncated them in the
indexed values. Aggregations (avg, percentiles) therefore returned 0 for
all sub-second response times, while _source kept the correct values.

The new mapping only applies to newly created (weekly) indices; existing
indices need a reindex to recover correct aggregations.
@chrisburr
chrisburr force-pushed the fix/monitoring-float-mapping branch from 978d1cc to fb4fcb2 Compare September 22, 2026 21:56
@chrisburr chrisburr changed the title fix: Use float mapping for fractional service/agent monitoring fields [v9.1] fix: Use float mapping for fractional service/agent monitoring fields Sep 22, 2026
@chrisburr chrisburr changed the title [v9.1] fix: Use float mapping for fractional service/agent monitoring fields [9.1] fix: Use float mapping for fractional service/agent monitoring fields Sep 22, 2026
@chrisburr
chrisburr marked this pull request as ready for review September 22, 2026 21:56
@chrisburr
chrisburr requested a review from fstagni as a code owner September 22, 2026 21:56

This branch has not been deployed

No deployments
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