Merge pull request #559 from michelleN/doctor-error

ref(helm/doctor): update error message
pull/562/head
Michelle Noorali 9 years ago
commit 368287edca

@ -42,7 +42,7 @@ func doctor(c *cli.Context) error {
if client.IsInstalled(runner) {
format.Success("You have everything you need. Go forth my friend!")
} else {
format.Warning("Looks like you don't have DM installed.\nRun: `helm server install`")
format.Warning("Looks like you don't have the helm server-side components installed.\nRun: `helm server install`")
}
return nil

@ -21,7 +21,6 @@ import (
"text/template"
"github.com/Masterminds/sprig"
"github.com/kubernetes/helm/pkg/format"
"github.com/kubernetes/helm/pkg/kubectl"
)
@ -73,9 +72,8 @@ func (i *Installer) expand() ([]byte, error) {
func IsInstalled(runner kubectl.Runner) bool {
// Basically, we test "all-or-nothing" here: if this returns without error
// we know that we have both the namespace and the manager API server.
out, err := runner.GetByKind("rc", "manager-rc", "helm")
_, err := runner.GetByKind("rc", "manager-rc", "helm")
if err != nil {
format.Err("Installation not found: %s %s", out, err)
return false
}
return true

Loading…
Cancel
Save