Skip to content

Don't expire all time total counts

Jonas Larsen requested to merge j_lar/fix_expiry_to_total_count into master

What does this MR do and why?

Fixes a bug: Total count metrics without time frame should never expire. It is important to get this merged as soon as possible so we don't loose any total counters.

We might also want to go through and remove the expiry date for all our Redis keys that doesn't have a a ´<year_week>` component.

We should also add a test to make sure this doesn't happen in the future again.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request reports