potiuk commented on code in PR #60079:
URL: https://github.com/apache/airflow/pull/60079#discussion_r2659011980


##########
scripts/ci/move_docker_to_mnt.sh:
##########
@@ -17,25 +17,37 @@
 # under the License.
 function cleanup_runner {
     set -x
-    local target_docker_volume_location="/mnt/var-lib-docker"
-    echo "Checking free space!"
-    df -H
-    # This is faster than docker prune
-    echo "Stopping docker"
-    sudo systemctl stop docker
-    echo "Checking free space!"
-    df -H
-    echo "Cleaning docker"
-    sudo rm -rf /var/lib/docker
-    echo "Checking free space!"
-    df -H
-    echo "Mounting ${target_docker_volume_location} to /var/lib/docker"
-    sudo mkdir -p "${target_docker_volume_location}" /var/lib/docker
-    sudo mount --bind "${target_docker_volume_location}" /var/lib/docker
-    sudo chown -R 0:0 "${target_docker_volume_location}"
-    sudo systemctl start docker
     echo "Checking free space!"
     df -H
+    # Note:
+    # Disk layout on x86_64 (2026-01):
+    # - root (/) on /dev/sdb1 (75G)
+    # - an additional mount as /mnt with 75G on /dev/sda1
+    # It seems that arm64 runners have
+    # - root (/) on /dev/sda1 (75G)
+    # - An un-used nvme with 220G
+    # Hence we only move docker to /mnt on x86_64 where we have a separate 
/mnt mount
+    # If we get short on disk space on arm64 as well we need to revisit this 
logic
+    # and make the idle nvme being used as well.
+    if uname -i|grep -q x86_64; then

Review Comment:
   Let's see if it works for "all" cases - I would rather check for filesystem 
presence, in case there is a bigger variation of runners, but let's see :) 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to