Skip to content

Commit

Permalink
Don't return error when repos have no chart
Browse files Browse the repository at this point in the history
Fix helm#632
I build a chartmuseum service which has no charts, then add the
chartmuseum repo into kubeapps. But I get a 'error' pod, that is
unfriendly. I think it is better to add log for it.
  • Loading branch information
panfengyun committed May 27, 2019
1 parent f8afab9 commit 66b3345
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cmd/chart-repo/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,8 @@ func syncRepo(dbSession datastore.Session, repoName, repoURL string, authorizati

charts := chartsFromIndex(index, r)
if len(charts) == 0 {
return errors.New("no charts in repository index")
log.Warnf("The repository %s has no charts.", repoName)
return nil
}
err = importCharts(dbSession, charts)
if err != nil {
Expand Down

0 comments on commit 66b3345

Please sign in to comment.