Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add more errors to targeted fault injector #4711

Conversation

alexshtin
Copy link
Member

@alexshtin alexshtin commented Aug 1, 2023

What changed?

  • Added ResourceExhausted and Unavailable errors to target fault injector error factory.
  • Small renames.

Why?
To support more fault testing scenarios such as persistence rate limiting.

How did you test it?
Run locally.

Potential risks
No risks.

Is hotfix candidate?
No.

@alexshtin alexshtin requested a review from a team as a code owner August 1, 2023 20:46
@alexshtin alexshtin force-pushed the feature/resource-exhousted-fault-injection branch from 0dda528 to 18edcf8 Compare August 2, 2023 01:05
@alexshtin alexshtin enabled auto-merge (squash) August 2, 2023 01:47
@alexshtin alexshtin merged commit 4efda3c into temporalio:master Aug 2, 2023
@alexshtin alexshtin deleted the feature/resource-exhousted-fault-injection branch August 2, 2023 06:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants