Skip to content

Make sure to delete all old sessions#35886

Merged
hakonhall merged 1 commit intomasterfrom
hmusum/create-session-if-not-found-in-cache
Feb 13, 2026
Merged

Make sure to delete all old sessions#35886
hakonhall merged 1 commit intomasterfrom
hmusum/create-session-if-not-found-in-cache

Conversation

@hmusum
Copy link
Copy Markdown
Member

@hmusum hmusum commented Feb 12, 2026

Sessions with status DELETE are not in remote session cache. When the flag USE_EXPERIMENTAL_DELETE_SESSIONS_CODE is true we now create a remote session instead if it is not found inthe cache. In addition, if flag is true and we have no creation time available for the session we will use creation time from file system for the session instead.

If the flag is false, the code should work as before.

New unit test works with flag enabled, fails with it disabled.

Sessions with status DELETE are not in remote session cache.
When the flag USE_EXPERIMENTAL_DELETE_SESSIONS_CODE is true we now
create a remote session instead if it is not found inthe cache.
In addition, if flag is true and we have no creation time available
for the session we will use creation time from file system for
the session instead.

If the flag is false, the code should work as before.
@hmusum hmusum force-pushed the hmusum/create-session-if-not-found-in-cache branch from 0e8bfc9 to 2a1b711 Compare February 12, 2026 22:11
@hmusum hmusum requested a review from hakonhall February 13, 2026 08:11
@hakonhall hakonhall merged commit 9d9c16c into master Feb 13, 2026
3 checks passed
@hakonhall hakonhall deleted the hmusum/create-session-if-not-found-in-cache branch February 13, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants