Commit 9b209008 authored by Michelle Noorali's avatar Michelle Noorali

ref(helm/doctor): update error message

This resolves issue #558.
parent abd1a5c1
...@@ -42,7 +42,7 @@ func doctor(c *cli.Context) error { ...@@ -42,7 +42,7 @@ func doctor(c *cli.Context) error {
if client.IsInstalled(runner) { if client.IsInstalled(runner) {
format.Success("You have everything you need. Go forth my friend!") format.Success("You have everything you need. Go forth my friend!")
} else { } 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 return nil
......
...@@ -21,7 +21,6 @@ import ( ...@@ -21,7 +21,6 @@ import (
"text/template" "text/template"
"github.com/Masterminds/sprig" "github.com/Masterminds/sprig"
"github.com/kubernetes/helm/pkg/format"
"github.com/kubernetes/helm/pkg/kubectl" "github.com/kubernetes/helm/pkg/kubectl"
) )
...@@ -73,9 +72,8 @@ func (i *Installer) expand() ([]byte, error) { ...@@ -73,9 +72,8 @@ func (i *Installer) expand() ([]byte, error) {
func IsInstalled(runner kubectl.Runner) bool { func IsInstalled(runner kubectl.Runner) bool {
// Basically, we test "all-or-nothing" here: if this returns without error // 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. // 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 { if err != nil {
format.Err("Installation not found: %s %s", out, err)
return false return false
} }
return true return true
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment