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

Transient failure in test AsyncIteratorWithCustomCode #34207

Closed
jcouv opened this issue Mar 18, 2019 · 10 comments
Closed

Transient failure in test AsyncIteratorWithCustomCode #34207

jcouv opened this issue Mar 18, 2019 · 10 comments

Comments

@jcouv
Copy link
Member

jcouv commented Mar 18, 2019

Link for all failures of this type.

In the Windows CoreClr Debug CI leg of PR #34141, the AsyncIteratorWithCustomCode failed.

Failing build20190316.11

Error message:
System.InvalidOperationException : Operation is not valid due to the current state of the object.

Stack trace

   at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCoreShared.ThrowInvalidOperationException()
    at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCoreShared.CompletionSentinel(Object _)    at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore`1.InvokeContinuation()

    at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore`1.SignalCompletion()
    at System.Threading.Tasks.Sources.ManualResetValueTaskSourceCore`1.SetResult(TResult result)
    at C.<Main>g__local|0_0()
    at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
    at System.Runtime.CompilerServices.AsyncIteratorMethodBuilder.MoveNext[TStateMachine](TStateMachine& stateMachine)
    at C.<Main>g__local|0_0()
    at C.Main()
    at C.Main()
    at C.<Main>()

Runfo Tracking Issue: Test failure in TestGetAsyncEnumeratorPatternOnRange

Build Definition Kind Run Name

Build Result Summary

Day Hit Count Week Hit Count Month Hit Count
0 0 0
@jcouv jcouv added this to the 16.1.P1 milestone Mar 18, 2019
@jcouv jcouv self-assigned this Mar 18, 2019
@gafter gafter modified the milestones: 16.1.P1, 16.1.P3 Apr 9, 2019
@jcouv jcouv modified the milestones: 16.1.P3, 16.1, 16.2 Apr 18, 2019
@jcouv jcouv modified the milestones: 16.2, 16.3 Jun 19, 2019
@jcouv jcouv modified the milestones: 16.3, Compiler.Next Jul 1, 2019
@jcouv
Copy link
Member Author

jcouv commented Sep 19, 2019

Have not observed this failure since the issue was filed. Closing

@jcouv jcouv closed this as completed Sep 19, 2019
@tmat tmat reopened this Dec 12, 2019
@333fred
Copy link
Member

333fred commented Sep 28, 2020

@sharwell
Copy link
Member

The underlying issue here is affecting multiple tests.

@jcouv jcouv modified the milestones: Compiler.Next, 16.10 Apr 22, 2021
@jinujoseph jinujoseph modified the milestones: 16.10, 17.0 Jul 16, 2021
@jaredpar jaredpar added the Bug label Aug 2, 2021
@jcouv jcouv modified the milestones: 17.0, 17.1 Aug 30, 2021
@vatsalyaagrawal
Copy link
Contributor

@jcouv this issue was hit twice in month of Oct. Please take a look.

@jcouv
Copy link
Member Author

jcouv commented Nov 16, 2021

I looked at the artifacts of builds 1471692, 1471568, 1471568, 1468541, 1463207, 1460280, 1428250 linked in OP today. Only 1460280 had some dump files and none of them seem related to the stacktrace in OP.
Will follow-up with Jared on how to get more information once he's back from training.

@jasonmalinowski
Copy link
Member

@jcouv @jaredpar Were you able to make some progress with this issue?

@jcouv
Copy link
Member Author

jcouv commented Jan 5, 2022

From discussion with Stephen, I'll update the source code that tests use for ManualResetValueTaskSourceCore (may contain a bug) and I'll instrument the async-iterator tests to capture a crash dump.

@jcouv
Copy link
Member Author

jcouv commented Feb 15, 2022

Yay, hit zero occurrences for a month. Closing the issue.
FYI @stephentoub Updating our ManualResetValueTaskSourceCore source did the trick.

@jcouv jcouv closed this as completed Feb 15, 2022
@stephentoub
Copy link
Member

FYI @stephentoub Updating our ManualResetValueTaskSourceCore source did the trick.

Excellent, glad to hear it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

10 participants