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 [email protected]

csviri and others added 14 commits January 15, 2026 15:06
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
simplified the javadoc

Signed-off-by: Attila Mészáros <[email protected]>
- 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 <[email protected]>
Signed-off-by: Steve Hawkins <[email protected]>
Co-authored-by: Steve Hawkins <[email protected]>
This should not be user facing. At least not in any obvious scenerio.

Signed-off-by: Attila Mészáros <[email protected]>
@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 <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Co-authored-by: Chris Laprun <[email protected]>
Stores performance tests as artifacts and compares the results with released
version performance tests

Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Chris Laprun <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
@csviri csviri linked an issue Feb 3, 2026 that may be closed by this pull request
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
Signed-off-by: Attila Mészáros <[email protected]>
@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

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Performance Tests

3 participants