pipelines icon indicating copy to clipboard operation
pipelines copied to clipboard

[feature] Support S3 virtual hosted style

Open sanchesoon opened this issue 1 year ago • 3 comments

Feature Area

/area backend

What feature would you like to see?

To be able use Virtual Host Style S3 protocol we should provide configuration option. Because "by default, the S3 client will use virtual hosted bucket addressing when possible." According aws-sdk-go documentation.

What is the use case or pain point?

Ability to switch between Virtual Host and Path Style

Is there a workaround currently?

There is no work round for this. Because this is hardcoded https://github.com/kubeflow/pipelines/blob/master/backend/src/v2/objectstore/object_store.go#L260


Love this idea? Give it a 👍.

sanchesoon avatar May 09 '24 22:05 sanchesoon

/assign @HumairAK

HumairAK avatar May 10 '24 19:05 HumairAK

@sanchesoon what additional benefit would this give us?

I am pretty sure it would take a lot of work, which would be better spent on other problems which actually affect the user experience.

thesuperzapper avatar May 11 '24 03:05 thesuperzapper

@thesuperzapper It give us ability to use Virtual Host style. Not sure if here a lot work, should be just additional parameter to pass launcher (but I never did PR here) BTW this was fixed in argo workflow https://github.com/argoproj/pkg/pull/368/files

sanchesoon avatar May 13 '24 18:05 sanchesoon

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jul 13 '24 07:07 github-actions[bot]

/remove-lifecycle stale

HumairAK avatar Jul 14 '24 16:07 HumairAK

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Sep 13 '24 07:09 github-actions[bot]

/remove-lifecycle stale

HumairAK avatar Sep 13 '24 13:09 HumairAK