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

[#5349] improvement(docker): Update Gravitino dockerfile to put cloud bundles into the docker image. #5402

Conversation

github-actions[bot]
Copy link

What changes were proposed in this pull request?

Copy cloud storage bundles jars to folder $GRAVITINO_HOME/catalogs/hadoop/libs when building Gravitino docker image to free from manually doing it.

Why are the changes needed?

We aim to make Gravitino user-friendly for managing cloud storage.

Fix: #5349

Does this PR introduce any user-facing change?

N/A.

How was this patch tested?

Manually verify locally.

… bundles into the docker image. (#5358)

### What changes were proposed in this pull request?

Copy cloud storage bundles jars to folder
`$GRAVITINO_HOME/catalogs/hadoop/libs` when building Gravitino docker
image to free from manually doing it.

### Why are the changes needed?

We aim to make Gravitino user-friendly for managing cloud storage. 

Fix: #5349 

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

Manually verify locally.
@jerryshao jerryshao merged commit 4dfeb4e into branch-0.7 Oct 31, 2024
26 checks passed
@jerryshao jerryshao deleted the cherry-pick-branch-0.7-a2f70932294fd517291d4dd9afbfaf3d4a439698 branch October 31, 2024 10:32
danhuawang pushed a commit to danhuawang/gravitino that referenced this pull request Nov 4, 2024
… cloud bundles into the docker image. (apache#5402)

### What changes were proposed in this pull request?

Copy cloud storage bundles jars to folder
`$GRAVITINO_HOME/catalogs/hadoop/libs` when building Gravitino docker
image to free from manually doing it.

### Why are the changes needed?

We aim to make Gravitino user-friendly for managing cloud storage. 

Fix: apache#5349 

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

Manually verify locally.

Co-authored-by: Qi Yu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants