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.

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

[FlakyTest] WithTimeoutPropagatesAnExceptionThatCompletesBeforeTimeout #12874

Closed
pakrym opened this issue Jun 18, 2020 · 2 comments
Closed

[FlakyTest] WithTimeoutPropagatesAnExceptionThatCompletesBeforeTimeout #12874

pakrym opened this issue Jun 18, 2020 · 2 comments
Assignees
@jsquire
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs Service Attention Workflow: This issue is responsible by Azure service team.
Milestone
[2020] July

Comments

@pakrym
Copy link
Contributor

pakrym commented Jun 18, 2020

Microsoft.Azure.EventHubs.Tests.TaskExtensionsTests.WithTimeoutPropagatesAnExceptionThatCompletesBeforeTimeout

https://dev.azure.com/azure-sdk/public/_build/results?buildId=431372&view=ms.vss-test-web.build-test-results-tab&runId=11547436&resultId=103502&paneView=debug

Assert.Throws() Failure
Expected: typeof(System.MissingMemberException)
Actual: typeof(System.TimeoutException): The operation has timed out.
---- System.TimeoutException : The operation has timed out.


Stack trace
   at Microsoft.Azure.EventHubs.Tests.TaskExtensions.<WithTimeout>d__0.MoveNext() in D:\a\1\s\sdk\eventhub\Microsoft.Azure.EventHubs\tests\Infrastructure\TaskExtensions.cs:line 60
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.Azure.EventHubs.Tests.TaskExtensionsTests.<<WithTimeoutPropagatesAnExceptionThatCompletesBeforeTimeout>b__10_0>d.MoveNext() in D:\a\1\s\sdk\eventhub\Microsoft.Azure.EventHubs\tests\Instrastructure.Tests\TaskExtensionTests.cs:line 128
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
----- Inner Stack Trace -----
   at Microsoft.Azure.EventHubs.Tests.TaskExtensions.<WithTimeout>d__0.MoveNext() in D:\a\1\s\sdk\eventhub\Microsoft.Azure.EventHubs\tests\Infrastructure\TaskExtensions.cs:line 60
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.Azure.EventHubs.Tests.TaskExtensionsTests.<<WithTimeoutPropagatesAnExceptionThatCompletesBeforeTimeout>b__10_0>d.MoveNext() in D:\a\1\s\sdk\eventhub\Microsoft.Azure.EventHubs\tests\Instrastructure.Tests\TaskExtensionTests.cs:line 128
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

cc @jsquire
@pakrym pakrym added Event Hubs Client This issue points to a problem in the data-plane of the library. Test Debt labels Jun 18, 2020
@jsquire jsquire added the Service Attention Workflow: This issue is responsible by Azure service team. label Jun 18, 2020
@ghost
Copy link

ghost commented Jun 18, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @jfggdl.

@jsquire jsquire added this to the Backlog milestone Jun 20, 2020
@jsquire jsquire modified the milestones: Backlog, [2020] July Jun 22, 2020
@jsquire jsquire mentioned this issue Jun 22, 2020
Merged
jsquire added a commit to jsquire/azure-sdk-for-net that referenced this issue Jun 23, 2020
@jsquire
[Event Hubs Client] Minor Refactorings and Tweaks
4680fea 
The focus of these changes is a set of small refactorings and tweaks, mostly
unrelated to any core theme.  Included are:

- Refactoring of the processor blob storage manager, mostly changes to formatting
  to help improve readability.

- Refactoring the initialization of starting position for partitions into a concurrent
  dictionary to resolve a potential race condition (Azure#12466)

- Fixing timing on some track one tests for test-infrastructure.  (Azure#12874)

- Ignoring tests with intermittent failures (Azure#12929, Azure#12930)

- Enhancing log information for when the Event Processor starts processing a
  partition to contain the starting position used.

- Enhancing log information for the core service operations (send/receive) to
  group the start/error/end events together with an operation identifier and
  report the total number of retries used when interacting with the service.
jsquire added a commit that referenced this issue Jun 23, 2020
@jsquire
[Event Hubs Client] Minor Refactorings and Tweaks (#12928)
f1b73c7 
The focus of these changes is a set of small refactorings and tweaks, mostly
unrelated to any core theme.  Included are:

- Refactoring of the processor blob storage manager, mostly changes to formatting
  to help improve readability.

- Refactoring the initialization of starting position for partitions into a concurrent
  dictionary to resolve a potential race condition (#12466)

- Fixing timing on some track one tests for test-infrastructure.  (#12874)

- Ignoring tests with intermittent failures (#12929, #12930)

- Enhancing log information for when the Event Processor starts processing a
  partition to contain the starting position used.

- Enhancing log information for the core service operations (send/receive) to
  group the start/error/end events together with an operation identifier and
  report the total number of retries used when interacting with the service.
@jsquire
Copy link
Member

jsquire commented Jun 23, 2020

A fix for this went in with #12928.

@jsquire jsquire closed this as completed Jun 23, 2020
@jsquire jsquire mentioned this issue Jun 23, 2020
Closed
10 tasks
prmathur-microsoft pushed a commit that referenced this issue Jul 8, 2020
@jsquire @prmathur-microsoft
[Event Hubs Client] Minor Refactorings and Tweaks (#12928)
56eab0c 
The focus of these changes is a set of small refactorings and tweaks, mostly
unrelated to any core theme.  Included are:

- Refactoring of the processor blob storage manager, mostly changes to formatting
  to help improve readability.

- Refactoring the initialization of starting position for partitions into a concurrent
  dictionary to resolve a potential race condition (#12466)

- Fixing timing on some track one tests for test-infrastructure.  (#12874)

- Ignoring tests with intermittent failures (#12929, #12930)

- Enhancing log information for when the Event Processor starts processing a
  partition to contain the starting position used.

- Enhancing log information for the core service operations (send/receive) to
  group the start/error/end events together with an operation identifier and
  report the total number of retries used when interacting with the service.
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jsquire jsquire

Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Milestone
[2020] July
Development

No branches or pull requests
4 participants
@jsquire @pakrym @JamesBirdsall @serkantkaraca