[00:14:59] TimStarling: I think the stuff from https://gerrit.wikimedia.org/r/c/operations/puppet/+/398007/ is still there [23:17:57] regarding #mediawiki_security multi-DC discussion 10 hours ago [23:19:37] I said that the number of cross-DC DB connections will be on the order of 10 per second [23:20:54] based on logstash count of ~800,000 messages of the form "Expectation (masterConns <=) 0 by MediaWiki::main not met" in the last 24 hours [23:21:04] is that correct? [23:21:25] that sounds to me like the latency hit of just using the built-in-to-PHP support for TLS would be completely acceptable [23:22:25] Feels higher than I'd guess based on code knowledge and traffic, but not too high I suppose. We have around 5000/sec app server reqs right? [23:22:41] Perhaps some of the routes aren't acknowledged in TrxProf yet [23:22:54] Eg loginwiki and rollback [23:24:28] 10/second when our usual appserver load is ~6k/second [23:25:05] consider 10/s to be an upper bound [23:26:53] if it's 1/s that makes the case for plain TLS even stronger, right? [23:29:52] ack, checks out with `channel:DBPerformance AND message:masterConns AND -server:login.wikimedia.org` on 'mediawiki' Logstash dashboard. around 5-6K per 15min, which is ~8/s [23:31:25] of which 4K are from BlockManager and 400 are MWEchoNotifUser [23:31:43] https://phabricator.wikimedia.org/T231961 [23:31:51] stashbot: T231961 [23:31:51] See https://wikitech.wikimedia.org/wiki/Tool:Stashbot for help. [23:31:52] T231961: DBPerformance warning: "Expectation masterConns <= 0 not met" from CentralAuth special pages - https://phabricator.wikimedia.org/T231961 [23:32:25] err. I mean T267945 [23:32:26] T267945: Increase in DBPerformance warnings since 1.36.0-wmf.13 - https://phabricator.wikimedia.org/T267945 [23:32:36] anyway, yeah, that's a pretty small tail after that already [23:33:10] apart from BlockManager, they're also mostly post-send [23:35:31] ok, if you're agreeing then let's close T196378 and update T134809 to say that we will go ahead with plain TLS [23:35:32] T134809: App servers <=> mariadb SSL/TLS for cross-datacenter writes - https://phabricator.wikimedia.org/T134809 [23:35:32] T196378: Investigate solutions for MySQL connection pooling - https://phabricator.wikimedia.org/T196378 [23:43:11] Sounds good to me, TimStarling