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

Investigate and fix test hang for EventProcessorTests.MainProcessingLoop.BackgroundProcessingStopsProcessingAllPartitionsWhenShutdown #11731

Closed
jsquire opened this issue May 1, 2020 · 0 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs
Milestone

Comments

@jsquire
Copy link
Member

jsquire commented May 1, 2020

Summary

After investigation, it was determined that the test EventProcessorTests.MainProcessingLoop.BackgroundProcessingStopsProcessingAllPartitionsWhenShutdown has been intermittently hanging during CI runs, most often on the net461 target. Thus far, a local reproduction has not been found. It is suspected that the approach to synchronization used on lines 578, 589, and 637 are the potential root cause.

Scope of Work

  • Investigate the test, attempting to locate and devise a fix for the intermittent hang.

  • Reenable the test to be run normally.

  • When the PR is submitted, use that as the basis to kick off several rounds of the CI build/test to help validate, as local run results are likely misleading.

Success Criteria

  • The root cause of the intermittent hang has been found and fixed.

  • The test has been reenabled and is included in test runs normally.

  • The fix has been validated by at least 3 consecutive successful CI runs against the pull request.

  • The existing test suite continues to pass consistently with deterministic results.

@jsquire jsquire added Test Debt Event Hubs Client This issue points to a problem in the data-plane of the library. labels May 1, 2020
@jsquire jsquire added this to the Backlog milestone May 1, 2020
@jsquire jsquire modified the milestones: Backlog, [2020] June May 2, 2020
@jsquire jsquire self-assigned this May 2, 2020
@jsquire jsquire closed this as completed May 7, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this issue Nov 19, 2020
Reverting the changed made for the address space update in peered vNets. PR: 11090 (Azure#11731)

Co-authored-by: Hari Prasad Perabattula <haperaba@microsoft.com>
@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.
Labels
Client This issue points to a problem in the data-plane of the library. Event Hubs
Projects
None yet
Development

No branches or pull requests

1 participant