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

[feature] Use s3:// for MinIO usage #10737

Open
AndersBennedsgaard opened this issue Apr 24, 2024 · 0 comments
Open

[feature] Use s3:// for MinIO usage #10737

AndersBennedsgaard opened this issue Apr 24, 2024 · 0 comments

Comments

@AndersBennedsgaard
Copy link

AndersBennedsgaard commented Apr 24, 2024

Feature Area

/area frontend
/area backend
/area sdk

(I am a little unsure which area is affected by this feature request, so feel free to change the area labels)

What feature would you like to see?

Use s3:// for MinIO instead of minio://.

What is the use case or pain point?

Since MinIO exposes an S3 compatible API we might as well just use s3:// as the URL protocol prefix.

A big reason for this, is that not many other tools use minio:// as prefix (in fact, I was not able to find any), so if you want to integrate with any of these tools in a pipeline (such as KServe), you have to do a url.replace("minio://", "s3://") based on the artifact location.

Since multiple vendors exposes cloud storage with an S3 compatible API (IBM, Google Cloud) and you can use a reverse proxy such as gaul/s3proxy for those that doesn't, it doesn't make sense to distinguish between the underlying backend solution.

Is there a workaround currently?

  • Use url.replace("minio://", "s3://") in pipelines.
  • Close your eyes and don't worry about it 🙈 🙉 🙊

Love this idea? Give it a 👍.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant