Skip to content

Conversation

@alicefr
Copy link
Contributor

@alicefr alicefr commented Feb 12, 2026

The Kernel Samepage Merging should help in reducing the memory footprint for the tests. See:
https://kubevirt.io/user-guide/cluster_admin/ksm/

The Kernel Samepage Merging should help in reducing the memory footprint
for the test. See:
  https://kubevirt.io/user-guide/cluster_admin/ksm/

Signed-off-by: Alice Frosi <afrosi@redhat.com>
@alicefr
Copy link
Contributor Author

alicefr commented Feb 12, 2026

When the CI is more stable, we could try to run some of the tests in parallel and the KSM can help in reducing the memory consumption

@alicefr
Copy link
Contributor Author

alicefr commented Feb 12, 2026

/cc @yalzhang

@openshift-ci openshift-ci bot requested a review from yalzhang February 12, 2026 12:25
Copy link
Contributor

@Jakob-Naucke Jakob-Naucke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm open to merging this in any case, but have you compared actual memory usage by any chance? If it allows to parallelize tests that OOMed previously it would be very useful

@openshift-ci
Copy link

openshift-ci bot commented Feb 12, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: alicefr, Jakob-Naucke

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the lgtm label Feb 12, 2026
@openshift-ci
Copy link

openshift-ci bot commented Feb 12, 2026

@alicefr: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/operator-lifecycle-verify 4b1b2df link true /test operator-lifecycle-verify

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@alicefr
Copy link
Contributor Author

alicefr commented Feb 12, 2026

I'm open to merging this in any case, but have you compared actual memory usage by any chance? If it allows to parallelize tests that OOMed previously it would be very useful

No I haven't, but it merges and shares several pages. So, it should help decreasing the memory footprint. I can try to come up with the memory saving on the next run of the tests

@alicefr
Copy link
Contributor Author

alicefr commented Feb 12, 2026

With the page sharing and letting running 4 VMs in parallel with 4GB memory for 2 min, we save around 2172 MB. I calculated with this command echo "$(($(cat /sys/kernel/mm/ksm/pages_sharing) * 4096 / 1024 / 1024)) MB". At the beginning, the sharing isn't that much but if the VM take a while for running, then the merging becomes significantly

@alicefr alicefr merged commit 4d69a38 into trusted-execution-clusters:main Feb 12, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants