Skip to content

Commit

Permalink
Remove logouts in keep alive handlers
Browse files Browse the repository at this point in the history
Signed-off-by: Stefan Büringer [email protected]
  • Loading branch information
sbueringer committed May 13, 2024
1 parent 472bfe0 commit d05e128
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions pkg/session/session.go
Original file line number Diff line number Diff line change
Expand Up @@ -250,9 +250,6 @@ func newClient(ctx context.Context, logger logr.Logger, sessionKey string, url *
if err != nil {
logger.Error(err, "failed to keep alive govmomi client")
logger.Info("clearing the session")
if errLogout := c.Logout(ctx); errLogout != nil {
logger.Error(err, "failed to logout keepalive failed session")
}
sessionCache.Delete(sessionKey)
}
return err
Expand Down Expand Up @@ -281,9 +278,6 @@ func newManager(ctx context.Context, logger logr.Logger, sessionKey string, clie
}

logger.Info("rest client session expired, clearing session")
if errLogout := rc.Logout(ctx); errLogout != nil {
logger.Error(err, "failed to logout keepalive failed rest session")
}
sessionCache.Delete(sessionKey)
return errors.New("rest client session expired")
})
Expand Down

0 comments on commit d05e128

Please sign in to comment.