Stale sessions left over in Redis Shared State

During today's demo we were looking at the flamegraph from #2538 (comment 1593195770):

flamegraph__6_.svg

@reprazent asked a poignant question: Why do we have session keys in ServiceRedis (Shared State), shouldn't those be in ServiceRedisSessions?

Looking at the differential flamegraphs from @msmiley we indeed see that the size of session:lookup:user:gitlab:$NUMBER has remained constant #2538 (comment 1593756454):

diff.2nd_minus_1st.svg

This is a strong indicator that these keys are no longer being updated.

Confirming in the code, we can see that there is no expiry set on these values.

Conclusion: These keys have been migrated to ServiceRedisSessions and are safe to be deleted from ServiceRedis (Shared State).