@Daimona, I was already working on a patch on the road to resolve these kinds of DB related issues and I included the solution to this issue in the patch. Let me know if the issue gets resolved for you locally with this patch, then you can review and do the honors :)

Let me look into this and see if I can resolve the issue. I confirm that I can reproduce the issue when I visit the /w/index.php page since I have a MySQL/MariaDB setup locally.

T360458: Migrate CentralAuth to use StatsLib has covered some of the metrics marked as done in the task in the patch: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CentralAuth/+/1012734

In T359436#9715608, @colewhite wrote:

Docs have been updated on mediawiki.org. The feature is gone.

Thank you for the extra notes @Krinkle, much appreciated.

@matmarex, I spent some time trying to reproduce this but no luck. Is this still happening today? It seems SessionBackend has changed over time in how it handles sessions/tokens when a wiki is in read only mode? I need to track that down when that changed (if that is the case).

I've done a first round of addressing the questions listed in the ticket. I'll do another one before the week runs out then we can begin discussing how to move this forward.

Thank you @daniel and @Krinkle for your reviews on this tasks and guidance to make it land.

@Aklapper, thanks for writting, sorry for the delay in response, I was OoO. This is a tracking tasks for me (little side project) to cleanup our config file(s) - InitialiseSettings(-labs).php, CommonSettings(-labs).php etc from configs that are no longer used by our infrastructure. So TBH, there is no date for this to be done, it's something I'll be doing from time to time and like you rightly put it, it's a tech debt payment task.

1. Risky Patch! 🚂🔥

I covered this in: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikimediaMaintenance/+/1008839.

@colewhite, anything to do there? I think I merged all your patches and this should be covered by now?

This will wait again till 1.43 before we remove the method and close this task.

In T344521#9639996, @Krinkle wrote:

@DAlangi_WMF With the unified container, and the way to create a unified container, gone. Can we remove the isUnified() callers, function, and SCHEMA_UNIFIED constant as well? Those callers should be impossible now.

It's already due and we should just kill this.

In T355960#9625163, @lmata wrote:

Thanks @DAlangi_WMF! 🚀 🌝

Having it be an optional missable step feels less-ideal. Would there be any drawback to just making it be something that update.php always does?

After some digging, I think the regression happened in January (Cc @matmarex) here: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/981576.

It was interesting to learn :) about the weirdness of gauges (especially in the scenario of no data) and how smart Graphite would fill up the gaps if we wanted to.

Looking at the example in the addStaticLabel() method doc-block (which happens to be in line with what the code does):