Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

redact host and instance tags for internal metrics #237

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

shaan420
Copy link
Collaborator

We want to avoid high cardinality on internal metrics due to host and instance tags.

@shaan420 shaan420 changed the title redact host and instance tags internal metrics redact host and instance tags for internal metrics Oct 27, 2023
@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Merging #237 (ab28ef7) into master (5cee107) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
+ Coverage   47.83%   47.85%   +0.01%     
==========================================
  Files          64       64              
  Lines        5991     5993       +2     
==========================================
+ Hits         2866     2868       +2     
  Misses       2685     2685              
  Partials      440      440              
Files Coverage Δ
m3/reporter.go 92.92% <100.00%> (+0.03%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@shaan420 shaan420 merged commit 8686c7d into master Oct 27, 2023
4 checks passed
@shaan420 shaan420 deleted the snair/redact-host-and-instance-tags-internal-metrics branch October 27, 2023 16:09
shaan420 added a commit that referenced this pull request Oct 27, 2023
* redact host and instance tags for internal metrics

* fix unit test
shaan420 added a commit that referenced this pull request Oct 27, 2023
* redact host and instance tags for internal metrics

* fix unit test
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.

1 participant