In T416155#11584484, @Ollie.Shotton_WMDE wrote:

In T416155#11584255, @dang wrote:

Why are all 3 PRs the same?

While similar, they are not all the same. If you look at the commits tab, you'll see that they each contain a different number of commits. #2558 contains the commits from #2557 and #2559 contains the commits from #2558 and #2557. This is because GitHub isn't good at displaying stacked PRs, where one PR is based on another PR. Gerrit is way better at handling this without this confusion (/me looks longingly at his long-lost love, Gerrit!).

In the GitHub Changes tab you can select which commits you want to see in the diff if you are interested in only the changes that are part of that PR.

In T312107#8080285, @dcausse wrote:

I think the reason why it's not finding this particular item is that the CirrusSearch index needs to be recreated to take into account the new mul language field. The problem is less visible during fulltext search as the term John Doe II is present in the main text field but this kind of search might still be affected because matches on the mul field might not be properly boosted.

In T415017#11544220, @Rosalie_WMDE wrote:

In T415017#11540792, @Rosalie_WMDE wrote:

Argument splitting happens at the shell level see. Laravel only receives the final argv array and has no knowledge of how the arguments were separated (spaces, tabs, etc.), so am not sure how best to unit test this. I also don’t think we should change the input handling, since mail:send {user*} is the standard Laravel pattern for this.

I am happy to hear thoughts on this @Andrew-WMDE @Ollie.Shotton_WMDE

In T415017#11536044, @Ollie.Shotton_WMDE wrote:

Emails can be separated by either tabs or multiple spaces

FYI, I did test the current script locally, and it does work with both tabs and multiple spaces.

@Anton.Kokh @Ollie.Shotton_WMDE please verify I didn't miss anything here.

wbstack/api/pull/1036 | Add tests for WikiUserEmailChecker

AFAIK, these are likely due to caching and shouldn't block the update. I wasn't able to reproduce them more than once. Also, whenever an error was thrown, there didn't appear to be any user-facing errors.

[rdbms] Expectation (masterConns <= 0) by MediaWiki\MediaWikiEntryPoint::restInPeace not met (actual: 1): [connect to sql-mariadb-primary.default.svc.cluster.local ()]

from https://console.cloud.google.com/errors/detail/COyO-aiI_5q2Zg;time=P7D;locations=global?project=wikibase-cloud
see T408101