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

FlyBackend supports mounting volume #28

Open
LiboShen opened this issue Mar 18, 2024 · 1 comment
Open

FlyBackend supports mounting volume #28

LiboShen opened this issue Mar 18, 2024 · 1 comment

Comments

@LiboShen
Copy link

I'm using FLAME for my project to run ML workload running on Fly. To avoid download the 4GB model data every time starting a FLAME runner, a Fly volume needs to be attached to the instance.

Currently there is a PR #22 by @benbot, however, it seems abandoned for 3 months.

Token a brief look, the tricky part seems that:

  • Fly volumes don't auto-scale when creating new instance,
  • and the API requires the caller to explicit specify the volume id instead of just the name.

This might become trickier when multiple nodes in the cluster trying to start the runners at the startup time causing potential race condition.

I currently forked the FlyBackend and hardcoded the volume info to make things working in my project. But a proper built-in solution would be very nice.

@benbot
Copy link

benbot commented Mar 18, 2024

I’ve been working off of my fork for a while, just haven’t had the need to update it yet.

despite being open for a few months, it got very little attention, so I’m not sure what the plan is for volumes with FLAME.

you could modify the backend further (it’s actually really simple) to use the fly api to create volumes on the fly, but that’d be very slow.

IMO this kind of functionality should be on fly and not in flame.

that being said I’m open to adding/changing anything in my PR if it’d help :)

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

No branches or pull requests

2 participants