trickstercache / trickster / 39556046233

94%

main: 94%

Build Type

Pull #1212

github

Committed by thinker0

Commit Message

fix(alb,tsm): skip timeseries unmarshaling for non-2xx gather responses

When an ALB TSM pool member returns a non-2xx status code (e.g., 400 Bad Request
with error JSON), TSM was attempting to unmarshal the error payload as a timeseries
dataset via prepareGatherContribution. This caused spurious decode failure warnings
and prevented clean error propagation.

This change guards prepareGatherContribution with a 2xx check in both
serveStandard and collectPlanResult, ensuring non-2xx responses are treated
as failed member contributions without attempting timeseries unmarshaling.

Signed-off-by: thinker0 <thinker0@gmail.com>
Co-authored-by: Gemini <noreply@google.com>

Pull Request Pull Request #1212: fix(alb,tsm): skip timeseries unmarshaling for non-2xx gather responses

Coverage Stats

10 of 12 new or added lines in 3 files covered. (83.33%)

3 existing lines in 2 files now uncovered.

89805 of 95799 relevant lines covered (93.74%)

16998.58 hits per line