Skip to content

exporter: add per collection debug output sums#125

Open
T-X wants to merge 1 commit intofreifunk-gluon:mainfrom
T-X:pr-collections-summaries
Open

exporter: add per collection debug output sums#125
T-X wants to merge 1 commit intofreifunk-gluon:mainfrom
T-X:pr-collections-summaries

Conversation

@T-X
Copy link
Contributor

@T-X T-X commented Jan 20, 2026

Adding a "Collections summaries" debug output line as follows:

[...]
2026-01-20 02:56:04 [info     ] Collections summaries          domain_sum=31569 model_sum=37589 version_sum=34872
2026-01-20 02:56:04 [info     ] Summary                        duplicate=82 unique=39152

This allows to more easily and early on check that a change likely/hopefully has no regression. Without needing to manually sum things up from the gluon-census.prom file, without needing to wait for rendered Grafana graphs.

@T-X T-X force-pushed the pr-collections-summaries branch 2 times, most recently from b3c0d3a to 9f0d516 Compare January 20, 2026 03:49
Copy link
Member

@AiyionPrime AiyionPrime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine with the result, but be not mistaken, globals are an error-prone antipattern, even if somebody smart put them there 😁

I'll take care of this after work.

Adding a "Collections summaries" debug output line as follows:

```
[...]
2026-01-20 02:56:04 [info     ] Collections summaries          domain_sum=31569 model_sum=37589 version_sum=34872
2026-01-20 02:56:04 [info     ] Summary                        duplicate=82 unique=39152
```

This allows to more easily and early on check that a change
likely/hopefully has no regression. Without needing to manually sum
things up from the gluon-census.prom file, without needing to wait for
rendered Grafana graphs.

Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
@T-X T-X force-pushed the pr-collections-summaries branch from 9f0d516 to bcd1cc0 Compare January 23, 2026 11:54
@T-X
Copy link
Contributor Author

T-X commented Jan 23, 2026

Changelog v2:

  • rebased to current main branch (no changes)

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