fix(cmd/helm): user friendly error message when repos are not configured

Signed-off-by: Adam Reese <adam@reese.io>
pull/6310/head
Adam Reese 5 years ago
parent f46df928cf
commit c4b76f27de
No known key found for this signature in database
GPG Key ID: 06F35E60A7A18DD6

@ -18,7 +18,9 @@ package main
import (
"io"
"os"
"github.com/pkg/errors"
"github.com/spf13/cobra"
"helm.sh/helm/cmd/helm/require"
@ -48,3 +50,7 @@ func newRepoCmd(out io.Writer) *cobra.Command {
return cmd
}
func isNotExist(err error) bool {
return os.IsNotExist(errors.Cause(err))
}

@ -35,10 +35,7 @@ func newRepoListCmd(out io.Writer) *cobra.Command {
Args: require.NoArgs,
RunE: func(cmd *cobra.Command, args []string) error {
f, err := repo.LoadFile(settings.RepositoryConfig)
if err != nil {
return err
}
if len(f.Repositories) == 0 {
if isNotExist(err) || len(f.Repositories) == 0 {
return errors.New("no repositories to show")
}
table := uitable.New()

@ -56,8 +56,8 @@ func newRepoRemoveCmd(out io.Writer) *cobra.Command {
func (o *repoRemoveOptions) run(out io.Writer) error {
r, err := repo.LoadFile(o.repoFile)
if err != nil {
return err
if isNotExist(err) || len(r.Repositories) == 0 {
return errors.New("no repositories configured")
}
if !r.Remove(o.name) {

@ -59,13 +59,9 @@ func newRepoUpdateCmd(out io.Writer) *cobra.Command {
}
func (o *repoUpdateOptions) run(out io.Writer) error {
f, err := repo.LoadFile(o.repoFile)
if err != nil {
return err
}
if len(f.Repositories) == 0 {
return errNoRepositories
f, err := repo.LoadFile(settings.RepositoryConfig)
if isNotExist(err) || len(f.Repositories) == 0 {
return errors.New("no repositories to update")
}
var repos []*repo.ChartRepository
for _, cfg := range f.Repositories {

@ -147,8 +147,8 @@ func (o *searchRepoOptions) formatSearchResults(res []*search.Result) string {
func (o *searchRepoOptions) buildIndex(out io.Writer) (*search.Index, error) {
// Load the repositories.yaml
rf, err := repo.LoadFile(o.repoFile)
if err != nil {
return nil, errors.Wrap(err, "loading repository config")
if isNotExist(err) || len(rf.Repositories) == 0 {
return nil, errors.New("no repositories configured")
}
i := search.NewIndex()

Loading…
Cancel
Save