Add performance test results for API-M 4.7.0#11160
Open
SavinduDimal wants to merge 6 commits intowso2:masterfrom
Open
Add performance test results for API-M 4.7.0#11160SavinduDimal wants to merge 6 commits intowso2:masterfrom
SavinduDimal wants to merge 6 commits intowso2:masterfrom
Conversation
SavinduDimal
requested review from
chamilaadhi and
tharikaGitHub
as code owners
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughTwo documentation files updated with new test data: Java runtime bumped to Temurin JDK 21, throughput and percentile values revised, a results repo link updated, some APIM CPU graphs removed, and new CPU/DB IOPS sections added for fixed 100 TPS. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@en/docs/reference/performance-tests-results.md`:
- Line 77: Fix the typo in the vendor name on the line that currently reads
"Java version is Termurin JDK 21" by changing "Termurin" to the correct
"Temurin" so the line becomes "Java version is Temurin JDK 21"; locate and
update that exact string in the document to correct the JDK distribution name.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 00dec1dc-cf02-4384-85ad-11046820947d
⛔ Files ignored due to path filters (23)
en/docs/assets/img/setup-and-install/performance-test-results/gc_0ms.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/lineplot-average-response-time-2g-1kib-0ms.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/lineplot-throughput-2g-1kib-0ms.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/lmplot-average-response-time-concurrent-users-2g.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/lmplot-average-response-time-message-size-2g.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/lmplot-throughput-concurrent-users-2g.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/lmplot-throughput-message-size-2g.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/response_time_summary_1KiB_0ms.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/cc_db_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/cc_iops.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/cc_fixed_apim_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/cc_fixed_db_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/cc_fixed_db_iops.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/pwd_fixed_apim_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/pwd_fixed_db_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/pwd_fixed_db_iops.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/refresh_fixed_apim_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/refresh_fixed_db_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/fixed_tps_metrics/refresh_fixed_db_iops.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/pwd_db_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/pwd_iops.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/refresh_db_cpu.pngis excluded by!**/*.pngen/docs/assets/img/setup-and-install/performance-test-results/token/refresh_iops.pngis excluded by!**/*.png
📒 Files selected for processing (2)
en/docs/reference/performance-test-results-token-persistence.mden/docs/reference/performance-tests-results.md
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
en/docs/reference/performance-tests-results.md (1)
89-89: Consider more descriptive link text.The link text "here" is non-descriptive. Readers scanning the document or using assistive technologies would benefit from more explicit text.
♻️ Proposed improvement
-The complete results can be found [here](https://github.com/wso2/performance-apim/blob/performance-test-644-2026-04-07_07-39-24/performance/benchmarks/summary.md). +The [complete benchmark results](https://github.com/wso2/performance-apim/blob/performance-test-644-2026-04-07_07-39-24/performance/benchmarks/summary.md) are available in the performance repository.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@en/docs/reference/performance-tests-results.md` at line 89, Replace the non-descriptive link text in the sentence "the complete results can be found [here](https://github.com/wso2/performance-apim/blob/performance-test-644-2026-04-07_07-39-24/performance/benchmarks/summary.md)" with a clear, accessible label (for example "complete performance test results on GitHub" or "performance-apim benchmark summary") so the link text describes the destination and purpose; update only the link text while keeping the same URL.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@en/docs/reference/performance-tests-results.md`:
- Line 118: Update the image filename in the markdown so its case matches the
actual asset: replace any occurrences of "response_time_summary_1kib_0ms.png"
with "response_time_summary_1KiB_0ms.png" in the document (the image reference
line and any linked/img paths), e.g., update the tag referencing
response_time_summary_1kib_0ms.png to use response_time_summary_1KiB_0ms.png to
avoid broken links.
---
Nitpick comments:
In `@en/docs/reference/performance-tests-results.md`:
- Line 89: Replace the non-descriptive link text in the sentence "the complete
results can be found
[here](https://github.com/wso2/performance-apim/blob/performance-test-644-2026-04-07_07-39-24/performance/benchmarks/summary.md)"
with a clear, accessible label (for example "complete performance test results
on GitHub" or "performance-apim benchmark summary") so the link text describes
the destination and purpose; update only the link text while keeping the same
URL.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: b4d45047-4f88-4c24-a948-e85e092b81e5
📒 Files selected for processing (2)
en/docs/reference/performance-test-results-token-persistence.mden/docs/reference/performance-tests-results.md
Add temp image name to bypass macOS file name case insensitive issue
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
1 participant
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.
This pull request updates the API Manager performance test documentation to reflect newer test environments, revised throughput and latency results, and improved visualizations for performance metrics. The most significant changes include updating the Java version used in tests, revising throughput and percentile data, refreshing result links, and adding new CPU/IOPS graphs under different load conditions.
Test Environment Updates:
performance-test-results-token-persistence.mdandperformance-tests-results.md. [1] [2]Performance Results and Data Updates:
Visualization and Documentation Improvements:
Summary by CodeRabbit