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

Add Fly.io as a provider #417

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Add Fly.io as a provider #417

wants to merge 18 commits into from

Conversation

mileszim
Copy link

Hello! This PR adds the cloud platform host Fly.io as a provider. It includes the auto-generated documentation too.

Let me know if anything is missing. Thank you!

Copy link
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing this, I've never heard of fly.io. I'm happy to merge this in but I don't think I'll be much help in maintaining it so if issues come up I'll ping you on them.

I noticed you copied in the whole SDK rather than adding it as a dependency. Can I ask why?

dask_cloudprovider/__init__.py Outdated Show resolved Hide resolved
dask_cloudprovider/cloudprovider.yaml Outdated Show resolved Hide resolved
@@ -15,6 +15,7 @@
"digitalocean": ["python-digitalocean>=1.15.0"],
"gcp": ["google-api-python-client>=1.12.5", "google-auth>=1.23.0"],
"hetzner": ["hcloud>=1.10.0"],
"fly": ["httpx>=0.24.0", "pydantic>=1.10.7"],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this include fly-python-sdk?

@mileszim
Copy link
Author

mileszim commented Nov 3, 2023

Thanks for contributing this, I've never heard of fly.io. I'm happy to merge this in but I don't think I'll be much help in maintaining it so if issues come up I'll ping you on them.

I noticed you copied in the whole SDK rather than adding it as a dependency. Can I ask why?

I'm not in control over that repo, and it isn't clear how much if any effort will be spent in the future by the maintainer to update or fix issues, so I figured the most reliable thing to do, especially for such a small subset of the Fly API, is to implement those calls myself

@jacobtomlinson
Copy link
Member

Just want to check in here and see if there is any interest in continuing with this PR?

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.

2 participants