Merge branch 'session-keepalive' of https://github.com/spectrocloud/c…

…luster-api-provider-vsphere into session-keepalive
spectrocloud · Apr 7, 2021 · bf7032d · bf7032d
2 parents cda025d + 1dc2499
commit bf7032d
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/pkg/services/govmomi/vcenter/clone_test.go b/pkg/services/govmomi/vcenter/clone_test.go
@@ -142,11 +142,6 @@ func initSimulator(t *testing.T) (*simulator.Model, *session.Session, *simulator
 	server := model.Service.NewServer()
 	pass, _ := server.URL.User.Password()
 
-	//authSession, err := session.GetOrCreate(
-	//	session.NewSessionContext(ctx.TODO(), klogr.New()),
-	//	server.URL.Host, "",
-	//	server.URL.User.Username(), pass, "",
-	//	session.DefaultFeature())
 	authSession, err := session.GetOrCreate(
 		session.NewSessionContext(ctx.TODO(), klogr.New()),
 		session.NewParams().

diff --git a/pkg/session/session.go b/pkg/session/session.go
@@ -117,7 +117,6 @@ func GetOrCreate(ctx Context, params *Params) (*Session, error) {
 	sessionMU.Lock()
 	defer sessionMU.Unlock()
 
-	sessionKey := params.server + params.userinfo.Username() + params.datacenter
 	if session, ok := sessionCache[sessionKey]; ok {
 		// if keepalive is enabled we depend upon roundtripper to reestablish the connection
 		// and remove the key if it could not