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 Requests - AWS Credential profiles, Deleting charts, Reindexing repo #5

Closed
xyin96 opened this issue Oct 12, 2017 · 2 comments
Closed
Assignees
Labels
needs-investigation Requires digging/reproducing, or not obvious how to solve the issue

Comments

@xyin96
Copy link

xyin96 commented Oct 12, 2017

First of all, great work with this plugin, it works like a charm for us, and alleviated a lot of friction we had using a private s3 bucket for our chart repo. Here a few feature suggestions that we would love to see (if I had the time, I would love to jump in and help out):

AWS Profiles

In my use case, my AWS credentials file has multiple profiles (one personal and one for work). It would be nice to see that there is an option to specify which credentials profile to use to access the bucket.

Possible usage: helm s3 ... [--profile (default: default)]

Reindexing the repository

If for some reason we manually go into the s3 bucket, and modify the existing charts, I think we should have a method to easily reindex the index.yaml directly from the helm command through this plugin

Possible usage: helm s3 reindex [repo]

Deleting charts from the repository

I think that there should be a method to delete charts from the repo, and regenerate the index.yaml to reflect that.

Possible usage: helm s3 delete [chartName] [repo] [--version (default: latest)].

@hypnoglow
Copy link
Owner

Thank you. These are definitely interesting suggestions. I am going to take some time to investigate.

@hypnoglow hypnoglow added the needs-investigation Requires digging/reproducing, or not obvious how to solve the issue label Oct 13, 2017
@hypnoglow hypnoglow self-assigned this Oct 13, 2017
This was referenced Oct 14, 2017
@hypnoglow
Copy link
Owner

Closing this issue, because AWS profiles and deleting charts features were implemented in #7 and #10, and the reindexing is on the way #22

Thanks again for raising the ideas!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-investigation Requires digging/reproducing, or not obvious how to solve the issue
Projects
None yet
Development

No branches or pull requests

2 participants