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

[Improvement] Update Gravitino dockerfile to put cloud storage bundles into the docker image. #5349

Closed
yuqi1129 opened this issue Oct 29, 2024 · 0 comments · Fixed by #5358
Closed
Assignees
Labels
0.7.0 Release v0.7.0 improvement Improvements on everything

Comments

@yuqi1129
Copy link
Contributor

What would you like to be improved?

It's better to put S3, GCP and OSS related bundle jar to Gravitino docker image.

How should we improve?

No response

@yuqi1129 yuqi1129 added the improvement Improvements on everything label Oct 29, 2024
@yuqi1129 yuqi1129 self-assigned this Oct 30, 2024
jerryshao pushed a commit that referenced this issue Oct 31, 2024
… 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 added the 0.7.0 Release v0.7.0 label Oct 31, 2024
github-actions bot pushed a commit that referenced this issue Oct 31, 2024
… 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 pushed a commit that referenced this issue Oct 31, 2024
… bundles into the docker image. (#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: #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]>
danhuawang pushed a commit to danhuawang/gravitino that referenced this issue 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]>
mplmoknijb pushed a commit to mplmoknijb/gravitino that referenced this issue Nov 6, 2024
… cloud bundles into the docker image. (apache#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: apache#5349 

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

N/A.

### How was this patch tested?

Manually verify locally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.7.0 Release v0.7.0 improvement Improvements on everything
Projects
None yet
2 participants