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

Test failure WaiterRemovedAfterDispose #5450

Closed
amcasey opened this issue Sep 26, 2024 · 7 comments
Closed

Test failure WaiterRemovedAfterDispose #5450

amcasey opened this issue Sep 26, 2024 · 7 comments
Assignees
Labels
area-fundamentals area-tests For historic purposes waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue.

Comments

@amcasey
Copy link
Member

amcasey commented Sep 26, 2024

Microsoft.Extensions.Time.Testing.Test.TimerTests.WaiterRemovedAfterDispose

Assert.Equal() Failure: Values differ
Expected: 2
Actual:   1
   at Microsoft.Extensions.Time.Testing.Test.TimerTests.WaiterRemovedAfterDispose() in D:\a\_work\1\s\test\Libraries\Microsoft.Extensions.TimeProvider.Testing.Tests\TimerTests.cs:line 237

https://dev.azure.com/dnceng-public/public/_build/results?buildId=818533

@amcasey
Copy link
Member Author

amcasey commented Sep 26, 2024

FYI @RussKie - last time you wanted to investigate, rather than skip

@amcasey
Copy link
Member Author

amcasey commented Sep 26, 2024

@RussKie
Copy link
Member

RussKie commented Nov 5, 2024

@dotnet/dotnet-extensions-fundamentals please triage

@amadeuszl
Copy link
Contributor

amadeuszl commented Nov 12, 2024

I'm checking this one, verifying it's a problem with test itself or the actual logic. Can't assign the issue to myself yet due to lack of permissions.

@rosebyte rosebyte assigned amadeuszl and unassigned omsoftdev Nov 13, 2024
@amadeuszl
Copy link
Contributor

To me it sounds like it could be problem with resource utilization on the agent running the build and unexpected behavior on memory or GC.
There're 2 options I see so far:

  1. Callback gets removed
  2. Second Timer is removed from Provider

For 1st point I'm not really sure how to look for more details. But for second we can exclude it if we check waitersCountAfter first, to see if provider still has a timer. I will issue PR for that.

@amadeuszl
Copy link
Contributor

I'll be checking pipelines from time to time, but let me know in case there's new failure.

@amadeuszl amadeuszl added the waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue. label Nov 18, 2024
Copy link
Contributor

This submission has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 14 days.

It will be closed if no further activity occurs within 7 days of this comment.

@github-actions github-actions bot locked and limited conversation to collaborators Jan 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-fundamentals area-tests For historic purposes waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue.
Projects
None yet
Development

No branches or pull requests

4 participants