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 Azure Blob Storage backend #191

Merged

Conversation

christoe
Copy link
Contributor

Hi!

I implemented support for Azure Blob Storage as a backend, inspired by the s3 and GS backends that already existed. I tried documenting and writing tests also based on what as available in the repo. Please get back to me if you have any feedback.

Thanks!

@christoe
Copy link
Contributor Author

I can see the tests are failing due to missing dependencies. Not sure how you want to handle this - it seems that e.g. boto3 is an indirect dependency of rio-tiler and thus installed as a runtime requirement which makes S3 tests succeed.

@christoe
Copy link
Contributor Author

Hi @vincentsarago! Sorry for tagging you explicitly, but since you seem to be the main contributor - is this project still active?

@vincentsarago
Copy link
Member

sorry @christoe, I am the main contributor and sadly I have a lot other priority right now :-(

@vincentsarago
Copy link
Member

@christoe in order to allow the CI to run tests you must add "az" in https://github.com/developmentseed/cogeo-mosaic/blob/master/.github/workflows/ci.yml#L35 -> python -m pip install -e .["test,az"]

@christoe
Copy link
Contributor Author

@vincentsarago Thanks and I totally understand. I was able to get the tests running now.

@vincentsarago vincentsarago merged commit 9c7f361 into developmentseed:master Oct 21, 2022
@christoe christoe deleted the add-azure-blob-storage-backend branch October 27, 2022 09:46
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.

None yet

2 participants