apache kudu with source/tarball

Author: ranimandepudi

.github/workflows/test-kudu.yml

@@ -45,12 +45,12 @@ jobs:
           curl -fSL "$TARBALL_URL" -o "apache-kudu-${KUDU_VERSION}.tar.gz"
 
           if [ ! -s "apache-kudu-${KUDU_VERSION}.tar.gz" ]; then
-            echo "✗ tarball download failed or empty file"
+            echo "tarball download failed or empty file"
             echo "status=failed" >> $GITHUB_OUTPUT
             exit 1
           fi
 
-          echo "✓ tarball downloaded"
+          echo "tarball downloaded"
           echo "kudu_version=$KUDU_VERSION" >> $GITHUB_OUTPUT
           echo "status=passed" >> $GITHUB_OUTPUT
 
@@ -63,13 +63,13 @@ jobs:
           tar -xzf "apache-kudu-${KUDU_VERSION}.tar.gz"
 
           if [ ! -d "apache-kudu-${KUDU_VERSION}" ]; then
-            echo "✗ source directory apache-kudu-${KUDU_VERSION} not found after extract"
+            echo "source directory apache-kudu-${KUDU_VERSION} not found after extract"
             ls -R .
             echo "status=failed" >> $GITHUB_OUTPUT
             exit 1
           fi
 
-          echo "✓ source extracted"
+          echo "source extracted"
           echo "KUDU_SRC=$PWD/apache-kudu-${KUDU_VERSION}" >> $GITHUB_ENV
           echo "status=passed" >> $GITHUB_OUTPUT
 
@@ -82,10 +82,10 @@ jobs:
           START_TIME=$(date +%s)
 
           if [ -d "$KUDU_SRC/src/kudu" ]; then
-            echo "✓ core kudu source directory present"
+            echo "core kudu source directory present"
             echo "status=passed" >> $GITHUB_OUTPUT
           else
-            echo "✗ core kudu source directory missing at $KUDU_SRC/src/kudu"
+            echo "core kudu source directory missing at $KUDU_SRC/src/kudu"
             ls -R "$KUDU_SRC"
             echo "status=failed" >> $GITHUB_OUTPUT
             exit 1
@@ -100,10 +100,10 @@ jobs:
           START_TIME=$(date +%s)
 
           if [ -f "$KUDU_SRC/CMakeLists.txt" ]; then
-            echo "✓ CMakeLists.txt present (build system detected)"
+            echo "CMakeLists.txt present (build system detected)"
             echo "status=passed" >> $GITHUB_OUTPUT
           else
-            echo "✗ CMakeLists.txt missing in $KUDU_SRC"
+            echo "CMakeLists.txt missing in $KUDU_SRC"
             ls -R "$KUDU_SRC"
             echo "status=failed" >> $GITHUB_OUTPUT
             exit 1
@@ -117,13 +117,12 @@ jobs:
         run: |
           START_TIME=$(date +%s)
 
-          # Look for aarch64/arm64 mentions in the source tree as a simple heuristic
+          # Look for aarch64 mentions in the source tree as a simple heuristic
           if grep -Rni "aarch64" "$KUDU_SRC" | head -n 5; then
-            echo "✓ found references to aarch64 in source (indicates Arm awareness)"
+            echo "found references to aarch64 in source (indicates Arm awareness)"
             echo "status=passed" >> $GITHUB_OUTPUT
           else
-            echo "⚠ no explicit 'aarch64' references found – this does NOT mean arm is unsupported,"
-            echo "  but we record this test as 'passed' since Kudu is known to support arm in recent releases."
+            echo "no explicit 'aarch64' references found; not fatal, marking as passed"
             echo "status=passed" >> $GITHUB_OUTPUT
           fi
 
@@ -140,10 +139,10 @@ jobs:
 
           # Just configure, do not build (too heavy for CI here)
           if cmake .. > /tmp/kudu_cmake_configure.log 2>&1; then
-            echo "✓ cmake configure ran successfully (basic build configuration works on arm runner)"
+            echo "cmake configure ran successfully (basic build configuration works on arm runner)"
             echo "status=passed" >> $GITHUB_OUTPUT
           else
-            echo "✗ cmake configure failed"
+            echo "cmake configure failed"
             cat /tmp/kudu_cmake_configure.log || true
             echo "status=failed" >> $GITHUB_OUTPUT
             exit 1
@@ -158,15 +157,15 @@ jobs:
           START_TIME=$(date +%s)
 
           if [ -f "$KUDU_SRC/LICENSE.txt" ] || [ -f "$KUDU_SRC/LICENSE" ]; then
-            echo "✓ LICENSE present"
+            echo "LICENSE present"
           else
-            echo "⚠ LICENSE file not found; not critical for arm, but unexpected"
+            echo "LICENSE file not found; not critical for arm, but unexpected"
           fi
 
           if [ -f "$KUDU_SRC/NOTICE.txt" ] || [ -f "$KUDU_SRC/NOTICE" ]; then
-            echo "✓ NOTICE present"
+            echo "NOTICE present"
           else
-            echo "⚠ NOTICE file not found; not critical for arm, but unexpected"
+            echo "NOTICE file not found; not critical for arm, but unexpected"
           fi
 
           # This test is informational, mark as passed
@@ -186,19 +185,22 @@ jobs:
           process_test() {
             local status=$1
             local duration=$2
-            if [ "$status" == "passed" ]; then
+            if [ "$status" = "passed" ]; then
               PASSED=$((PASSED + 1))
-            elif [ "$status" == "failed" ]; then
+            elif [ "$status" = "failed" ]; then
               FAILED=$((FAILED + 1))
             fi
+            if [ -z "$duration" ]; then
+              duration=0
+            fi
             TOTAL_DURATION=$((TOTAL_DURATION + duration))
           }
 
-          process_test "${{ steps.test1.outputs.status }}" "${{ steps.test1.outputs.duration || 0 }}"
-          process_test "${{ steps.test2.outputs.status }}" "${{ steps.test2.outputs.duration || 0 }}"
-          process_test "${{ steps.test3.outputs.status }}" "${{ steps.test3.outputs.duration || 0 }}"
-          process_test "${{ steps.test4.outputs.status }}" "${{ steps.test4.outputs.duration || 0 }}"
-          process_test "${{ steps.test5.outputs.status }}" "${{ steps.test5.outputs.duration || 0 }}"
+          process_test "${{ steps.test1.outputs.status }}" "${{ steps.test1.outputs.duration }}"
+          process_test "${{ steps.test2.outputs.status }}" "${{ steps.test2.outputs.duration }}"
+          process_test "${{ steps.test3.outputs.status }}" "${{ steps.test3.outputs.duration }}"
+          process_test "${{ steps.test4.outputs.status }}" "${{ steps.test4.outputs.duration }}"
+          process_test "${{ steps.test5.outputs.status }}" "${{ steps.test5.outputs.duration }}"
 
           echo "passed=$PASSED" >> $GITHUB_OUTPUT
           echo "failed=$FAILED" >> $GITHUB_OUTPUT
@@ -245,27 +247,27 @@ jobs:
                 {
                   "name": "Check core source tree exists",
                   "status": "${{ steps.test1.outputs.status }}",
-                  "duration_seconds": ${{ steps.test1.outputs.duration || 0 }}
+                  "duration_seconds": ${{ steps.test1.outputs.duration }}
                 },
                 {
                   "name": "Check CMake / build configuration present",
                   "status": "${{ steps.test2.outputs.status }}",
-                  "duration_seconds": ${{ steps.test2.outputs.duration || 0 }}
+                  "duration_seconds": ${{ steps.test2.outputs.duration }}
                 },
                 {
                   "name": "Quick Arm64 support check in source",
                   "status": "${{ steps.test3.outputs.status }}",
-                  "duration_seconds": ${{ steps.test3.outputs.duration || 0 }}
+                  "duration_seconds": ${{ steps.test3.outputs.duration }}
                 },
                 {
                   "name": "Lightweight CMake configure",
                   "status": "${{ steps.test4.outputs.status }}",
-                  "duration_seconds": ${{ steps.test4.outputs.duration || 0 }}
+                  "duration_seconds": ${{ steps.test4.outputs.duration }}
                 },
                 {
                   "name": "Check LICENSE / NOTICE presence",
                   "status": "${{ steps.test5.outputs.status }}",
-                  "duration_seconds": ${{ steps.test5.outputs.duration || 0 }}
+                  "duration_seconds": ${{ steps.test5.outputs.duration }}
                 }
               ]
             },
@@ -323,9 +325,9 @@ jobs:
         run: |
           echo "## Apache Kudu Test Results" >> $GITHUB_STEP_SUMMARY
           echo "" >> $GITHUB_STEP_SUMMARY
-          echo "- **Version:** ${{ steps.download.outputs.kudu_version }}" >> $GITHUB_STEP_SUMMARY
-          echo "- **Status:** ${{ steps.summary.outputs.overall_status }}" >> $GITHUB_STEP_SUMMARY
-          echo "- **Tests Passed:** ${{ steps.summary.outputs.passed }}" >> $GITHUB_STEP_SUMMARY
-          echo "- **Tests Failed:** ${{ steps.summary.outputs.failed }}" >> $GITHUB_STEP_SUMMARY
-          echo "- **Duration:** ${{ steps.summary.outputs.duration }}s" >> $GITHUB_STEP_SUMMARY
-          echo "- **Runner:** ubuntu-24.04 (arm64)" >> $GITHUB_STEP_SUMMARY
+          echo "- Version: ${{ steps.download.outputs.kudu_version }}" >> $GITHUB_STEP_SUMMARY
+          echo "- Status: ${{ steps.summary.outputs.overall_status }}" >> $GITHUB_STEP_SUMMARY
+          echo "- Tests Passed: ${{ steps.summary.outputs.passed }}" >> $GITHUB_STEP_SUMMARY
+          echo "- Tests Failed: ${{ steps.summary.outputs.failed }}" >> $GITHUB_STEP_SUMMARY
+          echo "- Duration: ${{ steps.summary.outputs.duration }}s" >> $GITHUB_STEP_SUMMARY
+          echo "- Runner: ubuntu-24.04 (arm64)" >> $GITHUB_STEP_SUMMARY