From 30ea9714aa73fd0a99079c6dfed677a4d9bad604 Mon Sep 17 00:00:00 2001
From: Dmitry Tantsur <dtantsur@protonmail.com>
Date: Wed, 22 Dec 2021 18:21:42 +0100
Subject: [PATCH] Change the default image to a DIB-built one

We should use production-friendly defaults. TinyIPA is not suitable for
real bare metal.

Change-Id: Ifcba626ba898bde443674beacd4417d2fe876bf4
---
 doc/source/user/troubleshooting.rst                    |  5 ++++-
 .../roles/bifrost-ironic-install/defaults/main.yml     |  2 +-
 releasenotes/notes/no-tinyipa-8d18f3b21dbb9fe9.yaml    | 10 ++++++++++
 scripts/test-bifrost.sh                                |  1 +
 4 files changed, 16 insertions(+), 2 deletions(-)
 create mode 100644 releasenotes/notes/no-tinyipa-8d18f3b21dbb9fe9.yaml

diff --git a/doc/source/user/troubleshooting.rst b/doc/source/user/troubleshooting.rst
index 12cc625c4..1779b2d14 100644
--- a/doc/source/user/troubleshooting.rst
+++ b/doc/source/user/troubleshooting.rst
@@ -162,7 +162,10 @@ Changing from TinyIPA to another IPA Image
 With-in the Newton cycle, the default IPA image for Bifrost was changed
 to TinyIPA, which is based on Tiny Core Linux. This has a greatly reduced
 boot time for testing, however should be expected to have less hardware
-support. If on a fresh install, or a re-install, you wish to change to
+support. In the Yoga cycle, the default image was changed to one based
+on CentOS.
+
+If on a fresh install, or a re-install, you wish to change to
 DIB-based or any other IPA image, you will need to take the following steps:
 
 #. Remove the existing IPA image ipa.kernel and ipa.initramfs.
diff --git a/playbooks/roles/bifrost-ironic-install/defaults/main.yml b/playbooks/roles/bifrost-ironic-install/defaults/main.yml
index 9be303170..9829fb7ce 100644
--- a/playbooks/roles/bifrost-ironic-install/defaults/main.yml
+++ b/playbooks/roles/bifrost-ironic-install/defaults/main.yml
@@ -70,7 +70,7 @@ firewalld_internal_zone: bifrost
 ipa_file_protocol: "http"
 
 ipa_upstream_release: "master"
-use_tinyipa: true
+use_tinyipa: false
 
 enable_uefi_ipxe: true
 ipxe_efi_binary: ipxe.efi
diff --git a/releasenotes/notes/no-tinyipa-8d18f3b21dbb9fe9.yaml b/releasenotes/notes/no-tinyipa-8d18f3b21dbb9fe9.yaml
new file mode 100644
index 000000000..595465044
--- /dev/null
+++ b/releasenotes/notes/no-tinyipa-8d18f3b21dbb9fe9.yaml
@@ -0,0 +1,10 @@
+---
+upgrade:
+  - |
+    TinyIPA (an IPA image based on TinyCoreLinux) is no longer used by default.
+    Instead, a CentOS image `published by the Ironic community
+    <https://tarballs.opendev.org/openstack/ironic-python-agent/dib/files/>`_
+    is used, unless ``use_tinyipa`` is set to ``true``.
+
+    The TinyIPA image is much lighter, but is not suitable for real bare metal
+    machines because of lack of drivers.
diff --git a/scripts/test-bifrost.sh b/scripts/test-bifrost.sh
index 648380f5c..fab429748 100755
--- a/scripts/test-bifrost.sh
+++ b/scripts/test-bifrost.sh
@@ -195,6 +195,7 @@ ${ANSIBLE} -vvvv \
     -i inventory/target \
     ${TEST_PLAYBOOK} \
     -e use_cirros=${USE_CIRROS} \
+    -e use_tinyipa=true \
     -e testing_user=${TESTING_USER} \
     -e test_vm_num_nodes=${TEST_VM_NUM_NODES} \
     -e inventory_dhcp=${INVENTORY_DHCP} \