This is an automated email from the ASF dual-hosted git repository.

lidavidm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-adbc.git


The following commit(s) were added to refs/heads/main by this push:
     new c94dd8e3c ci: free up more disk space in verification, fix test on 
Windows (#3814)
c94dd8e3c is described below

commit c94dd8e3c55f8391b3b9548972c8075b2330dd1b
Author: David Li <[email protected]>
AuthorDate: Wed Dec 17 07:39:08 2025 +0900

    ci: free up more disk space in verification, fix test on Windows (#3814)
    
    - Use CMake generator expressions instead of making assumptions about 
filepaths
    - Free up even more disk space so validation can pass
---
 .github/workflows/nightly-verify.yml | 40 +++++++++++++++++++++++++++++++-----
 .github/workflows/verify.yml         | 21 ++++++++++++++++---
 c/driver_manager/CMakeLists.txt      |  4 ++--
 ci/scripts/verify_ubuntu.sh          |  2 +-
 4 files changed, 56 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/nightly-verify.yml 
b/.github/workflows/nightly-verify.yml
index 392171fd3..b9fabf608 100644
--- a/.github/workflows/nightly-verify.yml
+++ b/.github/workflows/nightly-verify.yml
@@ -151,6 +151,25 @@ jobs:
         run: |
           sudo sysctl vm.mmap_rnd_bits=28
 
+      - name: Clean up disk space
+        if: runner.os == 'Linux'
+        run: |
+          # These tools use a lot of disk space, free up some space
+          # https://github.com/actions/runner-images/issues/2840
+          sudo rm -rf \
+            "$AGENT_TOOLSDIRECTORY" \
+            /opt/google/chrome \
+            /opt/microsoft/msedge \
+            /opt/microsoft/powershell \
+            /opt/pipx \
+            /usr/local/julia* \
+            /usr/local/lib/android \
+            /usr/local/lib/node_modules \
+            /usr/local/share/chromium \
+            /usr/local/share/powershell \
+            /usr/share/swift
+          df -h /
+
       - name: Verify
         if: matrix.os != 'windows-latest'
         env:
@@ -162,9 +181,6 @@ jobs:
           VERBOSE: "1"
           VERIFICATION_MOCK_DIST_DIR: ${{ github.workspace }}
         run: |
-          # Rust uses a lot of disk space, free up some space
-          # https://github.com/actions/runner-images/issues/2840
-          sudo rm -rf "$AGENT_TOOLSDIRECTORY"
           # Required for macOS
           export CXXFLAGS="${CXXFLAGS} -D_LIBCPP_DISABLE_AVAILABILITY"
           echo "CXXFLAGS=${CXXFLAGS}"
@@ -243,9 +259,23 @@ jobs:
 
       - name: Clean up disk space
         run: |
-          # Rust uses a lot of disk space, free up some space
+          # These tools use a lot of disk space, free up some space
           # https://github.com/actions/runner-images/issues/2840
-          sudo rm -rf "$AGENT_TOOLSDIRECTORY"
+          sudo rm -rf \
+            "$AGENT_TOOLSDIRECTORY" \
+            /opt/google/chrome \
+            /opt/microsoft/msedge \
+            /opt/microsoft/powershell \
+            /opt/pipx \
+            /usr/lib/mono \
+            /usr/local/julia* \
+            /usr/local/lib/android \
+            /usr/local/lib/node_modules \
+            /usr/local/share/chromium \
+            /usr/local/share/powershell \
+            /usr/share/dotnet \
+            /usr/share/swift
+          df -h /
 
       - name: Verify
         env:
diff --git a/.github/workflows/verify.yml b/.github/workflows/verify.yml
index 4dc4aefec..baa053564 100644
--- a/.github/workflows/verify.yml
+++ b/.github/workflows/verify.yml
@@ -97,6 +97,24 @@ jobs:
         if: matrix.os == 'ubuntu-latest'
         run: |
           sudo sysctl vm.mmap_rnd_bits=28
+      - name: Clean up disk space
+        if: runner.os == 'Linux'
+        run: |
+          # These tools use a lot of disk space, free up some space
+          # https://github.com/actions/runner-images/issues/2840
+          sudo rm -rf \
+            "$AGENT_TOOLSDIRECTORY" \
+            /opt/google/chrome \
+            /opt/microsoft/msedge \
+            /opt/microsoft/powershell \
+            /opt/pipx \
+            /usr/local/julia* \
+            /usr/local/lib/android \
+            /usr/local/lib/node_modules \
+            /usr/local/share/chromium \
+            /usr/local/share/powershell \
+            /usr/share/swift
+          df -h /
       - name: Verify
         if: matrix.os != 'windows-latest'
         env:
@@ -107,9 +125,6 @@ jobs:
           USE_CONDA: "1"
           VERBOSE: "1"
         run: |
-          # Rust uses a lot of disk space, free up some space
-          # https://github.com/actions/runner-images/issues/2840
-          sudo rm -rf "$AGENT_TOOLSDIRECTORY"
           # Required for macOS
           export CXXFLAGS="${CXXFLAGS} -D_LIBCPP_DISABLE_AVAILABILITY"
           echo "CXXFLAGS=${CXXFLAGS}"
diff --git a/c/driver_manager/CMakeLists.txt b/c/driver_manager/CMakeLists.txt
index 25fb6fe8c..35f89ac84 100644
--- a/c/driver_manager/CMakeLists.txt
+++ b/c/driver_manager/CMakeLists.txt
@@ -110,11 +110,11 @@ if(ADBC_BUILD_TESTS)
                    adbc_driver_no_entrypoint)
 
   target_compile_definitions(adbc-driver-manager-test
-                             PRIVATE 
ADBC_DRIVER_MANAGER_ENTRYPOINT_TEST_LIB="${CMAKE_BINARY_DIR}/driver_manager/${CMAKE_SHARED_LIBRARY_PREFIX}adbc_driver_entrypoint${CMAKE_SHARED_LIBRARY_SUFFIX}"
+                             PRIVATE 
ADBC_DRIVER_MANAGER_ENTRYPOINT_TEST_LIB="$<TARGET_FILE:adbc_driver_entrypoint>"
   )
 
   target_compile_definitions(adbc-driver-manager-test
-                             PRIVATE 
ADBC_DRIVER_MANAGER_NO_ENTRYPOINT_TEST_LIB="${CMAKE_BINARY_DIR}/driver_manager/${CMAKE_SHARED_LIBRARY_PREFIX}adbc_driver_no_entrypoint${CMAKE_SHARED_LIBRARY_SUFFIX}"
+                             PRIVATE 
ADBC_DRIVER_MANAGER_NO_ENTRYPOINT_TEST_LIB="$<TARGET_FILE:adbc_driver_no_entrypoint>"
   )
 
   if(ADBC_DRIVER_SQLITE)
diff --git a/ci/scripts/verify_ubuntu.sh b/ci/scripts/verify_ubuntu.sh
index 2ec249035..8daabc9fe 100755
--- a/ci/scripts/verify_ubuntu.sh
+++ b/ci/scripts/verify_ubuntu.sh
@@ -25,7 +25,7 @@
 set -euo pipefail
 
 : ${JDK:=21}
-: ${MAVEN:=3.9.11}
+: ${MAVEN:=3.9.12}
 
 main() {
     local -r source_dir="${1}"

Reply via email to