Skip to content

Isolate SeaweedFS control APIs from tenant actions - #155

Merged
sciabarracom merged 1 commit into
apache:0.9.0from
miki3421:fix/0.9.0-seaweedfs-isolation
Oct 26, 2026
Merged

sciabarracom merged 1 commit into
apache:0.9.0from
miki3421:fix/0.9.0-seaweedfs-isolation

Conversation

@miki3421

Copy link
Copy Markdown
Contributor

Summary

  • make the SeaweedFS service cluster-internal instead of exposing S3, Filer, and Master as NodePorts
  • allow tenant workloads to keep using authenticated S3 on port 9000 while restricting Filer 9090 and Master 9333 to trusted non-action pods
  • remove legacy Filer ingress resources and reconcile the boundary on operator resume/update
  • prevent action label impersonation by excluding pods carrying the user-action-pod marker from trusted peers

Compatibility

  • authenticated tenant S3 traffic remains available on port 9000
  • the existing S3 ingress remains supported
  • operator access to Filer and Master remains available
  • existing installations are migrated during operator resume without recreating SeaweedFS or tenant data
  • the legacy console configuration field remains accepted, but the unauthenticated Filer API is no longer published

Validation

  • git diff --check
  • changed Python modules compile successfully
  • full Python unit suite: 126 tests passed
  • clean 0.9.0 operator image build completed successfully
  • verified on the baseline K3s lab with real synthetic Alpha and Beta actions:
    • before the change, an Alpha action reached ports 9000, 9090, and 9333
    • after the change, the same action reached 9000 but was blocked from 9090 and 9333, including when attempting to copy the operator label
    • the operator still received HTTP 200 from 9090 and 9333
    • Alpha and Beta retained access to their own S3 buckets and received AccessDenied for the peer bucket
    • anonymous data bucket access remained HTTP 403 and public web bucket access remained HTTP 200

No existing tenant data was modified or removed by the migration.

@sciabarracom
sciabarracom merged commit 8c94d0f into apache:0.9.0 Oct 26, 2026
1 check failed
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