Do not reset except on client connect error

client/controller.go

@@ -105,7 +105,7 @@ func (hosts *HostsController) sendMetric(host config.Host, client *Client) {
 			}
 			log.Error(errorContent)
 			//FIXME: what kind of errors do we especially want to reset driver for
-			if _, ok := err.(*driver.SSHError); ok {
+			if _, ok := err.(*driver.SSHConnectError); ok {
 				hosts.resetDriver(host)
 			}
 			message := &SendMessage{

driver/ssh.go

@@ -13,12 +13,22 @@ var (
 	port = 22
 )
 
-type SSHError struct {
+type SSHConnectError struct {
 	content string
+	client  string
 }
 
-func (e *SSHError) Error() string {
-	return fmt.Sprintf("SSH Error: %s", e.content)
+func (e *SSHConnectError) Error() string {
+	return fmt.Sprintf("SSH Connect Error on %s: %s", e.client, e.content)
+}
+
+type SSHRunError struct {
+	content string
+	client  string
+}
+
+func (e *SSHRunError) Error() string {
+	return fmt.Sprintf("SSH Run Error on %s: %s", e.client, e.content)
 }
 
 // SSH : Driver for handling ssh executions
@@ -102,7 +112,10 @@ func (d *SSH) RunCommand(command string) (string, error) {
 	log.Debugf("Running remote command %s", command)
 	client, err := d.Client()
 	if err != nil {
-		return ``, &SSHError{content: err.Error()}
+		return ``, &SSHConnectError{
+			content: err.Error(),
+			client:  d.Host,
+		}
 	}
 	if len(d.EnvVars) != 0 {
 		// add env variable to command
@@ -120,6 +133,7 @@ func (d *SSH) GetDetails() SystemDetails {
 	if d.Info == nil {
 		// TODO: Check for goph specific errors
 		// within RunCommand and only return errors that are not
+		log.Debugf("Checking platform details for %s", d.Host)
 		// goph specific
 		uname, err := d.RunCommand(`uname`)
 		// try windows command