Skip to content

Refactor SkyPilot Multi-Tenant Project and SkyPilot EKS App#4

Merged
roulbac merged 3 commits into
mainfrom
sp-refactor
Feb 5, 2026
Merged

Refactor SkyPilot Multi-Tenant Project and SkyPilot EKS App#4
roulbac merged 3 commits into
mainfrom
sp-refactor

Conversation

@roulbac

@roulbac roulbac commented Feb 5, 2026

Copy link
Copy Markdown
Owner

Summary

This PR refactors the SkyPilot multi-tenant project and the SkyPilot EKS application for better modularity and maintainability.

Key Changes:

  • Project Refactor (projects/skypilot-multi-tenant/):
    • Extracted configuration models and loaders to config.py.
    • Simplified __main__.py by offloading configuration logic.
  • SkyPilot App Refactor (pulumi_eks_ml/eks_apps/skypilot/):
    • Created config_builder.py for Helm values and config generation.
    • Created credentials.py (as a ComponentResource) for admin credentials management.
    • Created iam.py for IAM policy documents.
    • Updated api_server.py to use these new modules, significantly reducing its complexity.
  • IAM Enhancements:
    • Added S3 read/write permissions to the SkyPilot API server's IAM policy to support data-intensive workloads.

Test Plan

  • Verified configuration loading in skypilot-multi-tenant.
  • Validated SkyPilotAPIServer instantiation with the new modular structure.
  • Inspected generated IAM policies for correctness.

Made with Cursor

@roulbac roulbac merged commit db1f07c into main Feb 5, 2026
1 check passed
@roulbac roulbac deleted the sp-refactor branch February 7, 2026 16:45
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.

1 participant