8000 [TEST] Add stress test for histogram metric for multiple threads validation by ThomsonTan · Pull Request #3388 · open-telemetry/opentelemetry-cpp · GitHub
[go: up one dir, main page]
Skip to content

[TEST] Add stress test for histogram metric for multiple threads validation #3388

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 16 commits into from
May 3, 2025
Merged

[TEST] Add stress test for histogram metric for multiple threads validation #3388

merged 16 commits into from
May 3, 2025
+191 −1

Conversation

ThomsonTan
Copy link
Contributor

Fixes #3387

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@ThomsonTan ThomsonTan requested a review from a team as a code owner May 2, 2025 21:34
@netlify Netlify
Copy link
netlify bot commented May 2, 2025 •
edited
Loading

✅ Deploy Preview for opentelemetry-cpp-api-docs canceled.

Name Link
🔨 Latest commit 6932d54
🔍 Latest deploy log https://app.netlify.com/sites/opentelemetry-cpp-api-docs/deploys/6815a1f58aed430008235093

@codecov Codecov
Copy link
codecov bot commented May 2, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.91%. Comparing base (9fd8511) to head (6932d54).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3388      +/-   ##
==========================================
+ Coverage   89.66%   89.91%   +0.25%     
==========================================
  Files         211      211              
  Lines        6832     6832              
==========================================
+ Hits         6125     6142      +17     
+ Misses        707      690      -17

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ThomsonTan ThomsonTan added the pr:please-review This PR is ready for review label May 3, 2025
lalitb
lalitb reviewed May 3, 2025
View reviewed changes
@marcalff marcalff merged commit a145a56 into open-telemetry:main May 3, 2025
66 checks passed
@ThomsonTan ThomsonTan deleted the add_stress_test branch May 3, 2025 14:23
@malkia malkia mentioned this pull request May 3, 2025
Merged
3 tasks
@BrewTestBot BrewTestBot mentioned this pull request May 29, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lalitb lalitb lalitb approved these changes

@marcalff marcalff marcalff approved these changes

Assignees

No one assigned

Labels

pr:please-review This PR is ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add stress test to validate the correctness of metrics

3 participants

@ThomsonTan @lalitb @marcalff
0