Skip to content

sap_vm_provision/kubevirt_vm: Make usage of hugepages configurable #147

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.

Sign up for GitHub

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

Merged
newkit merged 10 commits into from
Jan 12, 2026
Merged

sap_vm_provision/kubevirt_vm: Make usage of hugepages configurable #147

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
d8dac93
make VM memory configurable to use hugepages or not
Dec 24, 2025
e1b1aa3
improved explanation of what hugepages can be set to
Jan 5, 2026
88496df
renamed __sap_vm_provision_register_memory_definition -> __sap_vm_pro…
Jan 7, 2026
6696b6e
changed order of CNV HyperConverged instance creation and canged webh…
Dec 23, 2025
c9ec05e
renamed var hugepages -> kubevirt_vm_memory_hypervisor_hugepages
Jan 8, 2026
f40d04a
Made huge pages comments clearer and added code for checking of inval…
Jan 8, 2026
65a54d4
fixed spelling error
Jan 8, 2026
95f73f6
replaced meta: end_host with fail
Jan 8, 2026
6a081bd
use compact formatting for fail task when cpu/hugepage config is wrong
Jan 8, 2026
9b77211
Updated error message
Jan 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions roles/sap_vm_provision/defaults/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -782,6 +782,7 @@ sap_vm_provision_kubevirt_vm_host_specifications_dictionary:
kubevirt_vm_cpu_smt: 2
kubevirt_vm_cpu_cores: 2
kubevirt_vm_memory_gib: 24
hugepages: 1Gi # false | 1Gi | <hugepage size>
sap_system_type: project_dev # project_dev, project_tst, project_prd
sap_host_type: hana_primary # hana_primary, hana_secondary, anydb_primary, anydb_secondary, nwas_ascs, nwas_ers, nwas_pas, nwas_aas
# Provide either an existing PVC or a URL for an OS image
Expand Down
20 changes: 16 additions & 4 deletions roles/sap_vm_provision/tasks/platform_ansible/kubevirt_vm/execute_provision.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -217,6 +217,21 @@
{%- endfor %}
{{ networks }}
- name: Set fact for memory definition
ansible.builtin.set_fact:
__sap_vm_provision_register_memory_definition: >-
{{
{
'guest': __sap_vm_provision_register_vm_config.kubevirt_vm_memory_gib ~ 'Gi'
}
| combine(
(__sap_vm_provision_register_vm_config.hugepages | default(true)) | ternary(
{'hugepages': {'pageSize': (__sap_vm_provision_register_vm_config.hugepages | default('1Gi')) }},
{}
)
)
}}
- name: Set fact for VM deploy config
ansible.builtin.set_fact:
__sap_vm_provision_register_vm_deploy_config:
Expand Down Expand Up @@ -244,10 +259,7 @@
cores: "{{ __sap_vm_provision_register_vm_config.kubevirt_vm_cpu_cores }}"
threads: "{{ __sap_vm_provision_register_vm_config.kubevirt_vm_cpu_smt }}"

memory:
guest: "{{ __sap_vm_provision_register_vm_config.kubevirt_vm_memory_gib }}Gi"
hugepages:
pageSize: 1Gi
memory: "{{ __sap_vm_provision_register_memory_definition }}"

- name: Apply CPU performance settings
ansible.builtin.set_fact:
Expand Down