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

Support "package" subcommand from Helm #177

Open
kkaempf opened this issue Jul 12, 2021 · 2 comments
Open

Support "package" subcommand from Helm #177

kkaempf opened this issue Jul 12, 2021 · 2 comments

Comments

@kkaempf
Copy link
Collaborator

kkaempf commented Jul 12, 2021

Background: I'm using hypper as a full Helm replacement when working with Epinio and the latest release calls helm package which makes Hypper break.

Acceptance criteria: hypper is a full Helm replacement in an Epinio context

See also: epinio/epinio#556

@viccuad
Copy link
Member

viccuad commented Jul 12, 2021

Initially I thought that it would be a good idea to add a hypper package, but the more I think about it, the more I would like to not wrap all of Helm with Hypper.

Hypper has backwards compat with Helm, and I would like to rely on that: everything should be working fine calling helm package, and installing the resulting charts with either Helm or Hypper (the latter of course brings more functionality).

For Helm commands that gain functionality when run in Hypper, I am all in though. Maybe we could add some Hypper-specific benefit to doing hypper package, such as checking for hypper repo consistency, but that sounds like a task for hypper repo index.

@mattfarina
Copy link
Collaborator

We should have the package command check the shared and optional dependencies for use of file:// for the repo and either throw and error or warning

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

No branches or pull requests

3 participants