Skip to content

Add basic connection interruption tests #3292

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
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add basic connection interruption tests #3292

wants to merge 2 commits into from

Conversation

uglide
Copy link
Collaborator

@uglide uglide commented May 13, 2025

  • Use reactor-netty-http client to interact with the Fault Injector
  • Test command execution and PubSub with dmc_restart, network_failure failures injected

uglide added 2 commits May 13, 2025 16:43
@uglide
Add basic connection interruption tests
73627ab 
- Use reactor-netty-http client to interact
with Fault Injector
- Test command execution and PubSub with
dmc_restart, network_failure failures injected
@uglide
Fix formatting
d774e05
kiryazovi-redis
kiryazovi-redis reviewed May 14, 2025
View reviewed changes
Copy link
Collaborator

@kiryazovi-redis kiryazovi-redis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good sample tests
Will prove invaluable as a starting template to further testing with fault injector
not an approver myself, though.

@tishun tishun added this to the Async milestone May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kiryazovi-redis kiryazovi-redis kiryazovi-redis left review comments

@tishun tishun Awaiting requested review from tishun

@ggivo ggivo Awaiting requested review from ggivo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Async
Development

Successfully merging this pull request may close these issues.
3 participants
@uglide @kiryazovi-redis @tishun