Skip to content

Commit

Permalink
Merge pull request #619 from Chronojam/master
Browse files Browse the repository at this point in the history
Pass error to user if we fail to correctly pull or export oci chart images
  • Loading branch information
luisdavim authored Aug 10, 2021
2 parents 6d7e6dd + ac40176 commit 8565597
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion internal/app/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,9 @@ func downloadFile(file string, dir string, outfile string) string {
case "oci":
dest := filepath.Dir(outfile)
fileName := strings.Split(filepath.Base(file), ":")[0]
helmExportChart(strings.ReplaceAll(file, "oci://", ""), dest)
if err := helmExportChart(strings.ReplaceAll(file, "oci://", ""), dest); err != nil {
log.Fatal(err.Error())
}
return filepath.Join(dest, fileName)
case "https", "http":
if err := downloadFileFromURL(file, outfile); err != nil {
Expand Down

0 comments on commit 8565597

Please sign in to comment.