m3 icon indicating copy to clipboard operation
m3 copied to clipboard

Add IncludeTags to R2 Rollup Options

Open kentzeng12 opened this issue 1 year ago • 4 comments

What this PR does / why we need it:

To add new reserved metric tags, we need to add the reserved tag to the initial metric id, and the rollup id otherwise the reserved tag will be lost during the R2 rollup process. Currently to do this, we would need to manually update the R2 rules to filter for the new reserved tag. However, this is a time consuming process because we need to update all R2 rules in the repo to ensure the reserved tag is added to all rollup ids.

To address this issue, in the matcher config, I added functionality to specify a list of "include tags" which are tags that if seen in the metric id, we should add to the rollup id. So if the metric id contains a reserved tag that needs to be added to the rollup id, the forward matching logic will add the reserved tag to the rollup id.

In the future, if there are more reserved tags that R2 forward match should add to rollup ids, we should add those tags in the include tags config.

kentzeng12 avatar May 28 '24 17:05 kentzeng12

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar May 28 '24 17:05 CLAassistant

Please provide more clear description of the problem.

abliqo avatar Jun 06 '24 19:06 abliqo

Please provide more clear description of the problem.

Please provide more clear description of the problem.

kentzeng12 avatar Jun 06 '24 22:06 kentzeng12

Please provide more clear description of the problem

Updated the problem description. Let me know if I should expand on anything.

kentzeng12 avatar Jun 06 '24 23:06 kentzeng12