Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[devseed] Expand deployer exec homes command to optionally include EBS volumes for AWS clusters #4794

Closed
2 tasks
Tracked by #4647
sgibson91 opened this issue Sep 11, 2024 · 2 comments · Fixed by #4803
Closed
2 tasks
Tracked by #4647

Comments

@sgibson91
Copy link
Member

sgibson91 commented Sep 11, 2024

We have a deployer exec homes command that creates a throwaway k8s pod with the home directories mounted from NFS with read-write access. As we migrate our AWS hubs to using in-cluster NFS backed by EBS volumes, we will also need to mount these data sources. Being able to do this will also assist with the migration process tracked in #4728 as we tried to mount the EFS instance to an EC2 instance but failed.

This issue tracks expansion of the deployer exec homes command to optionally include EBS volumes for AWS clusters.

Definition of Done

  • deployer exec homes has an optional flag to also mount EBS volumes for AWS clusters
  • when executed, the throwaway pod has access to both EFS and EBS file systems
@sgibson91 sgibson91 changed the title Expand deployer exec homes command to optionally include EBS volumes for AWS clusters [devseed] Expand deployer exec homes command to optionally include EBS volumes for AWS clusters Sep 11, 2024
@sunu
Copy link
Contributor

sunu commented Sep 16, 2024

@sgibson91 I've created a PR at #4803 to address this issue but it takes a slightly different approach.

I realized the jupyter-home-nfs NFS server pod and the throwaway pod could be running on different nodes. And we can't safely mount the EBS backed PVC created by jupyter-home-nfs to multiple nodes at once in read/write mode since EBS volumes don't support that.

Instead, I've added some optional arguments to the root_homes command so that we can mount an extra NFS backed volume to the throwaway pod. This should allow us to mount a volume backed by the existing EFS instance and another volume backed by the NFS server of jupyter-home-nfs to the same throwaway pod and migrate the data from one to the other.

Let me know if that sounds reasonable.

@sgibson91
Copy link
Member Author

Yep, that sounds reasonable, thanks!

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 a pull request may close this issue.

2 participants