Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swap to using integers for extended bounds in the data histogram aggregation #61

Conversation

kyle-sammons
Copy link
Collaborator

Summary

With the swap to using OpenSearch aggregation parsing in the main Astra app, we need to update the backend of this Grafana plugin to send over the time ranges for aggregations as integers, otherwise data histogram aggregations on the _timesinceepoch field will fail.

Demo

This demo creates a basic dashboard that uses a date histogram aggregation on the _timesinceepoch field, and tests out an alert on said dashboard:

Previous alerting behavior
image

New alerting behavior
image

Requirements (place an x in each [ ])

…egation instead of strings so that alerts using _timesinceepoch continue to work
ermontross
ermontross previously approved these changes Oct 24, 2024
@kyle-sammons kyle-sammons merged commit 41a28e5 into master Oct 24, 2024
2 checks passed
@kyle-sammons kyle-sammons deleted the ksammons-use-integers-instead-of-strings-for-extended-bounds branch October 24, 2024 20:30
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