dilim/helm
1
0
Fork 0
mirror of https://github.com/helm/helm.git synced 2026-08-07 00:21:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Errors out if install/upgrade fails to load-requirements

Browse Source 
Fixes https://github.com/kubernetes/helm/issues/2480
This commit is contained in:
Sushil Kumar
2017-05-22 15:28:58 -07:00
parent d76c1049d0
commit 18f900e443
2 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cmd/helm/install.go
View File

@@ -232,6 +232,8 @@ func (i *installCmd) run() error {
if err := checkDependencies(chartRequested, req, i.out); err != nil {
return prettyError(err)
}
} else if err != chartutil.ErrRequirementsNotFound {
return fmt.Errorf("cannot load requirements: %v", err)
}
res, err := i.client.InstallReleaseFromChart(

4
cmd/helm/upgrade.go
View File

@@ -185,7 +185,11 @@ func (u *upgradeCmd) run() error {
if err := checkDependencies(ch, req, u.out); err != nil {
return err
}
} else if err != chartutil.ErrRequirementsNotFound {
return fmt.Errorf("cannot load requirements: %v", err)
}
} else {
return prettyError(err)
}
resp, err := u.client.UpdateRelease(