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

[Do Not Merge] add a stress test to verify there is no memory leak from key components #2671

Closed
wants to merge 4 commits into from

Conversation

reyang
Copy link
Member

@reyang reyang commented Nov 24, 2021

Not sure if we want this to be merged. Just share the code in case folks find it useful during major refactor.

The test keeps creating Meter/LoggerProvider/Logger/ActivitySource/Activity together with the OpenTelemetry providers/readers/processors/exporter, using them and then dropping them on the floor.

@reyang reyang requested a review from a team November 24, 2021 00:05
@reyang reyang added perf Performance related infra Infra work - CI/CD, code coverage, linters pr:do-not-merge pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package labels Nov 24, 2021
@codecov
Copy link

codecov bot commented Nov 24, 2021

Codecov Report

Merging #2671 (0bd627f) into main (e7f026e) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2671      +/-   ##
==========================================
- Coverage   82.27%   82.26%   -0.02%     
==========================================
  Files         249      249              
  Lines        8688     8688              
==========================================
- Hits         7148     7147       -1     
- Misses       1540     1541       +1     
Impacted Files Coverage Δ
...ter.ZPages/Implementation/ZPagesActivityTracker.cs 97.14% <0.00%> (-2.86%) ⬇️

@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale Issues and pull requests which have been flagged for closing due to inactivity label Jan 28, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2022

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Feb 5, 2022
@reyang reyang deleted the reyang/stress branch February 15, 2022 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Infra work - CI/CD, code coverage, linters perf Performance related pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package Stale Issues and pull requests which have been flagged for closing due to inactivity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant