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

Readme and notebook on using Zarr on CESNET Swift #15

Merged
merged 7 commits into from
Sep 17, 2022

Conversation

guillaumeeb
Copy link
Member

Work on #14 and discussion.

I added a Readme with all the necessary things to do to obtain authentication for Swift. But I could not go to the end as I'm currently experiencing errors (in discussion with @sebastian-luna-valero).

I also prepared a notebook for testing S3, Swift access and various libraries. In the end the notebook propose some rough performance tests on the object storage. I couldn't test all of it.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@guillaumeeb guillaumeeb marked this pull request as ready for review September 15, 2022 18:04
@guillaumeeb
Copy link
Member Author

So I solved the problem with getting correct values for Swift URL and AUTH_KEY, and also the problem I add when using aa Dask Cluster (zarr-swift not installed on workers, classic). This is ready for a review.

As this is just a first test (and not real performance test and all), review can just be some high level ideas or suggestions, or typos also!

@guillaumeeb
Copy link
Member Author

Something I did not dig into is the consolidate=true kwarg, did not have the time to check what it implies when creating a Zarr dataset with Xarray.

Copy link
Collaborator

@sebastian-luna-valero sebastian-luna-valero left a comment

Choose a reason for hiding this comment

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

Thanks!

My suggestions below.

EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@sebastian-luna-valero sebastian-luna-valero left a comment

Choose a reason for hiding this comment

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

Adding how to create S3 credentials at CESNET

EGI-CLI-Swift.md Outdated Show resolved Hide resolved
EGI-CLI-Swift.md Outdated Show resolved Hide resolved
guillaumeeb and others added 2 commits September 16, 2022 15:46
Co-authored-by: Sebastian Luna-Valero <sebastian-luna-valero@users.noreply.github.com>
Co-authored-by: Sebastian Luna-Valero <sebastian-luna-valero@users.noreply.github.com>
@tinaok
Copy link
Collaborator

tinaok commented Sep 16, 2022

Hi, I have a test pangeo config to run, but I need to make some part of data available only for a few user. Is it possible to create Zarr file on Cesnet's swift 'private' (the owner) 'group' (some of people who have access to pangeo-eosc platform) and 'public' internet open ?

guillaumeeb and others added 2 commits September 16, 2022 20:33
Co-authored-by: Sebastian Luna-Valero <sebastian-luna-valero@users.noreply.github.com>
@guillaumeeb
Copy link
Member Author

@tinaok I think that you should open a new issue for this question (and I'm not sure if it can be easily answered).

@sebastian-luna-valero I think I answered all your comments, and I've updated this notebook to take S3 writes into account.

@guillaumeeb
Copy link
Member Author

@tinaok I've opened #17 to discuss your need.

@guillaumeeb
Copy link
Member Author

I'm gonna merge this, because it gives a first working base to use CESNET object storage. Feel free to open issues or PR to improve it!

@guillaumeeb guillaumeeb merged commit a663cd7 into pangeo-data:main Sep 17, 2022
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.

3 participants