-
Notifications
You must be signed in to change notification settings - Fork 5
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
limited to google cloud? #29
Comments
It's limited to google cloud, since it only supports google cloud storage buckets and not an AWS storage service like S3. The bucket interface code is simple though, and if you were up for writing an S3 backend we'd be happy to have it. I haven't used kubernetes on AWS but besides the storage backend I don't see any reason this wouldn't work. |
Sounds good, looks simple enough. The idea here is to support binary uploads/downloads to pass around arguments/return values to the containers? A couple questions:
|
Awesome. Yep, that's right. The uploads and downloads are used to pass input and output data to and from tasks. I updated the README for the project to hopefully give a better overview of things. Let me know if anything is unclear.
|
Thanks. One more question -- does this library provide any support for copying a source tree over to the docker instances before running the function, or is the client responsible for handling that? E.g. if we have a module |
Good question. We don't try to copy and source over to the worker currently. That seems error prone to me to try to do in general, but if you know of a good way to handle that I'd be interested. What we do support for this case is a set of pip packages to install on the workers before running user code (the |
I see, that's a pretty reasonable solution. In our case researchers will be working in their own docker instances so it seems like we can just commit/push the instance itself and use that as the base image. |
Hi, just wondering if this library is limited to Google Cloud or if it can be used on other kubernetes deployments as well (e.g. on EC2)?
The text was updated successfully, but these errors were encountered: