Fix KraftCloud public deployment timeouts - #1261
Merged
Merged
Conversation
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
🤖 Augment PR SummarySummary: Extends the KraftCloud public deployment RPC timeout from the default one minute to five minutes. 🤖 Was this summary useful? React with 👍 or 👎 |
Contributor
There was a problem hiding this comment.
Benchmark (community)
Details
| Benchmark suite | Current: d2ee11c | Previous: 13a1dc6 | Ratio |
|---|---|---|---|
Add one schema (0 existing) |
230 ms |
240 ms |
0.96 |
Add one schema (100 existing) |
38 ms |
43 ms |
0.88 |
Add one schema (1000 existing) |
96 ms |
92 ms |
1.04 |
Add one schema (10000 existing) |
786 ms |
728 ms |
1.08 |
Update one schema (1 existing) |
31 ms |
31 ms |
1 |
Update one schema (101 existing) |
38 ms |
38 ms |
1 |
Update one schema (1001 existing) |
107 ms |
97 ms |
1.10 |
Update one schema (10001 existing) |
855 ms |
797 ms |
1.07 |
Cached rebuild (1 existing) |
10 ms |
10 ms |
1 |
Cached rebuild (101 existing) |
14 ms |
13 ms |
1.08 |
Cached rebuild (1001 existing) |
42 ms |
40 ms |
1.05 |
Cached rebuild (10001 existing) |
355 ms |
302 ms |
1.18 |
Index 100 schemas |
486 ms |
409 ms |
1.19 |
Index 1000 schemas |
1411 ms |
1441 ms |
0.98 |
Index 10000 schemas |
12290 ms |
12649 ms |
0.97 |
Index 10000 schemas (custom meta-schema) |
14115 ms |
14593 ms |
0.97 |
Index 10000 schemas ($ref fan-out) |
14472 ms |
14832 ms |
0.98 |
test/e2e/html: Schema Fetch (p50) |
388 us |
418 us |
0.93 |
test/e2e/html: Schema Fetch (p99) |
470 us |
513 us |
0.92 |
This comment was automatically generated by workflow using github-action-benchmark.
Contributor
There was a problem hiding this comment.
Benchmark (enterprise)
Details
| Benchmark suite | Current: d2ee11c | Previous: 13a1dc6 | Ratio |
|---|---|---|---|
Add one schema (0 existing) |
301 ms |
300 ms |
1.00 |
Add one schema (100 existing) |
117 ms |
125 ms |
0.94 |
Add one schema (1000 existing) |
165 ms |
180 ms |
0.92 |
Add one schema (10000 existing) |
763 ms |
802 ms |
0.95 |
Update one schema (1 existing) |
106 ms |
114 ms |
0.93 |
Update one schema (101 existing) |
115 ms |
128 ms |
0.90 |
Update one schema (1001 existing) |
168 ms |
185 ms |
0.91 |
Update one schema (10001 existing) |
1136 ms |
1135 ms |
1.00 |
Cached rebuild (1 existing) |
14 ms |
14 ms |
1 |
Cached rebuild (101 existing) |
17 ms |
18 ms |
0.94 |
Cached rebuild (1001 existing) |
48 ms |
47 ms |
1.02 |
Cached rebuild (10001 existing) |
342 ms |
340 ms |
1.01 |
Index 100 schemas |
544 ms |
597 ms |
0.91 |
Index 1000 schemas |
1533 ms |
1490 ms |
1.03 |
Index 10000 schemas |
13012 ms |
12410 ms |
1.05 |
Index 10000 schemas (custom meta-schema) |
14968 ms |
14538 ms |
1.03 |
Index 10000 schemas ($ref fan-out) |
14773 ms |
14479 ms |
1.02 |
enterprise/e2e/auth: Schema Anonymous (p50) |
398 us |
375 us |
1.06 |
enterprise/e2e/auth: Schema Anonymous (p99) |
507 us |
496 us |
1.02 |
enterprise/e2e/auth: Schema API Key Identity (p50) |
402 us |
378 us |
1.06 |
enterprise/e2e/auth: Schema API Key Identity (p99) |
519 us |
486 us |
1.07 |
enterprise/e2e/auth: Schema API Key SHA256 (p50) |
411 us |
388 us |
1.06 |
enterprise/e2e/auth: Schema API Key SHA256 (p99) |
534 us |
494 us |
1.08 |
enterprise/e2e/auth: Schema JWT (p50) |
537 us |
532 us |
1.01 |
enterprise/e2e/auth: Schema JWT (p99) |
691 us |
704 us |
0.98 |
test/e2e/html: Schema Fetch (p50) |
405 us |
383 us |
1.06 |
test/e2e/html: Schema Fetch (p99) |
497 us |
468 us |
1.06 |
This comment was automatically generated by workflow using github-action-benchmark.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Juan Cruz Viotti jv@jviotti.com