Provider shouldn't call log.Fatal, return error instead

IBM-Cloud · May 12, 2017 · f6f2568 · f6f2568
1 parent ddb4417
commit f6f2568
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/builtin/providers/ibmcloud/config.go b/builtin/providers/ibmcloud/config.go
@@ -195,13 +195,13 @@ func (c *Config) ClientSession() (interface{}, error) {
 
 	accClient, err := accountv2.New(sess.BluemixSession)
 	if err != nil {
-		log.Fatal(err)
+		return nil, err
 	}
 	accountAPI := accClient.Accounts()
 
 	clusterClient, err := k8sclusterv1.New(sess.BluemixSession)
 	if err != nil {
-		log.Fatal(err)
+		return nil, err
 	}
 	clustersAPI := clusterClient.Clusters()
 	clusterWorkerAPI := clusterClient.Workers()
@@ -226,7 +226,7 @@ func (c *Config) ClientSession() (interface{}, error) {
 		bluemixAccountClient: accountAPI,
 	}
 
-	return session, err
+	return session, nil
 }
 
 func newSession(c *Config) (*Session, error) {