Merge "Redfish virtual media boot in UEFI"

This commit is contained in:
Zuul 2020-11-03 11:37:19 +00:00 committed by Gerrit Code Review
commit 5b654dfe3d
16 changed files with 119 additions and 1 deletions

View File

@ -17,3 +17,4 @@
NOAUTH_MODE: "{{ noauth_mode | default(false) | bool | lower }}"
ENABLE_TLS: "{{ enable_tls | default(false) | bool | lower }}"
ENABLE_PROMETHEUS_EXPORTER: "{{ enable_prometheus_exporter | default(false) | bool | lower }}"
USE_VMEDIA: "{{ use_vmedia | default(false) | bool | lower }}"

View File

@ -96,6 +96,8 @@ ipa_ramdisk_upstream_checksum_algo: "sha256"
ipa_ramdisk_upstream_checksum_url: "{{ ipa_ramdisk_upstream_url }}.{{ ipa_ramdisk_upstream_checksum_algo }}"
deploy_image_filename: "deployment_image.qcow2"
deploy_image: "{{http_boot_folder}}/{{deploy_image_filename}}"
esp_image: "{{ http_boot_folder }}/esp.img"
esp_image_url: "{{ ipa_file_protocol }}://{{ internal_ip }}:{{ file_url_port }}/esp.img"
# Use cirros instead of building an image via diskimage-builder
use_cirros: false
# Download IPA by default

View File

@ -0,0 +1,3 @@
---
efi_distro: centos
grub_efi_binary: /boot/efi/EFI/centos/grubx64.efi

View File

@ -6,6 +6,9 @@ ironic_rootwrap_dir: /usr/local/bin/
nginx_user: www-data
mysql_service_name: mysql
tftp_service_name: tftpd-hpa
efi_distro: debian
grub_efi_binary: /usr/lib/grub/x86_64-efi-signed/grubx64.efi.signed
shim_efi_binary: /usr/lib/shim/shimx64.efi.signed
required_packages:
- mariadb-server
- python-dev
@ -35,6 +38,8 @@ required_packages:
- dnsmasq
- apache2-utils
- isolinux
- grub-efi-amd64-signed
- shim-signed
# NOTE(TheJulia): The above entry for dnsmasq must be the last entry in the
# package list as the installation causes name resolution changes that can
# temporarily block packages following it while the system is being

View File

@ -6,11 +6,14 @@ ironic_rootwrap_dir: /usr/bin/
nginx_user: nginx
mysql_service_name: mariadb
tftp_service_name: tftp
efi_distro: fedora
grub_efi_binary: /boot/efi/EFI/fedora/grubx64.efi
required_packages:
- dnsmasq
- firewalld
- gcc
- genisoimage
- grub2-efi-x64
- httpd-tools
- ipmitool
- ipxe-bootimgs
@ -31,6 +34,7 @@ required_packages:
- python3-PyMySQL
- qemu-img
- qemu-kvm
- shim-x64
- socat
- syslinux-nonlinux
- tftp-server

View File

@ -0,0 +1,3 @@
---
efi_distro: rhel
grub_efi_binary: /boot/efi/EFI/rhel/grubx64.efi

View File

@ -7,6 +7,7 @@ ironic_rootwrap_dir: /usr/bin/
nginx_user: nginx
mysql_service_name: mariadb
tftp_service_name: tftp
shim_efi_binary: /boot/efi/EFI/BOOT/BOOTX64.EFI
required_packages:
- mariadb-server
- dnsmasq
@ -38,6 +39,8 @@ required_packages:
- python3-firewall
- httpd-tools
- syslinux-nonlinux
- grub2-efi-x64
- shim-x64
iscsi_required_packages:
- iscsi-initiator-utils
- gdisk

View File

@ -7,6 +7,11 @@ ironic_rootwrap_dir: /usr/bin/
nginx_user: nginx
mysql_service_name: mysql
tftp_service_name: tftp
# FIXME(dtantsur): this may be incorrect, but I cannot figure out the correct
# path from just looking at various sources.
efi_distro: BOOT
grub_efi_binary: /usr/share/efi/x86_64/grub.efi
shim_efi_binary: /usr/share/efi/x86_64/shim.efi
required_packages:
- python-selinux
- mariadb-server
@ -40,6 +45,8 @@ required_packages:
- python-PyMySQL
- apache2-utils
- syslinux
- grub2-x86_64-efi
- shim
iscsi_required_packages:
- open-iscsi
- gptfdisk

View File

@ -1,5 +1,6 @@
---
init_dest_dir: /lib/systemd/system/
efi_distro: ubuntu
required_packages:
- mariadb-server
- python3-dev
@ -29,6 +30,8 @@ required_packages:
- dnsmasq
- apache2-utils
- isolinux
- grub-efi-amd64-signed
- shim-signed
# NOTE(TheJulia): The above entry for dnsmasq must be the last entry in the
# package list as the installation causes name resolution changes that can
# temporarily block packages following it while the system is being

View File

@ -222,7 +222,11 @@
args: '--config-file /etc/ironic/ironic.conf'
- name: "Create and populate /tftpboot"
include: create_tftpboot.yml
import_tasks: create_tftpboot.yml
- name: "Create an ESP image"
import_tasks: create_esp.yml
- name: "Setup Inventory Hosts Directory"
file:
path: "/etc/dnsmasq.d/bifrost.hosts.d"

View File

@ -0,0 +1,55 @@
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
# implied.
# See the License for the specific language governing permissions and
# limitations under the License.
---
- name: "Create a temporary directory for mounting ESP"
tempfile:
state: directory
register: esp_temp_dir
- name: "Create an empty ESP image"
command: dd if=/dev/zero of={{ esp_image }} bs=4096 count=1024
- name: "Initialize ESP filesystem"
command: mkfs.fat -s 4 -r 512 -S 4096 {{ esp_image }}
- name: "Mount ESP filesystem"
command: mount {{ esp_image }} {{ esp_temp_dir.path }} # noqa 303
- block:
- name: "Create ESP boot directory"
file:
path: "{{ esp_temp_dir.path }}/EFI/BOOT"
state: directory
- name: "Copy the shim image to ESP"
copy:
src: "{{ shim_efi_binary }}"
remote_src: true
dest: "{{ esp_temp_dir.path }}/EFI/BOOT/BOOTX64.efi"
- name: "Copy the grub2 image to ESP"
copy:
src: "{{ grub_efi_binary }}"
remote_src: true
dest: "{{ esp_temp_dir.path }}/EFI/BOOT/GRUBX64.efi"
always:
- name: "Unmount ESP filesystem"
command: umount {{ esp_temp_dir.path }}
- name: "Ensure ESP image is readable"
file:
path: "{{ esp_image }}"
owner: ironic
group: ironic
mode: 0644

View File

@ -19,6 +19,10 @@ enabled_deploy_interfaces = {{ enabled_deploy_interfaces }}
enabled_hardware_types = {{ enabled_hardware_types }}
{% if default_boot_interface is defined %}
default_boot_interface = {{ default_boot_interface }}
{% endif %}
default_resource_class = {{ default_resource_class }}
rpc_transport = json-rpc
@ -38,6 +42,8 @@ log_dir = {{ ironic_log_dir }}
isolinux_bin = {{ isolinux_bin_path }}
grub_config_path = EFI/{{ efi_distro }}/grub.cfg
{% if enable_tls | bool %}
[api]
enable_ssl_api = True
@ -92,6 +98,7 @@ deploy_kernel = {{ ipa_kernel_url }}
deploy_ramdisk = {{ ipa_ramdisk_url }}
rescue_kernel = {{ ipa_kernel_url }}
rescue_ramdisk = {{ ipa_ramdisk_url }}
bootloader = {{ esp_image_url }}
{% if enable_prometheus_exporter | bool %}
send_sensor_data = true
send_sensor_data_for_undeployed_nodes = {{ sensor_data_undeployed_nodes }}

View File

@ -0,0 +1,4 @@
---
features:
- |
Adds support for Redfish virtual media in UEFI mode.

View File

@ -15,6 +15,7 @@ CLI_TEST=${CLI_TEST:-false}
BOOT_MODE=${BOOT_MODE:-}
ENABLE_TLS=${ENABLE_TLS:-false}
ENABLE_PROMETHEUS_EXPORTER=${ENABLE_PROMETHEUS_EXPORTER:-false}
USE_VMEDIA=${USE_VMEDIA:-false}
# Set defaults for ansible command-line options to drive the different
# tests.
@ -107,6 +108,13 @@ if [[ -n "$BOOT_MODE" ]]; then
VM_SETUP_EXTRA+=" -e default_boot_mode=$BOOT_MODE"
fi
if [ ${USE_VMEDIA} = "true" ]; then
TEST_VM_NODE_DRIVER=redfish
CLOUD_CONFIG+=" -e default_boot_interface=redfish-virtual-media"
# The default won't work for other hardware types
CLOUD_CONFIG+=" -e enabled_hardware_types=redfish"
fi
logs_on_exit() {
$SCRIPT_HOME/collect-test-info.sh
}

View File

@ -109,6 +109,14 @@
test_driver: redfish
enable_prometheus_exporter: true
- job:
name: bifrost-integration-redfish-vmedia-uefi-ubuntu-focal
parent: bifrost-integration-tinyipa-ubuntu-focal
vars:
boot_mode: uefi
test_driver: redfish
use_vmedia: true
- job:
name: bifrost-cli-ubuntu-focal
parent: bifrost-integration-tinyipa-ubuntu-focal

View File

@ -12,6 +12,7 @@
- bifrost-integration-tinyipa-centos-8
- bifrost-integration-tinyipa-keystone-ubuntu-focal
- bifrost-integration-tinyipa-keystone-centos-8
- bifrost-integration-redfish-vmedia-uefi-ubuntu-focal
# Non-voting jobs
- bifrost-integration-tinyipa-ubuntu-bionic:
voting: false