From 2cb5b5ec0e9eee76e2f7ca7243c554d66c3fa94d Mon Sep 17 00:00:00 2001 From: Robert Roland Date: Tue, 6 Dec 2016 13:12:49 -0800 Subject: [PATCH] Applying changes per PR feedback - Check for error conditions from the vSphere API and return the err if one occurs. The vSphere API does not return an err for unauthenticated users, it just returns a nil user object. --- pkg/cloudprovider/providers/vsphere/vsphere.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/pkg/cloudprovider/providers/vsphere/vsphere.go b/pkg/cloudprovider/providers/vsphere/vsphere.go index b4bfdc42fac..9aef4d053fa 100644 --- a/pkg/cloudprovider/providers/vsphere/vsphere.go +++ b/pkg/cloudprovider/providers/vsphere/vsphere.go @@ -360,8 +360,11 @@ func vSphereLogin(vs *VSphere, ctx context.Context) error { m := session.NewManager(vs.client.Client) // retrieve client's current session u, err := m.UserSession(ctx) - if err == nil && u != nil { - // current session is valid + if err != nil { + glog.Errorf("Error while obtaining user session. err: %q", err) + return err + } + if u != nil { return nil }