Skip to content

fix(metrics): Increase sample rate of internal metrics to 100%#55982

Merged
iambriccardo merged 3 commits into
masterfrom
riccardo/fix/sample-100
Sep 11, 2023
Merged

fix(metrics): Increase sample rate of internal metrics to 100%#55982
iambriccardo merged 3 commits into
masterfrom
riccardo/fix/sample-100

Conversation

@iambriccardo

@iambriccardo iambriccardo commented Sep 11, 2023

Copy link
Copy Markdown
Contributor

This PR sets sample rate to 100% for all internal metrics coming from minimetrics and improves handling of thread locals.

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 11, 2023
@codecov

codecov Bot commented Sep 11, 2023

Copy link
Copy Markdown

Codecov Report

Merging #55982 (846703a) into master (81116ea) will decrease coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #55982      +/-   ##
==========================================
- Coverage   79.95%   79.39%   -0.56%     
==========================================
  Files        5060     5060              
  Lines      217580   217580              
  Branches    36833    36833              
==========================================
- Hits       173963   172756    -1207     
- Misses      38290    39382    +1092     
- Partials     5327     5442     +115     

@iambriccardo iambriccardo merged commit 4395fa0 into master Sep 11, 2023
@iambriccardo iambriccardo deleted the riccardo/fix/sample-100 branch September 11, 2023 13:09
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants