Skip to content

Commit

Permalink
Launch the test VM off a hugepages-enabled flavor
Browse files Browse the repository at this point in the history
Signed-off-by: Bohdan Dobrelia <[email protected]>
  • Loading branch information
bogdando committed Nov 6, 2024
1 parent 7065044 commit bef1c74
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 2 deletions.
5 changes: 4 additions & 1 deletion docs_dev/assemblies/development_environment.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -145,8 +145,9 @@ To use Wallaby content instead, run the following:
[,bash]
----
cd ~/install_yamls/devsetup
make standalone
EDPM_CONFIGURE_HUGEPAGES=false make standalone <1>
----
<1> To configure the host for a simplistic hugepages setup, set `EDPM_CONFIGURE_HUGEPAGES=true`.

To deploy using TLS everywhere enabled, instead run:

Expand Down Expand Up @@ -231,12 +232,14 @@ Create a test VM instance with a test volume attachement:
cd ~/data-plane-adoption
export CINDER_VOLUME_BACKEND_CONFIGURED=true <1>
export CINDER_BACKUP_BACKEND_CONFIGURED=true
export EDPM_CONFIGURE_HUGEPAGES=false <2>
export OPENSTACK_COMMAND="ssh -i ~/install_yamls/out/edpm/ansibleee-ssh-key-id_rsa [email protected] OS_CLOUD=standalone openstack"
OS_CLOUD_IP=192.168.122.100 OS_CLOUD_NAME=standalone \
bash tests/roles/development_environment/files/pre_launch.bash
----
<1> Use `CINDER_*_BACKEND_CONFIGURED=false`, if Cinder Volume or Backup services' storage backends have been not configured for the source cloud,
or won't be configured for the target cloud. That might be a valid case for some developement setups, but not for a production scenarios.
<2> To configure a flavor with hugepages set `EDPM_CONFIGURE_HUGEPAGES=true`.

This also creates a test Cinder volume, a backup from it, and a snapshot of it.

Expand Down
1 change: 1 addition & 0 deletions tests/roles/development_environment/defaults/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ edpm_privatekey_path: ~/install_yamls/out/edpm/ansibleee-ssh-key-id_rsa
run_pre_adoption_validation: true
os_cloud_name: standalone
source_osp_ssh_user: root
use_hugepages: false
# Supported storage backends for Cinder of the source cloud
# must match those configured for the target cloud
supported_volume_backends: []
Expand Down
6 changes: 5 additions & 1 deletion tests/roles/development_environment/files/pre_launch.bash
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,12 @@ ${BASH_ALIASES[openstack]} image show cirros || \
${BASH_ALIASES[openstack]} image create --container-format bare --disk-format $DISK_FORMAT cirros < /tmp/$RAW

# Create flavor
HPARGS=" "
if [ "${EDPM_CONFIGURE_HUGEPAGES:-false}" = "true" ] ; then
HPARGS="set huge_pages --property hw:mem_page_size=2MB"
fi
${BASH_ALIASES[openstack]} flavor show m1.small || \
${BASH_ALIASES[openstack]} flavor create --ram 512 --vcpus 1 --disk 1 --ephemeral 1 m1.small
${BASH_ALIASES[openstack]} flavor create $HPARGS --ram 512 --vcpus 1 --disk 1 --ephemeral 1 m1.small

# Create networks
${BASH_ALIASES[openstack]} network show private || ${BASH_ALIASES[openstack]} network create private --share
Expand Down
1 change: 1 addition & 0 deletions tests/roles/development_environment/tasks/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
cmd: |
{{ shell_header }}
export OPENSTACK_COMMAND="{{ openstack_command }}"
export EDPM_CONFIGURE_HUGEPAGES={{ use_hugepages | string | lower }}
export CINDER_VOLUME_BACKEND_CONFIGURED={{ cinder_volume_backend_configured | string | lower }}
export CINDER_BACKUP_BACKEND_CONFIGURED={{ cinder_backup_backend_configured | string | lower }}
{{ lookup('ansible.builtin.file', prelaunch_test_instance_script) }}
Expand Down
1 change: 1 addition & 0 deletions tests/vars.sample.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ supported_backup_backends: #CUSTOMIZE_THIS

# Whether the adopted node will host compute services
compute_adoption: true
use_hugepages: false

# Where perform or not telemetry installation during adoption
telemetry_adoption: true
Expand Down

0 comments on commit bef1c74

Please sign in to comment.