Skip to content

[WIP] feat: naive performance test and handling results storing#3140

Draft
csviri wants to merge 26 commits intonextfrom
perf-test
Draft

[WIP] feat: naive performance test and handling results storing#3140
csviri wants to merge 26 commits intonextfrom
perf-test

Conversation

@csviri
Copy link
Collaborator

@csviri csviri commented Jan 28, 2026

Stores performance tests as artifacts and compares the results with released version performance test results.

(plan to add more sophisticated performance tests with JMH: https://github.com/openjdk/jmh )

Signed-off-by: Attila Mészáros a_meszaros@apple.com

csviri and others added 14 commits January 15, 2026 15:06
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Steve Hawkins <shawkins@redhat.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
simplified the javadoc

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Chris Laprun <metacosm@gmail.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
- Adds utility that provides methods to update resources using comparable resource versions
- Integrates this utility to the core of the framework (thus simplifying `ReconciliationDispatcher`)
  -  note that this introduces a change in the behavior for the `UpdateControl.patchStatus` (and others), since it won't trigger the reconiliation for the event in that update.
-  Renames former `ReconcilerUtils` to `ReconcilerUtilsInternal`, this is breaking but that utils was never advertised for non-internal usage 
- Includes also fixes for ControllerEventSource event filtering
- Improves TemporaryResourceCache event filtering algorithm

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Steve Hawkins <shawkins@redhat.com>
Co-authored-by: Steve Hawkins <shawkins@redhat.com>
)

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
This should not be user facing. At least not in any obvious scenerio.

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 28, 2026
@csviri csviri changed the title feat: naive performance test and handling results storing [WIP] feat: naive performance test and handling results storing Jan 28, 2026
csviri and others added 6 commits February 2, 2026 09:36
…om Context (#3142)

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Chris Laprun <metacosm@gmail.com>
Co-authored-by: Chris Laprun <metacosm@gmail.com>
#3146)

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Stores performance tests as artifacts and compares the results with released
version performance tests

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Chris Laprun <metacosm@gmail.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@csviri csviri linked an issue Feb 3, 2026 that may be closed by this pull request
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Performance Test Results

Naive performance test

Duration (ms) Max Memory (GB) Processors Parameters
1219 3.90 4 -
1235 3.91 4 -
1098 3.90 4 -
1147 3.91 4 -
1190 3.91 4 -
1144 3.90 4 -
1213 3.91 4 -
1180 3.91 4 -
1103 3.91 4 -
1184 3.90 4 -
970 3.91 4 -
1165 3.91 4 -
1205 3.91 4 -
1217 3.91 4 -
1102 3.91 4 -

@csviri
Copy link
Collaborator Author

csviri commented Feb 3, 2026

TODO create sample with async profile

@csviri
Copy link
Collaborator Author

csviri commented Feb 6, 2026

@metacosm @xstefank I was thinking more about this, and probably would be nicer to store data on a branch rather than in artifact. It would give us more space, and we would not have the 90-day retention limit.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 7, 2026
@openshift-merge-robot
Copy link

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Performance Tests

4 participants