Commit Graph
11 Commits
Author SHA1 Message Date
chrislu 150d084b3b fix: use localhost publicUrl and -max=100 for host-based Spark tests
The previous fix enabled master-to-volume communication but broke client writes.

Problem:
- Volume server uses -ip=seaweedfs-volume (Docker hostname)
- Master can reach it ✓
- Spark tests run on HOST (not in Docker container)
- Host can't resolve 'seaweedfs-volume' → UnknownHostException ✗

Solution:
- Keep -ip=seaweedfs-volume for master gRPC communication
- Change -publicUrl to 'localhost:8080' for host-based clients
- Change -max=0 to -max=100 (matches other integration tests)

Why -max=100:
- Pre-allocates volume capacity at startup
- Volumes ready immediately for writes
- Consistent with other test configurations
- More reliable than on-demand (-max=0)

This configuration allows:
- Master → Volume: seaweedfs-volume:18080 (Docker network)
- Clients → Volume: localhost:8080 (host network via port mapping)
2025-11-22 23:18:35 -08:00
chrislu ce40e2fd58 fix: use container hostname for volume server to enable automatic volume creation
Root cause identified:
- Volume server was using -ip=127.0.0.1
- Master couldn't reach volume server at 127.0.0.1 from its container
- When Spark requested assignment, master tried to create volume via gRPC
- Master's gRPC call to 127.0.0.1:18080 failed (reached itself, not volume server)
- Result: 'No writable volumes' error

Solution:
- Change volume server to use -ip=seaweedfs-volume (container hostname)
- Master can now reach volume server at seaweedfs-volume:18080
- Automatic volume creation works as designed
- Kept -publicUrl=127.0.0.1:8080 for external clients (host network)

Workflow changes:
- Remove forced volume creation (curl POST to /vol/grow)
- Volumes will be created automatically on first write request
- Keep diagnostic output for troubleshooting
- Simplified startup verification

This matches how other SeaweedFS tests work with Docker networking.
2025-11-22 23:09:27 -08:00
chrislu fd51091abd fix: add persistent volume data directory for volume server
- Add -dir=/data flag to volume server command
- Mount Docker volume seaweedfs-volume-data to /data
- Ensures volume server has persistent storage for volume files
- Fixes issue where volume server couldn't create writable volumes
- Volume data persists across container restarts during tests
2025-11-22 21:32:37 -08:00
chrislu a1a14259c3 fix: add -max=0 to volume server for unlimited volumes
- Add -max=0 flag to volume server command
- Allows volume server to create unlimited 50MB volumes
- Fixes 'No writable volumes' error during Spark tests
- Volume server will create new volumes as needed for writes
- Consistent with other integration test configurations
2025-11-22 20:42:03 -08:00
chrislu fab383dc10 fix: use 127.0.0.1 for volume server IP registration
- Change volume -ip from seaweedfs-volume to 127.0.0.1
- Change -publicUrl from localhost:8080 to 127.0.0.1:8080
- Volume server now registers with master using 127.0.0.1
- Filer will return 127.0.0.1:8080 URL that's resolvable from host
- Fixes UnknownHostException for seaweedfs-volume hostname
2025-11-22 14:21:02 -08:00
chrislu abaf933971 fix: add publicUrl to volume server for host network access
- Add -publicUrl=localhost:8080 to volume server command
- Ensures filer returns localhost URL instead of Docker service name
- Fixes UnknownHostException when tests run on host network
- Volume server is accessible via localhost from CI runner
2025-11-22 14:08:20 -08:00
chrislu e8e9df2680 test: improve docker-compose config for Spark tests
- Add -volumeSizeLimitMB=50 to master (consistent with other integration tests)
- Add -defaultReplication=000 to master for explicit single-copy storage
- Add explicit -port and -port.grpc flags to all services
- Add -preStopSeconds=1 to volume for faster shutdown
- Add healthchecks to master and volume services
- Use service_healthy conditions for proper startup ordering
- Improve healthcheck intervals and timeouts for faster startup
- Use -ip flag instead of -ip.bind for service identity
2025-11-22 13:35:52 -08:00
chrislu dd9c1c6190 fix: add -peers=none to master command for standalone mode
- Ensures master runs in standalone single-node mode
- Prevents master from trying to form a cluster
- Required for proper initialization in test environment
2025-11-22 13:34:01 -08:00
chrislu e63feb48fc ci: fix Maven artifact restoration in workflow
- Add step to restore Maven artifacts from download to ~/.m2/repository
- Restructure artifact upload to use consistent directory layout
- Remove obsolete 'version' field from docker-compose.yml to eliminate warnings
- Ensures SeaweedFS Java dependencies are available during test execution
2025-11-22 13:14:52 -08:00
chrislu e15e2e23c4 fix building 2025-11-22 12:56:43 -08:00
chrislu 89a6d42cee Complete Spark integration test suite 2025-11-22 12:23:48 -08:00