[02:47:19] legoktm: I see GUP is spamming dbperformance.log ... loading users from the master DB indirectly [03:20:19] * legoktm looks [04:28:31] 6MediaWiki-Core-Team, 5Patch-For-Review: Investigate/prevent master revision query on enwiki edit form - https://phabricator.wikimedia.org/T86862#1107215 (10aaron) [10:38:26] 6MediaWiki-Core-Team, 6CA-team, 10SUL-Finalization, 6operations: db1068 (s4/commonswiki slave) is missing data about at least 6 users - https://phabricator.wikimedia.org/T91920#1108878 (10Krenair) >>! In T91920#1107363, @Springle wrote: >>>! In T91920#1105596, @Krenair wrote: >> Are the other two db1062 an... [14:24:48] 6MediaWiki-Core-Team, 10MediaWiki-API, 5Patch-For-Review: Clean up ApiResult and ApiFormatXml, create new formatversion - https://phabricator.wikimedia.org/T76728#1109456 (10Anomie) [14:24:58] 6MediaWiki-Core-Team, 10MediaWiki-API, 5Patch-For-Review: Clean up ApiResult and ApiFormatXml, create new formatversion - https://phabricator.wikimedia.org/T76728#819003 (10Anomie) [14:25:11] 6MediaWiki-Core-Team, 10MediaWiki-API, 5Patch-For-Review: Clean up ApiResult and ApiFormatXml, create new formatversion - https://phabricator.wikimedia.org/T76728#819003 (10Anomie) [14:29:54] legoktm: Are you making enough progress on reviewing the ApiResult thing (thanks, BTW) that I should start pressing Flow and Wikidata to resolve their blockers of T76728? [15:08:09] anomie: yeah, I should be done by the end of the week at the latest [16:22:50] 6MediaWiki-Core-Team, 6Multimedia, 3Multimedia-Sprint-2015-03-11, 5Patch-For-Review: SHA-1 file name support - https://phabricator.wikimedia.org/T1210#1109748 (10Gilles) [16:44:57] bd808: Who's our person who goes to scrum-of-scrums? I forget, and you probably know. [16:45:28] Me, Aaron and Chris usually [16:45:33] I won't be there this week [16:46:02] speaking of which ... I need to update the notes and ask chris to cover for me [16:46:24] bd808: Can you add T76728 being blocked by Flow and Wikidata to the notes? [16:46:38] yup [16:47:05] You should tell Chris what that means I guess [17:00:09] bd808: rock, nicely done with that ieg app blocker task [17:00:18] ty [17:00:46] cool. [17:05:44] bd808: is aaron around today? [17:05:49] or greg-g [17:06:08] i am looking at https://gerrit.wikimedia.org/r/#/c/195773/ and it breaks wikibase [17:06:24] so we need to update wikibase and i am not sure enough what to do [17:06:56] aude: I haven't seen Aaron yet today [17:07:00] ok [17:07:09] do you have a wikibase fix? [17:07:13] no [17:07:21] i don't 100% understand this code [17:07:28] or what to do instead [17:07:28] so do we need to revert? [17:07:45] prefer to avoid, but would like the patch to be more clear what to do instead [17:07:56] * bd808 doesn't know if branch is cut yet [17:08:15] oh [17:08:37] good point [17:08:49] looks like wmf20 is still the newest [17:09:06] twentyafterfour: is that right? No wmf21 branch cut yet today? [17:09:27] https://github.com/wikimedia/mediawiki-extensions-Wikibase/blob/master/client/includes/UpdateRepo/UpdateRepo.php#L167 is the problem [17:09:29] bd808: I'm working on it [17:09:32] will be a fatal error [17:09:40] I think cirrus will also break [17:09:43] I cut a branch...but I typod the version [17:09:51] and not sure enough, if we can leave the check out and leave the parameter there always [17:09:55] and it does the right thing [17:09:58] bd808: so it's 1.20wmf21 ...trying to fix that now [17:10:00] I would revert [17:10:06] ok [17:10:32] twentyafterfour: fun. sorry that process sucks so much [17:10:33] should I wait a bit before a re-cut the branch with the proper name? [17:10:36] https://gerrit.wikimedia.org/r/#/c/195953/1 [17:10:44] or are you guys cool if I do it nowish? [17:11:04] twentyafterfour: https://gerrit.wikimedia.org/r/#/c/195953/ needs to be in the branch [17:11:14] legoktm has the patch in flight [17:11:17] legoktm: ok merge it :) [17:11:29] it'll be a few minutes before I rerun the make-wmf-branch script [17:11:43] aude: good catch [17:12:03] gonna fix some things in the script that weren't quite right anyway...still gotta figure out how to delete a whole bunch of wmf/1.20wmf21 branches now [17:12:16] thanks legoktm [17:12:47] and thanks jenkins :) [17:12:59] csteipp: Can you cover core for SoS again this week? I'm in a crazy meeting [17:13:07] I updated http://etherpad.wikimedia.org/p/Scrum-of-Scrums with notes [17:17:08] bd808: Totally fine [17:17:18] thanks [17:17:51] twentyafterfour: it's merged now [17:17:54] The last bullet - Brad's soft blockers - is the big thing to try and get out [17:18:14] I know that wikidata is aware; I haven't talked to flow [17:19:15] bd808: the api changes? [17:19:37] aude: Yeah. I emailed Lydia about the task last week [17:19:43] ah, yes [17:19:53] it's a matter of scheduling time for this, but seems important [17:19:58] I haven't checked back to find out if it got in a sprint yet [17:20:01] *nod* [17:20:10] not this one, but i think soon [17:51:25] Krinkle: any additional feedback on ? [17:51:45] 6MediaWiki-Core-Team, 10hardware-requests, 6operations, 5Patch-For-Review: Fluorine needs bigger disks - https://phabricator.wikimedia.org/T92417#1110216 (10Andrew) 3NEW [17:52:01] 6MediaWiki-Core-Team, 6operations, 5Patch-For-Review: Store unsampled API and XFF logs - https://phabricator.wikimedia.org/T88393#1110224 (10Andrew) [17:55:59] ori: One strey ')' I'm fixing locally, and one question: the old request() version looked up inArray before adding to the queue array. The new code does not do that. [17:56:14] I assume either it's an impossible condition or it's deduped later already? [18:00:46] 6MediaWiki-Core-Team, 6operations, 5Patch-For-Review: Store unsampled API and XFF logs - https://phabricator.wikimedia.org/T88393#1110251 (10Andrew) OK, current log retention policy looks like this: API logs: 30 days api-feature-usage logs: 90 days xff logs: 88 days everything else: 180 days If we were... [18:01:29] blue wizard needs food badly [18:04:01] 6MediaWiki-Core-Team, 6operations, 5Patch-For-Review: Store unsampled API and XFF logs - https://phabricator.wikimedia.org/T88393#1110267 (10hoo) IMO 30 days are enough for API logs (and probably also for XFF logs, although I think we decided to no longer collect these at all?). Other logs (like exception an... [18:06:47] legoktm: https://gerrit.wikimedia.org/r/#/c/195966/ silly fix [18:09:45] 6MediaWiki-Core-Team, 10hardware-requests, 6operations, 5Patch-For-Review: Fluorine needs bigger disks - https://phabricator.wikimedia.org/T92417#1110300 (10RobH) We don't have any 4TB disks on site, but they could be ordered. What is the overall capacity and raid requirements for the logging server? [18:10:00] 6MediaWiki-Core-Team, 10hardware-requests, 6operations, 5Patch-For-Review: Fluorine needs bigger disks - https://phabricator.wikimedia.org/T92417#1110301 (10RobH) I forgot to ask speed requirements for disks. [18:11:35] ok I'm branching now, after finally fixing make-wmf-branch bugs (hopefully...) and my own typos (: [18:13:32] 6MediaWiki-Core-Team, 10hardware-requests, 6operations, 5Patch-For-Review: Fluorine needs bigger disks - https://phabricator.wikimedia.org/T92417#1110320 (10Andrew) This might be moot -- sounds like we're maybe just retaining way more logs than anyone actually wants. Stay tuned... [18:15:56] legoktm: actually https://gerrit.wikimedia.org/r/#/c/195969/1 then since that conflicted [18:24:44] Krinkle: the omitted inArray check in request() was my mistake [18:24:51] since i'm amending that, where's the stray ')'? [18:25:01] ori: [18:25:02] - // since those only work when placed at the start of a stylesheet; bug 35562). [18:25:02] + // since those only work when placed at the start of a stylesheet; bug 35562. [18:25:39] Krinkle: done; thanks! [18:30:24] csteipp: http://www.wired.com/2015/03/google-hack-dram-memory-electric-leaks/ [18:32:15] aspiecat: Yeah, that was a cool hack [18:32:50] The joy of having unlimited funding to just do security research.... :) [18:38:13] * aspiecat should have got ECC [19:47:54] ori: I killed the windows stuff in https://gerrit.wikimedia.org/r/#/c/195507/3 while at it [19:51:02] I doubt many people use windows and want to use redis (it's a pain getting a good driver dll plus you need the MS open tech redis compiled via visual studio) [19:51:09] * aspiecat did that stuff before, but stil... [19:51:12] *still [19:56:39] aspiecat: i'll be in the office later if you want to deploy the changes [19:56:43] along w/the puppet change [19:56:55] * aspiecat will be in the office later too ;) [20:30:17] 6MediaWiki-Core-Team, 10MediaWiki-RfCs, 7Epic, 5Patch-For-Review: Consider moving various DB writes on page views to using local jobs - https://phabricator.wikimedia.org/T91837#1097241 (10daniel) Marking this as "draft" for now, needs to be fleshed out a bit before discussion. [20:56:46] 6MediaWiki-Core-Team, 10MediaWiki-RfCs, 7Epic, 5Patch-For-Review: Consider moving various DB writes on page views to using local jobs - https://phabricator.wikimedia.org/T91837#1111009 (10Catrope) The summary is pretty brief, but I (and the other archcom people) support the general idea. Theoretically this... [20:57:20] 6MediaWiki-Core-Team, 10MediaWiki-RfCs, 7Epic, 5Patch-For-Review: Consider moving various DB writes on page views to using local jobs - https://phabricator.wikimedia.org/T91837#1097241 (10Catrope) (BTW why is this tagged Patch to Review? Is there a patch?) [21:00:41] 6MediaWiki-Core-Team, 10MediaWiki-RfCs, 7Epic, 5Patch-For-Review: Consider moving various DB writes on page views to using local jobs - https://phabricator.wikimedia.org/T91837#1111042 (10aaron) The epic task keeps getting that tag, and "create subtask" inherits it unless one keeps remembering to uncheck it. [21:18:29] 6MediaWiki-Core-Team, 10MediaWiki-API, 5Patch-For-Review: Clean up ApiResult and ApiFormatXml, create new formatversion - https://phabricator.wikimedia.org/T76728#1111108 (10Quiddity) [22:01:18] 6MediaWiki-Core-Team, 10MediaWiki-API, 5Patch-For-Review: Clean up ApiResult and ApiFormatXml, create new formatversion - https://phabricator.wikimedia.org/T76728#1111330 (10DannyH)