trickstercache / trickster / 23880393174

74%

push

github

feat: [alb/tsm] per-query merge strategy for ALB time-series merge (#956)

* feat: [alb/tsm] configurable merge strategy for aggregation across backends (#927, #928)

Add a `merge_strategy` option to ALB TSM config (sum, avg, min, max,
count, dedup) so that values from matching series across backends are
re-aggregated instead of silently deduplicated. Fixes incorrect results
when fan-out queries use PromQL aggregation functions like sum().

Signed-off-by: Chris Randles <randles.chris@gmail.com>

* fix: [ci] run one publish workflow from main at a time

Signed-off-by: Chris Randles <randles.chris@gmail.com>

* test: bump coverage, address bugs

Signed-off-by: Chris Randles <randles.chris@gmail.com>

* fix: [alb/tsm] strip injected labels before merge when using aggregation strategy

When merge_strategy is active, collect all injected label keys from pool
backends and strip them from each dataset before merging. This ensures
series from different backends hash identically despite having different
injected labels, enabling correct aggregation.

Signed-off-by: Chris Randles <randles.chris@gmail.com>

* docs: update to reference merge strategy

Signed-off-by: Chris Randles <randles.chris@gmail.com>

* fix: revert test config

Signed-off-by: Chris Randles <randles.chris@gmail.com>

* use per-query merge strategy. NOTE: Agentic AI used

Signed-off-by: James Ranson <james@ranson.org>

* linter

Signed-off-by: James Ranson <james@ranson.org>

* cleanup yaml

Signed-off-by: James Ranson <james@ranson.org>

* bump dependencies; golang 1.26 -> 1.26.1

---------

Signed-off-by: Chris Randles <randles.chris@gmail.com>
Signed-off-by: James Ranson <james@ranson.org>
Co-authored-by: James Ranson <james@ranson.org>

382 of 632 new or added lines in 13 files covered. (60.44%)

15849 of 21321 relevant lines covered (74.34%)

0.86 hits per line