Page MenuHomePhabricator

Update logging and monitoring for multiple session storage backends
Closed, ResolvedPublic

Description

Currently we have session logging in two places:

  • Logging session writes to the session-sampled Logstash channel in SessionManager and SessionBackend
  • Sending Prometheus metrics in BagOStuff

That might be fine as is, but we should consider if we want to change something there after T399192: Create new session store abstraction to replace BagOStuff in SessionManager / T399194: Implement different backends for anonymous and authenticated sessions.

Event Timeline

Change #1178870 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/core@master] session: Segregate anonymous sessions from authenticated sessions (p2)

https://gerrit.wikimedia.org/r/1178870

Change #1178870 merged by jenkins-bot:

[mediawiki/core@master] session: Segregate anonymous sessions from authenticated sessions (p2)

https://gerrit.wikimedia.org/r/1178870

Change #1187513 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/core@master] session: Follow-up for I10101c8b928a12 (3fde556f95b4b1ce43)

https://gerrit.wikimedia.org/r/1187513

Change #1187513 merged by jenkins-bot:

[mediawiki/core@master] session: Follow-up on I10101c8b928a12 (3fde556f95b4b1ce43)

https://gerrit.wikimedia.org/r/1187513

Change #1188733 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/core@master] session: Improve logging and monitoring in SessionStore implementations

https://gerrit.wikimedia.org/r/1188733

Change #1188733 merged by jenkins-bot:

[mediawiki/core@master] session: Improve logging and monitoring in SessionStore implementations

https://gerrit.wikimedia.org/r/1188733

Change #1191351 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/core@wmf/1.45.0-wmf.20] session: Improve logging and monitoring in SessionStore implementations

https://gerrit.wikimedia.org/r/1191351

Change #1191360 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/core@wmf/1.45.0-wmf.19] session: Improve logging and monitoring in SessionStore implementations

https://gerrit.wikimedia.org/r/1191360

Change #1191360 merged by jenkins-bot:

[mediawiki/core@wmf/1.45.0-wmf.19] session: Improve logging and monitoring in SessionStore implementations

https://gerrit.wikimedia.org/r/1191360

Change #1191351 merged by jenkins-bot:

[mediawiki/core@wmf/1.45.0-wmf.20] session: Improve logging and monitoring in SessionStore implementations

https://gerrit.wikimedia.org/r/1191351

Mentioned in SAL (#wikimedia-operations) [2025-09-25T13:27:03Z] <tgr@deploy1003> Started scap sync-world: Backport for [[gerrit:1191359|objectcache: Add a hit/miss flag to CachedBagOStuff]], [[gerrit:1191360|session: Improve logging and monitoring in SessionStore implementations (T399195 T402808)]], [[gerrit:1191361|hCaptcha: Fix mock for StatsFactory]], [[gerrit:1191362|NewcomerTasks: Use StatsFactory unit test helper]], [[gerrit:1191350|objectcache: Add a hit/miss flag to CachedB

Mentioned in SAL (#wikimedia-operations) [2025-09-25T13:33:33Z] <tgr@deploy1003> d3r1ck01, wmde-fisch, tgr: Backport for [[gerrit:1191359|objectcache: Add a hit/miss flag to CachedBagOStuff]], [[gerrit:1191360|session: Improve logging and monitoring in SessionStore implementations (T399195 T402808)]], [[gerrit:1191361|hCaptcha: Fix mock for StatsFactory]], [[gerrit:1191362|NewcomerTasks: Use StatsFactory unit test helper]], [[gerrit:1191350|objectcache: Add a hit/miss flag to Cache

Mentioned in SAL (#wikimedia-operations) [2025-09-25T13:43:12Z] <tgr@deploy1003> Finished scap sync-world: Backport for [[gerrit:1191359|objectcache: Add a hit/miss flag to CachedBagOStuff]], [[gerrit:1191360|session: Improve logging and monitoring in SessionStore implementations (T399195 T402808)]], [[gerrit:1191361|hCaptcha: Fix mock for StatsFactory]], [[gerrit:1191362|NewcomerTasks: Use StatsFactory unit test helper]], [[gerrit:1191350|objectcache: Add a hit/miss flag to Cached

Change #1192051 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):

[mediawiki/core@master] tests: Expand tests for MultiBackendSessionStore

https://gerrit.wikimedia.org/r/1192051