From f4e472d3968f8b6d02288a37aae7742d8775ff8a Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Wed, 28 Jan 2026 01:28:47 -0800 Subject: [PATCH] workflow: fix s3 tables tests path and working directory The workflow was failing because it was running inside 'weed' directory, but the tests are at the repository root. Removed working-directory default and updated relative paths to weed source. --- .github/workflows/s3-tables-tests.yml | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/.github/workflows/s3-tables-tests.yml b/.github/workflows/s3-tables-tests.yml index 8cfbd40a9..ac2e03243 100644 --- a/.github/workflows/s3-tables-tests.yml +++ b/.github/workflows/s3-tables-tests.yml @@ -10,9 +10,6 @@ concurrency: permissions: contents: read -defaults: - run: - working-directory: weed jobs: s3-tables-tests: @@ -90,7 +87,7 @@ jobs: run: | set -x echo "=== Building S3 Tables package ===" - go build ./s3api/s3tables || { + go build ./weed/s3api/s3tables || { echo "❌ S3 Tables package build failed" exit 1 } @@ -100,7 +97,7 @@ jobs: run: | set -x echo "=== Building S3 API with S3 Tables integration ===" - go build ./s3api || { + go build ./weed/s3api || { echo "❌ S3 API build with S3 Tables failed" exit 1 } @@ -110,7 +107,7 @@ jobs: run: | set -x echo "=== Running Go unit tests for S3 Tables ===" - go test -v -race -timeout 5m ./s3api/s3tables/... || { + go test -v -race -timeout 5m ./weed/s3api/s3tables/... || { echo "❌ S3 Tables unit tests failed" exit 1 } @@ -135,10 +132,10 @@ jobs: run: | set -x echo "=== Checking S3 Tables Go format ===" - unformatted=$(go fmt ./s3api/s3tables/... 2>&1 | wc -l) + unformatted=$(go fmt ./weed/s3api/s3tables/... 2>&1 | wc -l) if [ "$unformatted" -gt 0 ]; then echo "❌ Go format check failed - files need formatting" - go fmt ./s3api/s3tables/... + go fmt ./weed/s3api/s3tables/... exit 1 fi echo "✓ All S3 Tables files are properly formatted" @@ -174,7 +171,7 @@ jobs: run: | set -x echo "=== Running go vet on S3 Tables package ===" - go vet ./s3api/s3tables/... || { + go vet ./weed/s3api/s3tables/... || { echo "❌ go vet check failed" exit 1 }