Skip to content

fix(exporter/signozclickhousemetrics): do not count reduced metrics towards usage and other fixes#852

Merged
srikanthccv merged 5 commits into
mainfrom
839-follow-up
Jun 30, 2026
Merged

fix(exporter/signozclickhousemetrics): do not count reduced metrics towards usage and other fixes#852
srikanthccv merged 5 commits into
mainfrom
839-follow-up

Conversation

@srikanthccv

@srikanthccv srikanthccv commented Jun 27, 2026

Copy link
Copy Markdown
Member

Follow up to #839

  • exclude reduced metrics from usage count, and report the retained samples separately
  • fix the issue with histogram .bucket reduced rule not working, the metric name is further normalised
  • add the missing 1.5x we discussed, good it works without it but as an added benefit, update
  • update the usage to report from the reduced tables in hourly fashion, with 2 windows look back to tolerate 2-3 hours of delay in ingesting data (under regular circumstances). it can always be rectified if needed.

@srikanthccv srikanthccv changed the title fix(exporter/signozclickhousemetrics): do not count reduced fingerprints towards usage and other fixes fix(exporter/signozclickhousemetrics): do not count reduced metrics towards usage and other fixes Jun 30, 2026
@srikanthccv srikanthccv marked this pull request as ready for review June 30, 2026 04:05
Comment thread usage/usage.go
@srikanthccv srikanthccv merged commit e344216 into main Jun 30, 2026
7 checks passed
@srikanthccv srikanthccv deleted the 839-follow-up branch June 30, 2026 13:56
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.

3 participants