Skip to content

Fix SeaweedFS integration access after NodePort removal - #156

Merged
sciabarracom merged 2 commits into
apache:0.9.0from
miki3421:fix/0.9.0-seaweedfs-private-tests
Oct 27, 2026
Merged

sciabarracom merged 2 commits into
apache:0.9.0from
miki3421:fix/0.9.0-seaweedfs-private-tests

Conversation

@miki3421

Copy link
Copy Markdown
Contributor

Summary

  • access the private SeaweedFS filer through the existing service_endpoint() helper after the Service and NetworkPolicy are applied
  • keep in-cluster operator traffic on the Kubernetes Service while using an RBAC-authorized, loopback-only, ephemeral port-forward for out-of-cluster integration runs
  • pass the scoped filer endpoint explicitly to the SeaweedFS client and close the tunnel after readiness and bootstrap operations
  • remove the obsolete Kind 9090 -> 32091 mapping and fixed localhost:9090 assumptions from the affected integration scripts
  • make wait_for_http() raise TimeoutError when its deadline expires, as documented

The SeaweedFS Service remains ClusterIP; this change does not restore any filer or master NodePort and does not relax the NetworkPolicy.

Validation

  • python -m unittest discover -s tests -p 'test_*.py' -q — 129 tests passed
  • focused private-service and SeaweedFS tests — 29 tests passed
  • Python syntax validation for the five affected .ipy integration scripts
  • YAML parsing for .github/kind.yaml
  • git diff --check

The destructive integration scripts were not run against a shared cluster. The PR CI uses an isolated Kind cluster and will execute the full and slim SeaweedFS/Milvus scenarios.

Fixes apache/openserverless#281

@miki3421
miki3421 requested a review from sciabarracom October 27, 2026 13:34
@sciabarracom
sciabarracom merged commit ffcea0c into apache:0.9.0 Oct 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants