Merge pull request #13172 from vespa-engine/revert-13170-musum/reduce-time-to-keep-expired-sessions

Revert "Reduce the time we keep expired remotes sessions and fix comment"

1 files changed, 4 insertions, 3 deletions

diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java
index c0211fb6e91..c4f0c5fdd5d 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java
@@ -27,10 +27,11 @@ public class SessionsMaintainer extends ConfigServerMaintainer {
     protected void maintain() {
         applicationRepository.deleteExpiredLocalSessions();
 
-        // Expired remote sessions are sessions that belong to an application that have external deployments that
-        // are no longer active
+        // Expired remote sessions are not expected to exist, they should have been deleted when
+        // a deployment happened or when the application was deleted. We still see them from time to time,
+        // probably due to some race or another bug
         if (hostedVespa) {
-            Duration expiryTime = Duration.ofDays(7);
+            Duration expiryTime = Duration.ofDays(30);
             applicationRepository.deleteExpiredRemoteSessions(expiryTime);
         }
     }