Skip to content
This repository was archived by the owner on Apr 14, 2026. It is now read-only.

Update docker base images#8

Open
bdattoma wants to merge 6 commits intokserve:mainfrom
bdattoma:update_docker_images
Open

Update docker base images#8
bdattoma wants to merge 6 commits intokserve:mainfrom
bdattoma:update_docker_images

Conversation

@bdattoma
Copy link
Copy Markdown

@bdattoma bdattoma commented Feb 26, 2024

I've noticed that the base docker image is 2 years old, so I thought it could be worth to update the dockerfile

What do you think?

@oss-prow-bot
Copy link
Copy Markdown

oss-prow-bot bot commented Feb 26, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bdattoma

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bdattoma
Copy link
Copy Markdown
Author

bdattoma commented Feb 26, 2024

sorry this doesn't work well with console UI because it doesn't assign correctly permissions to the right user. I wasn't able to add a new user using the minio base image, so I guess in order to achieve the image update you should update the quay.io/cloudservices/minio image to fetch latest minio. Where can I find the dockerfile for quay.io/cloudservices/minio ?

@spolti
Copy link
Copy Markdown

spolti commented Jan 2, 2025

Do we still want this PR?
@bdattoma will you work on this?

@bdattoma
Copy link
Copy Markdown
Author

bdattoma commented Jan 7, 2025

Do we still want this PR? @bdattoma will you work on this?

Hi @spolti , sorry I didn't find time to look at this once again. However I saw that a related conversation is being held in #13. Basically the new minio images don't allow to pre-store files via Dockerfile, so with this image the files cannot be accessed via Minio Console.

I think we could close it and re-open in the future if somebody find a different solution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants