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

[azeventhubs] Prep for release, fixing tests #19364

Merged
merged 3 commits into from
Oct 17, 2022

Conversation

richardpark-msft
Copy link
Member

This is just a simple update for QA, no fixes and also pushing the changelog date forward since it got delayed.

  • Added in some more tests for recovery. There were some incidents in the stress cluster but they turned out to be red herrings. They did reveal that some more granular tests should exist and now we have them for both link and connection level recovery.
  • Fixed a bug in one of the stress tests. We kept getting an odd (but non-fatal) error as the test tore down (after passing!) about the connection idling out. Turns out it was accurate - the batch test is a little unusual in that it doesn't send events continually - it sends them once at the beginning of the test and the consumers just keep rewinding to pick them up again. So the producer client just needed to be closed after it's part in the test was done.

Fixes #19220

…he producer client is only used for a small time in the very beginning of the test and does actually idle out at the end)
@richardpark-msft
Copy link
Member Author

/azp run go - azeventhubs

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@richardpark-msft
Copy link
Member Author

/azp run go - azeventhubs

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@richardpark-msft richardpark-msft merged commit e50c6cc into Azure:main Oct 17, 2022
@richardpark-msft richardpark-msft deleted the eh-connection-retry branch October 17, 2022 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[azeventhubs] Idle/close behavior when batch stress test exits
2 participants