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

Show metrics needing measurement in the UI. #8660

Conversation

fniessink
Copy link
Member

@fniessink fniessink commented May 10, 2024

In the UI, show a metric as needing a measurement because the source parameters were recently changed.

Implemented by:

  • In the collector: adding a hash of the source parameters to new measurements (and removing it from old measurements).
  • In the API-server: compare the hash of the latest measurement with the hash of the current configuration when adding recent measurements to a report and mark the metric as outdated.
  • In the UI: show metrics as needing a measurement if the last measurement is outdated.

Closes #3134.

@fniessink fniessink force-pushed the 3134-show-whether-the-latest-measurement-of-metric-has-been-done-with-the-current-configuration-of-its-sources branch 4 times, most recently from d56583b to c3bfa83 Compare May 10, 2024 14:47
@fniessink fniessink force-pushed the 3134-show-whether-the-latest-measurement-of-metric-has-been-done-with-the-current-configuration-of-its-sources branch from c3bfa83 to 391ab84 Compare May 10, 2024 15:28
@fniessink fniessink force-pushed the 3134-show-whether-the-latest-measurement-of-metric-has-been-done-with-the-current-configuration-of-its-sources branch from 391ab84 to c550cc3 Compare May 10, 2024 15:48
fniessink added a commit that referenced this pull request May 10, 2024
fniessink added a commit that referenced this pull request May 10, 2024
@fniessink fniessink force-pushed the 3134-show-whether-the-latest-measurement-of-metric-has-been-done-with-the-current-configuration-of-its-sources branch 14 times, most recently from 70e23bd to b1c5827 Compare May 15, 2024 13:41
@fniessink fniessink marked this pull request as ready for review May 15, 2024 13:51
@fniessink fniessink requested a review from wkoot May 15, 2024 13:54
@fniessink fniessink modified the milestone: v4.9.1 May 16, 2024
In the UI, show a metric as needing a measurement because the source parameters were recently changed.

Implemented by:
- In the collector: adding a hash of the source parameters to new measurements (and removing it from old measurements).
- In the API-server: compare the hash of the latest measurement with the hash of the current configuration when adding recent measurements to a report and mark the metric as outdated.
- In the UI: show metrics as needing a measurement if the last measurement is outdated.

Closes #3134.
@fniessink fniessink force-pushed the 3134-show-whether-the-latest-measurement-of-metric-has-been-done-with-the-current-configuration-of-its-sources branch from b1c5827 to c2666be Compare May 16, 2024 14:10
Copy link

sonarcloud bot commented May 16, 2024

@fniessink fniessink merged commit 6afc491 into master May 16, 2024
39 checks passed
@fniessink fniessink deleted the 3134-show-whether-the-latest-measurement-of-metric-has-been-done-with-the-current-configuration-of-its-sources branch May 16, 2024 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show whether the latest measurement of metric has been done with the current configuration of its source(s)
2 participants