From b9a02e9abc8b2f073a8fc97f13b5a3e9534a427f Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Mon, 21 Sep 2020 09:14:25 -0700 Subject: [PATCH] CI: Remove the build check for pre-build ramdisks only In the devstack plugin, we identify RAX hosts and change the settings slightly to help ensure CI job passage. This is because the machines are fully emulated as opposed to paravirtualized which causes a performance impact with our test VMs that we create. Since we can hit issues while scheduling on RAX in general, it only makes sense to also swap the job out for centos based IPA jobs that some how happen on to a RAX node. Since the CI jobs have largely been "de-typed" name wise, this should be fine. Change-Id: I63fc5de05c3975433c12ddd9ba21ebb676de2e6c (cherry picked from commit 4e8f664ea2d01fe13f55ed7cf3a424cd3c17d289) --- devstack/lib/ironic | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/devstack/lib/ironic b/devstack/lib/ironic index 1d48d54615..cc1fd2808f 100644 --- a/devstack/lib/ironic +++ b/devstack/lib/ironic @@ -258,7 +258,7 @@ fi # Prevent a case that will likely result in a failure. if [[ "$hostdomain" =~ "rax" ]]; then - if [[ "$IRONIC_RAMDISK_TYPE" == "dib" ]] && [[ "$IRONIC_BUILD_DEPLOY_RAMDISK" == "False" ]]; then + if [[ "$IRONIC_RAMDISK_TYPE" == "dib" ]]; then echo "** WARNING ** - DIB based IPA images have been defined, however we are running devstack on RAX VM. Due to virtualization constraints, we are automatically falling back to TinyIPA to ensure CI job passage." IRONIC_RAMDISK_TYPE="tinyipa" fi