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

feat: allow user set forcePathStyle for s3 storage #731

Merged
merged 3 commits into from
Nov 3, 2023

Conversation

warjiang
Copy link
Contributor

!!! Note: This PR depends on the merger of another PR to be merged. We have mentioned a PR in chartmuseum/storage to support users to manually set s3ForcePathStyle. After the pr of storage is merged, I will remove the WIP status of the current PR.

This PR mainly provides support for command line parameters, allowing users to set s3ForcePathStyle through command line parameters.

If possible, anyone can help me review the PR in chartmuseum/storage.
thks

@warjiang warjiang force-pushed the feat/amazon-force-path-style branch 2 times, most recently from ca7f6ec to 7a98ed3 Compare October 25, 2023 04:12
@warjiang warjiang force-pushed the feat/amazon-force-path-style branch from 7a98ed3 to 9cc2c62 Compare October 25, 2023 06:07
@warjiang
Copy link
Contributor Author

somthing wrong with git(dco not works with ssh ), still make dco finally

@scbizu
Copy link
Contributor

scbizu commented Oct 27, 2023

@warjiang Hi , we just release the storage v0.14.1 , can you fix your go.mod ?

@pull-request-size pull-request-size bot added size/M and removed size/S labels Oct 27, 2023
@warjiang warjiang force-pushed the feat/amazon-force-path-style branch 3 times, most recently from db317d6 to f25b9d2 Compare October 27, 2023 16:10
@warjiang warjiang force-pushed the feat/amazon-force-path-style branch from f25b9d2 to 5d5bdf5 Compare October 27, 2023 16:13
@warjiang warjiang changed the title WIP: allow user set forcePathStyle for s3 storage feat: allow user set forcePathStyle for s3 storage Oct 27, 2023
@warjiang
Copy link
Contributor Author

warjiang commented Oct 27, 2023

@warjiang Hi , we just release the storage v0.14.1 , can you fix your go.mod ?

@scbizu Already update the dependencies of the chartmuseum and resolve the WIP status. Also update usage of new options of amazon-storage in the README.md. BTW thank you very much for your quick response ⚡

@scbizu scbizu self-requested a review October 28, 2023 10:01
@scbizu scbizu self-assigned this Oct 28, 2023
@scbizu scbizu requested a review from cbuto October 28, 2023 11:16
@scbizu
Copy link
Contributor

scbizu commented Oct 28, 2023

@cbuto PTAL

@warjiang
Copy link
Contributor Author

warjiang commented Nov 2, 2023

can anyone help to merge this pr, i want to use official chartmuseum instead of self build, thks

@scbizu scbizu merged commit 0cfeb2e into helm:main Nov 3, 2023
2 checks passed
@scbizu
Copy link
Contributor

scbizu commented Nov 3, 2023

@warjiang try our canary image .

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

Successfully merging this pull request may close these issues.

2 participants