Skip to content

Commit

Permalink
use ufei default instead of bios
Browse files Browse the repository at this point in the history
Apart from recent potential issues related to using bios, uefi
default makes makes more sense in general as for a long time
all enterprise hardware is either using uefi or secure-uefi.

This commit:
  - switched the default libvirt boot loader firmware from bios
    to the more modern uefi
  • Loading branch information
Rozzii committed Dec 18, 2023
1 parent 5808ae4 commit 8f0b5df
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion config_example.sh
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@
#
# Set libvirt firmware and BMC bootMode
# Choose "legacy" (bios), "UEFI", or "UEFISecureBoot"
# Defaults to legacy for ipv4, UEFI for ipv6
# Defaults to uefi
# export BOOT_MODE="UEFI"

# Select the Cluster API provider Metal3 version
Expand Down
1 change: 1 addition & 0 deletions lib/common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ fi
source "${CONFIG}"

# Set variables
export BOOT_MODE="${BOOT_MODE:-UEFI}"
export MARIADB_HOST="mariaDB"
export MARIADB_HOST_IP="127.0.0.1"
# Additional DNS
Expand Down
1 change: 0 additions & 1 deletion lib/network.sh
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ if [[ "${BARE_METAL_PROVISIONER_SUBNET_IPV6_ONLY}" == "true" ]]; then
export BOOT_MODE="${BOOT_MODE:-UEFI}"
export BARE_METAL_PROVISIONER_NETWORK="${BARE_METAL_PROVISIONER_NETWORK:-fd2e:6f44:5dd8:b856::/64}"
else
export BOOT_MODE="${BOOT_MODE:-legacy}"
export BARE_METAL_PROVISIONER_NETWORK="${BARE_METAL_PROVISIONER_NETWORK:-172.22.0.0/24}"
fi

Expand Down
2 changes: 1 addition & 1 deletion vars.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ assured that they are persisted.
| BMO_ROLLOUT_WAIT | Number of minutes(Until max 10m that is the default value of deployment.spec.progressDeadlineSeconds) to wait for BMO rollout | | 5 |
| BMC_DRIVER | Set the BMC driver | "ipmi", "redfish", "redfish-virtualmedia" | "mixed" |
| BMORELEASEBRANCH | BMO Release branch | "main", "release-0.4" | Set via jjb for CI, for local dev it gets default value based on CAPM3 branch |
| BOOT_MODE | Set libvirt firmware and BMH bootMode | "legacy", "UEFI", "UEFISecureBoot" | "legacy" |
| BOOT_MODE | Set libvirt firmware and BMH bootMode | "legacy", "UEFI", "UEFISecureBoot" | "UEFI" |
| IMAGE_OS | OS of the image to boot the nodes from, overriden by IMAGE\_\* if set | "centos", "cirros", "FCOS", "ubuntu", "flatcar" | "centos" |
| IMAGE_NAME | Image for target hosts deployment | | "CENTOS_9_NODE_IMAGE_K8S_${KUBERNETES_VERSION}.qcow2" |
| IMAGE_LOCATION | Location of the image to download | | https://artifactory.nordix.org/artifactory/metal3/images/${KUBERNETES_VERSION} |
Expand Down
2 changes: 1 addition & 1 deletion vm-setup/roles/libvirt/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ libvirt_diskbus: scsi
libvirt_cdrombus: sata
libvirt_arch: x86_64
libvirt_cpu_mode: host-model
libvirt_firmware: bios
libvirt_firmware: uefi
libvirt_secure_boot: false

# Where to log serial console output
Expand Down

0 comments on commit 8f0b5df

Please sign in to comment.