Skip to content

Commit a260cbd

Browse files
authored
Increase retry delay in conversation handler tests (#2807)
* Increase retry delay in conversation handler tests * Increase retry delay in conversation handler tests * Increase retry delay in conversation handler tests
1 parent 360b3ac commit a260cbd

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

.changeset/short-rings-tan.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
---
2+
---

packages/integration-tests/src/test-project-setup/conversation_handler_project.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1189,14 +1189,16 @@ class ConversationHandlerTestProject extends TestProjectBase {
11891189
};
11901190

11911191
/**
1192-
* Bedrock sometimes produces half-baked response or we run into throttling.
1192+
* Bedrock sometimes produces half-baked or no response when we run into throttling.
11931193
* On the other hand we have to run some assertions on those responses.
11941194
* Therefore, we wrap transactions in retry loop.
11951195
*/
11961196
private executeWithRetry = async (
11971197
callable: (attempt: number) => Promise<void>,
11981198
) => {
1199-
const retryDelayMs = 30 * 1000; // 30 seconds.
1199+
// Bedrock has low request per minute quota for our test accounts.
1200+
// Therefore, retrying earlier than one minute might not be successful.
1201+
const retryDelayMs = 90 * 1000; // 90 seconds. (RPM quota window + some buffer).
12001202
await runWithRetry(callable, () => true, 4, retryDelayMs);
12011203
};
12021204
}

0 commit comments

Comments
 (0)