Skip to content

Commit 35aacca

Browse files
authored
Add RESOURCE_EXHAUSTED_CAUSE_WORKER_DEPLOYMENT_LIMITS (#662)
_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._ <!-- Describe what has changed in this PR --> **What changed?** Adding a new RESOURCE_EXHAUSTED_CAUSE enum item <!-- Tell your future self why have you made these changes --> **Why?** User in returned ResourceExhausted errors related to worker deployments. <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** None <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR** temporalio/temporal#8620 is waiting for this API change to merge
1 parent bb50066 commit 35aacca

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

temporal/api/enums/v1/failed_cause.proto

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,8 @@ enum ResourceExhaustedCause {
123123
RESOURCE_EXHAUSTED_CAUSE_CIRCUIT_BREAKER_OPEN = 8;
124124
// Namespace exceeds operations rate limit.
125125
RESOURCE_EXHAUSTED_CAUSE_OPS_LIMIT = 9;
126+
// Limits related to Worker Deployments are reached.
127+
RESOURCE_EXHAUSTED_CAUSE_WORKER_DEPLOYMENT_LIMITS = 10;
126128
}
127129

128130
enum ResourceExhaustedScope {

0 commit comments

Comments
 (0)