From 01cbbbddd2257668342af49f2911c827482a7992 Mon Sep 17 00:00:00 2001 From: Ed Santiago Date: Tue, 26 Mar 2024 09:51:19 -0600 Subject: [PATCH] Make /tmp be tmpfs History dates back to PR #30, in 2020. Podman *really* needs /tmp to be tmpfs. Let's make it so. All the /tmp->/var/tmp changes are because Rawhide build failed on reboot, because (duh) /tmp does not survive reboot Signed-off-by: Ed Santiago --- .cirrus.yml | 2 +- IMG_SFX | 2 +- base_images/cloud.yml | 8 ++++---- base_images/fedora_base-setup.sh | 6 ------ cache_images/cloud.yml | 10 +++++----- cache_images/fedora_packaging.sh | 3 ++- image_builder/gce.yml | 6 +++--- lib.sh | 2 +- 8 files changed, 17 insertions(+), 22 deletions(-) diff --git a/.cirrus.yml b/.cirrus.yml index 95ef7f58..982d99f4 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -102,7 +102,7 @@ container_images_task: &container_images REG_PASSWORD: ENCRYPTED[255ec05057707c20237a6c7d15b213422779c534f74fe019b8ca565f635dba0e11035a034e533a6f39e146e7435d87b5] script: ci/make_container_images.sh; package_cache: &package_cache - folder: "/tmp/automation_images_tmp/.cache/**" + folder: "/var/tmp/automation_images_tmp/.cache/**" fingerprint_key: "${TARGET_NAME}-cache-version-1" diff --git a/IMG_SFX b/IMG_SFX index 3f0caa55..b59e0be5 100644 --- a/IMG_SFX +++ b/IMG_SFX @@ -1 +1 @@ -20240320t153921z-f39f38d13 +20240326t173017z-f39f38d13 diff --git a/base_images/cloud.yml b/base_images/cloud.yml index 083551c7..f9803c38 100644 --- a/base_images/cloud.yml +++ b/base_images/cloud.yml @@ -187,23 +187,23 @@ provisioners: # Debian images come bundled with GCE integrations provisioned - type: 'shell' inline: - 'set -e' - - 'mkdir -p /tmp/automation_images' + - 'mkdir -p /var/tmp/automation_images' - type: 'file' source: '{{ pwd }}/' - destination: '/tmp/automation_images/' + destination: '/var/tmp/automation_images/' - except: ['debian'] type: 'shell' inline: - 'set -e' - - '/bin/bash /tmp/automation_images/base_images/fedora_base-setup.sh' + - '/bin/bash /var/tmp/automation_images/base_images/fedora_base-setup.sh' - only: ['debian'] type: 'shell' inline: - 'set -e' - - 'env DEBIAN_FRONTEND=noninteractive /bin/bash /tmp/automation_images/base_images/debian_base-setup.sh' + - 'env DEBIAN_FRONTEND=noninteractive /bin/bash /var/tmp/automation_images/base_images/debian_base-setup.sh' post-processors: # Must be double-nested to guarantee execution order diff --git a/base_images/fedora_base-setup.sh b/base_images/fedora_base-setup.sh index c00f7080..06c22042 100644 --- a/base_images/fedora_base-setup.sh +++ b/base_images/fedora_base-setup.sh @@ -95,10 +95,4 @@ if ! ((CONTAINER)); then /lib/$METADATA_SERVICE_PATH | $SUDO tee -a /etc/$METADATA_SERVICE_PATH fi -if [[ "$OS_RELEASE_ID" == "fedora" ]] && ((OS_RELEASE_VER>=33)); then - # Ref: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=783509 - echo "Disabling automatic /tmp (tmpfs) mount" - $SUDO systemctl mask tmp.mount -fi - finalize diff --git a/cache_images/cloud.yml b/cache_images/cloud.yml index 98829bfc..d8f87cf1 100644 --- a/cache_images/cloud.yml +++ b/cache_images/cloud.yml @@ -183,30 +183,30 @@ provisioners: - type: 'shell' inline: - 'set -e' - - 'mkdir -p /tmp/automation_images' + - 'mkdir -p /var/tmp/automation_images' - type: 'file' source: '{{ pwd }}/' - destination: "/tmp/automation_images" + destination: "/var/tmp/automation_images" - only: ['rawhide'] type: 'shell' expect_disconnect: true # VM will be rebooted at end of script inline: - 'set -e' - - '/bin/bash /tmp/automation_images/cache_images/rawhide_setup.sh' + - '/bin/bash /var/tmp/automation_images/cache_images/rawhide_setup.sh' - except: ['debian'] type: 'shell' inline: - 'set -e' - - '/bin/bash /tmp/automation_images/cache_images/fedora_setup.sh' + - '/bin/bash /var/tmp/automation_images/cache_images/fedora_setup.sh' - only: ['debian'] type: 'shell' inline: - 'set -e' - - 'env DEBIAN_FRONTEND=noninteractive /bin/bash /tmp/automation_images/cache_images/debian_setup.sh' + - 'env DEBIAN_FRONTEND=noninteractive /bin/bash /var/tmp/automation_images/cache_images/debian_setup.sh' post-processors: # This is critical for human-interaction. Copntents will be used diff --git a/cache_images/fedora_packaging.sh b/cache_images/fedora_packaging.sh index 02b888ef..7b3a5a8a 100644 --- a/cache_images/fedora_packaging.sh +++ b/cache_images/fedora_packaging.sh @@ -194,7 +194,8 @@ msg "Installing general build/test dependencies" bigto $SUDO dnf install -y "${INSTALL_PACKAGES[@]}" # 2024-03-18 new pasta, built today, not yet stable anywhere -timebomb 20240325 "package not yet in stable for fc38" +# 2024-03-25 still not stable +timebomb 20240330 "package not yet in stable" if [[ "$OS_RELEASE_VER" -le 41 ]]; then arch=$(uname -m) n=passt diff --git a/image_builder/gce.yml b/image_builder/gce.yml index dce74345..e6208d5f 100644 --- a/image_builder/gce.yml +++ b/image_builder/gce.yml @@ -45,16 +45,16 @@ provisioners: - type: 'shell' inline: - 'set -e' - - 'mkdir -p /tmp/automation_images' + - 'mkdir -p /var/tmp/automation_images' - type: 'file' source: '{{ pwd }}/' - destination: '/tmp/automation_images/' + destination: '/var/tmp/automation_images/' - type: 'shell' inline: - 'set -e' - - '/bin/bash /tmp/automation_images/image_builder/setup.sh' + - '/bin/bash /var/tmp/automation_images/image_builder/setup.sh' post-processors: # Must be double-nested to guarantee execution order diff --git a/lib.sh b/lib.sh index 3d38643e..eb86fa31 100644 --- a/lib.sh +++ b/lib.sh @@ -300,7 +300,7 @@ common_finalize() { $SUDO rm -rf /var/lib/cloud/instanc* $SUDO rm -rf /root/.ssh/* $SUDO rm -rf /etc/ssh/*key* - $SUDO rm -rf /tmp/* + $SUDO rm -rf /tmp/* /var/tmp/automation_images $SUDO rm -rf /tmp/.??* echo -n "" | $SUDO tee /etc/machine-id $SUDO sync