Commit 04a7e241 authored by Alon Lavi's avatar Alon Lavi

Fix a bug causing 'helm depndency update' to delete required charts

parent 475de1f4
...@@ -216,12 +216,15 @@ func (m *Manager) downloadAll(deps []*chartutil.Dependency) error { ...@@ -216,12 +216,15 @@ func (m *Manager) downloadAll(deps []*chartutil.Dependency) error {
return fmt.Errorf("%q is not a directory", destPath) return fmt.Errorf("%q is not a directory", destPath)
} }
fmt.Fprintf(m.Out, "Saving %d charts\n", len(deps)) fmt.Fprintln(m.Out, "Deleting outdated charts")
for _, dep := range deps { for _, dep := range deps {
if err := m.safeDeleteDep(dep.Name, destPath); err != nil { if err := m.safeDeleteDep(dep.Name, destPath); err != nil {
return err return err
} }
}
fmt.Fprintf(m.Out, "Saving %d charts\n", len(deps))
for _, dep := range deps {
if strings.HasPrefix(dep.Repository, "file://") { if strings.HasPrefix(dep.Repository, "file://") {
if m.Debug { if m.Debug {
fmt.Fprintf(m.Out, "Archiving %s from repo %s\n", dep.Name, dep.Repository) fmt.Fprintf(m.Out, "Archiving %s from repo %s\n", dep.Name, dep.Repository)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment