-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[system test] [perf] Performance perf report into PR #12124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
see-quick
wants to merge
47
commits into
strimzi:main
Choose a base branch
from
see-quick:perf-report-st
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
54ea727
[system test] [perf] Performance perf report into PR
see-quick e24a841
correct doc after rebase
see-quick af0ee3a
also add oracle agent instead ubuntu latest
see-quick 83de883
check 2 agents
see-quick 975dc3d
correct javadoc
see-quick cdee595
move logic stuff to actions and also lower the creation
see-quick 9425e2f
do not merge perf tests within workflow
see-quick bfd5afa
some fix
see-quick 60af3fc
have epthemeral storage
see-quick 2288b8a
also imports
see-quick 1df141f
try both arches to display
see-quick 76c1da9
[system test] [perf] Performance perf report into PR
see-quick 6beb9a6
refactor a bit
see-quick eee3dd3
[system test] [perf] Performance perf report into PR 4
see-quick 44c0c95
fix actions
see-quick 7839122
fix workflow
see-quick 7ff193e
[system test] [perf] Performance perf report into PR 5
see-quick e490ee7
update action to make to more compact with 2 arches
see-quick f9b6eed
update to eliminate use case in user-operator
see-quick 27a2657
try
see-quick 0f8eb38
try
see-quick af6db46
update
see-quick f47f064
try tests also remove multiple one-step actions
see-quick d621644
local testing
see-quick 760f8f1
a bit fixing
see-quick 565e19f
multi both
see-quick 4f1ca1d
try unifity and one action + with aux js script
see-quick 9acc0f6
avoid error
see-quick 5715fca
try
see-quick fb5ec4f
add doc
see-quick b35c0d8
update to have both reporting arches
see-quick 02e6f78
update rest
see-quick 3b26f70
use persistent
see-quick e7f11bf
try more storage for Kafka + do not limit UO with resource HW as its …
see-quick b54c21f
update latency test case to include some metrics (to check what we sp…
see-quick 090e424
update limits to test case now as we know memory
see-quick 618c8dd
lets try to add more cpus
see-quick b1b05a5
do not limit cpu
see-quick 84ee02b
keep IN: and OUT: parameters within results-table.txt
see-quick ab130c5
adjust cpus specific to GHA runners
see-quick 4423660
modify input tests in perf report
see-quick c5a05b1
update perf-report expected results
see-quick a347815
Jakub review + fix tests + re-name .txt to .md + refactor generation.js
see-quick a4f356e
remove ref from run-perf-report step
see-quick c5c416c
udpate workflow of tests
see-quick 55fcd5f
try
see-quick 42a8dd1
a few leftover and ready to review again
see-quick File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,64 @@ | ||
| name: Run Performance Report | ||
| description: Downloads artifacts, generates performance report, and posts results | ||
| inputs: | ||
| artifacts-pattern: | ||
| description: 'Pattern for performance artifacts to download' | ||
| required: false | ||
| default: 'performance-results-*' | ||
| artifacts-path: | ||
| description: 'Path where artifacts should be downloaded' | ||
| required: false | ||
| default: 'systemtest/target/performance-artifacts' | ||
| runs: | ||
| using: composite | ||
| steps: | ||
| - name: Download performance results | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| pattern: ${{ inputs.artifacts-pattern }} | ||
| path: ${{ inputs.artifacts-path }} | ||
| # Each artifact gets its own subdirectory (not merged) to preserve arch-specific results | ||
|
|
||
| - name: List downloaded artifacts | ||
| shell: bash | ||
| run: | | ||
| echo "Downloaded artifacts:" | ||
| find "${{ inputs.artifacts-path }}" -type f | head -n 20 | ||
|
|
||
| - name: Generate performance report | ||
| id: generate_report | ||
| uses: actions/github-script@v7 | ||
| env: | ||
| PERF_DIR: ${{ inputs.artifacts-path }} | ||
| with: | ||
| github-token: ${{ github.token }} | ||
| script: | | ||
| const { generatePerformanceReport } = require('./.github/actions/systemtests/run-perf-report/generate-report.js'); | ||
| const perfDir = process.env.PERF_DIR || 'systemtest/target/performance'; | ||
| const result = generatePerformanceReport(perfDir, core); | ||
| core.setOutput('has_results', result.has_results); | ||
| core.setOutput('summary', result.summary); | ||
| core.setOutput('timestamp', result.timestamp); | ||
|
|
||
| - name: Add performance report comment | ||
| if: ${{ steps.generate_report.outputs.has_results == 'true' }} | ||
| uses: ./.github/actions/utils/add-comment | ||
| with: | ||
| commentMessage: | | ||
| ${{ steps.generate_report.outputs.summary }} | ||
|
|
||
| - name: Add performance report to job summary | ||
| if: ${{ steps.generate_report.outputs.has_results == 'true' }} | ||
| shell: bash | ||
| env: | ||
| SUMMARY_CONTENT: ${{ steps.generate_report.outputs.summary }} | ||
| run: | | ||
| echo "$SUMMARY_CONTENT" >> "$GITHUB_STEP_SUMMARY" | ||
|
|
||
| - name: No results warning | ||
| if: ${{ steps.generate_report.outputs.has_results == 'false' }} | ||
| shell: bash | ||
| env: | ||
| WARNING_MESSAGE: "No performance results found in artifacts" | ||
| run: | | ||
| echo "::warning::$WARNING_MESSAGE" | ||
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is cool, maybe there could be some usage for common sts execution as well?