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

feat(Dockerfile): add docker file for the storage provider #120

Merged
merged 12 commits into from
Jul 9, 2024

Conversation

cernicc
Copy link
Member

@cernicc cernicc commented Jul 8, 2024

Description

Add Dockerfile and readme for how to run the storage server.

Important points for reviewers

The Dockerfile is not part of the build process. The idea is that for the community we should "manually" build the image and then upload that to some public image repository. The best solution would be if we build the image as part of our ci process and then add the image as the artifact of the repository. But the thing is that until the repo is private that image can't be exposed to the community.

cargo-chef is used to enable docker layer caching.

Checklist

  • Are there important points that reviewers should know?
  • Make sure that you described what this change does.
  • Have you tested this solution?
  • Did you document new (or modified) APIs?

@cernicc cernicc self-assigned this Jul 8, 2024
@cernicc cernicc requested review from th7nder, jmg-duarte and aidan46 July 8, 2024 09:27
@cernicc cernicc added the ready for review Review is needed label Jul 8, 2024
ci/Dockerfile.provider Outdated Show resolved Hide resolved
@jmg-duarte
Copy link
Contributor

The questions I left in the other PR still apply

th7nder
th7nder previously requested changes Jul 8, 2024
ci/Dockerfile.provider Outdated Show resolved Hide resolved
ci/Dockerfile.provider Outdated Show resolved Hide resolved
ci/PROVIDER.md Outdated Show resolved Hide resolved
ci/PROVIDER.md Outdated Show resolved Hide resolved
Copy link
Contributor

@jmg-duarte jmg-duarte left a comment

Choose a reason for hiding this comment

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

Nice job!

PROVIDER.md Outdated Show resolved Hide resolved
PROVIDER.md Outdated Show resolved Hide resolved
PROVIDER.md Outdated Show resolved Hide resolved
PROVIDER.md Outdated Show resolved Hide resolved
Base automatically changed from feat/98/rpc-method-to-upload-the-file to develop July 8, 2024 15:24
@cernicc cernicc added ready for review Review is needed and removed ready for review Review is needed labels Jul 9, 2024
@cernicc cernicc requested review from th7nder and jmg-duarte July 9, 2024 08:49
PROVIDER.md Outdated Show resolved Hide resolved
docker/dockerfiles/storage-provider/Dockerfile Outdated Show resolved Hide resolved
@cernicc cernicc requested a review from jmg-duarte July 9, 2024 10:10
@cernicc cernicc added ready for review Review is needed and removed ready for review Review is needed labels Jul 9, 2024
Copy link
Contributor

@jmg-duarte jmg-duarte left a comment

Choose a reason for hiding this comment

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

Good job 🐳

@cernicc cernicc added ready for review Review is needed and removed ready for review Review is needed labels Jul 9, 2024
@jmg-duarte jmg-duarte dismissed th7nder’s stale review July 9, 2024 13:32

Handled in subsequent commits

@jmg-duarte jmg-duarte merged commit 0282466 into develop Jul 9, 2024
3 checks passed
@jmg-duarte jmg-duarte deleted the feat/116/demo-polka-storage-provider-dockerfile branch July 9, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review Review is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants