[00:01:17] (03CR) 10Smalyshev: Process dump files (031 comment) [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [00:01:50] (03CR) 10Sbisson: [C: 031] Fix Flow config variable for template compilation [vagrant] - 10https://gerrit.wikimedia.org/r/202623 (owner: 10Mattflaschen) [00:02:25] 10UI-Standardization: mw-ui-buttons should have a max width - https://phabricator.wikimedia.org/T95367#1188094 (10Jdlrobson) [00:03:07] (03CR) 10BryanDavis: [C: 032] Fix Flow config variable for template compilation [vagrant] - 10https://gerrit.wikimedia.org/r/202623 (owner: 10Mattflaschen) [00:03:27] (03Merged) 10jenkins-bot: Fix Flow config variable for template compilation [vagrant] - 10https://gerrit.wikimedia.org/r/202623 (owner: 10Mattflaschen) [00:04:07] superm401, stephanebisson: Please feel free to +2 changes like that to roles you help maintain :) [00:09:15] (03PS1) 10Prtksxna: wip: fontsize and leading stuff [extensions/Popups] - 10https://gerrit.wikimedia.org/r/202627 [00:10:00] bd808: I don't have the power to +2 :( [00:10:48] well we should fix that [00:10:55] * bd808 checks the gerrit config [00:13:31] stephanebisson: I think we need to get you added to the "wmf" ldap group and then you should have +2 in many places [00:14:02] (03PS4) 10Manybubbles: Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 [00:14:27] (03CR) 10Manybubbles: "PS4 rebases and addresses all outstanding comments (I think)." [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [00:16:01] bd808, do we have to file a Phab task for that? [00:16:40] The last time I asked, the "official" way to get into the wmf ldap group was "ping ^d on irc". He's probably commuting home about now. [00:17:26] A phab task probably wouldn't hurt. Not sure what project to tag it to. Anyone in ops should be able to update ldap I think [00:17:30] (03CR) 10Manybubbles: [C: 04-1] "+2 but for whitespace differences." (031 comment) [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 (owner: 10Smalyshev) [00:17:47] just put it in operations, RT duty person should do it [00:17:54] (*or* bribe people with M&Ms) [00:17:57] speaking of which, I should get some [00:18:20] YuviPanda: I'll be in town next week. :) [00:18:35] hehe :) [00:18:55] bd808: superm401 if you file a phab task I’ll just do it. [00:19:23] need ldap username, and some proof that he’s with the WMF. In this case I already saw the welcome email so that’s good enough :) [00:19:30] (03PS2) 10Smalyshev: Add poll delay and ignore repeating last seen change [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 [00:19:44] (03CR) 10Smalyshev: "fixed ws" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 (owner: 10Smalyshev) [00:20:16] YuviPanda, sbisson [00:20:46] somebody needs to update the staff page on foundationwiki too [00:21:15] (03CR) 10jenkins-bot: [V: 04-1] Add poll delay and ignore repeating last seen change [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 (owner: 10Smalyshev) [00:22:34] (03PS3) 10Smalyshev: Add poll delay and ignore repeating last seen change [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 [00:23:23] (03PS1) 10MarkAHershberger: Add initial README [extensions/HitCounters] - 10https://gerrit.wikimedia.org/r/202629 [00:24:58] (03PS1) 10Aaron Schulz: [WIP] Made LoadBalancer avoid lagged slaves more aggressively [core] - 10https://gerrit.wikimedia.org/r/202630 [00:27:22] 10MediaWiki-General-or-Unknown, 10Analytics-EventLogging, 7Performance: Add event tracking queue to MediaWiki core for loose coupling with EventLogging or other interested consumers - https://phabricator.wikimedia.org/T95356#1188141 (10bd808) [00:27:58] (03CR) 10TTO: [C: 04-1] Use single hook to register tags and performance improvements [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) (owner: 10Cenarium) [00:28:30] (03PS1) 10Aaron Schulz: Set a query group on SpecialRecentChanges query [core] - 10https://gerrit.wikimedia.org/r/202632 [00:29:43] (03CR) 10TTO: "Replied on PS2" (031 comment) [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) (owner: 10Cenarium) [00:30:23] (03PS1) 10Aaron Schulz: Made mergeActiveUsers use "recentchanges" query group [core] - 10https://gerrit.wikimedia.org/r/202634 [00:32:41] 10Programs-Evaluation: Retention Graph WLM - https://phabricator.wikimedia.org/T93379#1188154 (10JAnstee_WMF) [00:32:43] 10Programs-Evaluation: Retention Graphs - https://phabricator.wikimedia.org/T94075#1188153 (10JAnstee_WMF) 5Open>3Resolved [00:34:52] 10Programs-Evaluation: Programs database coding for final reportability - https://phabricator.wikimedia.org/T93402#1188158 (10JAnstee_WMF) 5Open>3Resolved [00:35:15] 10Programs-Evaluation: Evaluation Pulse Y2Y Data Check - https://phabricator.wikimedia.org/T93398#1188161 (10JAnstee_WMF) 5Open>3Resolved [00:35:24] 10Programs-Evaluation: Evaluation Pulse Charts Dev - https://phabricator.wikimedia.org/T93399#1188163 (10JAnstee_WMF) 5Open>3Resolved [00:35:32] 10Programs-Evaluation: Evaluation Pulse Data Narrative - https://phabricator.wikimedia.org/T93400#1188164 (10JAnstee_WMF) 5Open>3Resolved [00:35:56] 10Programs-Evaluation: Wiki Loves reporting summary points, key findings and next steps - https://phabricator.wikimedia.org/T93410#1188166 (10JAnstee_WMF) 5Open>3Resolved [00:36:28] (03CR) 10Mattflaschen: [C: 032] "We tested this together." [vagrant] - 10https://gerrit.wikimedia.org/r/202592 (owner: 10Sbisson) [00:36:47] (03Merged) 10jenkins-bot: Enable browser tests for Flow [vagrant] - 10https://gerrit.wikimedia.org/r/202592 (owner: 10Sbisson) [00:37:35] 10Wikimedia-Shop: Globe Hoodie unisex medium is unorderable - https://phabricator.wikimedia.org/T92541#1188167 (10Catrope) [00:38:02] 10UI-Standardization: mw-ui-buttons should have a max width - https://phabricator.wikimedia.org/T95367#1188170 (10Ricordisamoa) [00:41:05] 6MediaWiki-Core-Team, 6Availability-Team: Look into Maria 10 parallel-replication - https://phabricator.wikimedia.org/T85266#1188175 (10aaron) Although DB1055 had a spike at 00:00 similar to 1072 (https://tendril.wikimedia.org/chart?hosts=db10%2855%29&vars=^seconds_behind_master&mode=delta) today. Even with it... [00:47:13] 3Team-Practices-This-Week: Create definitions for remaining (4) starred terms - https://phabricator.wikimedia.org/T93654#1188176 (10ksmith) Kunal pointed out that [[#anchor]] actually does work (yay!), although not in preview (boo!). Everything has now been linkified. [00:48:24] 3Team-Practices-This-Week: Document best practice for phabricator issue resolution - https://phabricator.wikimedia.org/T94189#1188179 (10ksmith) [00:52:58] 10Wikimedia-Site-requests: Set $wgMFEditorOptions['anonymousEditing'] = false for kowiki - https://phabricator.wikimedia.org/T94388#1188197 (10Revi) 5Open>3Resolved a:3Revi Please get consensus on local first. Until then, setting as resolved, as current request (disable anon edit on mobile) is fulfilled. [00:55:33] (03CR) 10Manybubbles: [C: 032] Add poll delay and ignore repeating last seen change [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 (owner: 10Smalyshev) [00:56:25] Glaisher: I'm (almost) sure that we won't change our mind [00:56:31] (03PS8) 10PleaseStand: Remove $wgBrowserBlackList and associated EditPage code [core] - 10https://gerrit.wikimedia.org/r/138840 (https://bugzilla.wikimedia.org/65297) [00:56:38] anyway have to go :p [00:57:07] (03Merged) 10jenkins-bot: Add poll delay and ignore repeating last seen change [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202617 (owner: 10Smalyshev) [00:59:27] 10MediaWiki-extensions-PageImages: Add support for finding a freely licensed page image - https://phabricator.wikimedia.org/T95378#1188204 (10Deskana) 3NEW [01:02:39] 10Wikimedia-Extension-setup, 7I18n: Install Josa extension parser function on all Korean language wikis - https://phabricator.wikimedia.org/T15712#1188217 (10csteipp) [01:02:42] 10MediaWiki-extensions-Josa, 10Security-Reviews: Security review for Extension:Josa - https://phabricator.wikimedia.org/T88261#1188215 (10csteipp) 5Open>3Resolved Coding should follow style guide, otherwise it looks ok [01:05:48] (03PS6) 10BryanDavis: Install non-vendor services to /vagrant/srv [vagrant] - 10https://gerrit.wikimedia.org/r/200624 (https://phabricator.wikimedia.org/T89919) (owner: 10Dduvall) [01:05:50] (03PS1) 10BryanDavis: Cleanup .gitignore [vagrant] - 10https://gerrit.wikimedia.org/r/202638 [01:05:52] (03PS1) 10BryanDavis: Update run-git-update for new /vagrant/srv directory [vagrant] - 10https://gerrit.wikimedia.org/r/202639 [01:06:34] 10MediaWiki-extensions-Graph, 6Services, 6operations, 10service-template-node, 7service-runner: Deploy graphoid service into production - https://phabricator.wikimedia.org/T90487#1188226 (10Yurik) [01:06:37] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 6Services: Improve Graphoid code - https://phabricator.wikimedia.org/T93881#1188224 (10Yurik) 5Open>3Resolved Please file additional issues as separate tasks. [01:10:02] (03CR) 10BryanDavis: [C: 031] "This is working for me locally. As a bonus I placed the clones previously in /phab/* into /srv* directly to solve the same problem as I3f" [vagrant] - 10https://gerrit.wikimedia.org/r/200624 (https://phabricator.wikimedia.org/T89919) (owner: 10Dduvall) [01:10:33] (03CR) 10PleaseStand: [C: 04-1] "Updated patch to apply to 1.26. I am still considering whether to add a Unicode check field to prevent corruption by Lynx, and whether to " [core] - 10https://gerrit.wikimedia.org/r/138840 (https://bugzilla.wikimedia.org/65297) (owner: 10PleaseStand) [01:10:40] (03PS1) 10Aaron Schulz: Log huge write queries in CLI scripts [core] - 10https://gerrit.wikimedia.org/r/202640 [01:11:50] 4(no projects): Make Share a Fact cards only use free images - https://phabricator.wikimedia.org/T95379#1188236 (10Deskana) 3NEW [01:12:06] 10MediaWiki-extensions-PageImages: Add support for finding a freely licensed page image - https://phabricator.wikimedia.org/T95378#1188244 (10Deskana) [01:12:08] 4(no projects): Make Share a Fact cards only use free images - https://phabricator.wikimedia.org/T95379#1188243 (10Deskana) [01:13:24] 10MediaWiki-extensions-WikiHiero, 6Zero: Zero: prepare for WikiHiero changes - https://phabricator.wikimedia.org/T93808#1188250 (10MaxSem) Note that if the move of the APIs to the main domains per T95229 will happen, users will not be charged in any case, but still need to assess the impact on traffic usage an... [01:16:14] (03CR) 10Sbisson: [C: 031 V: 031] "Looks good but I don't have enough context to guarantee it doesn't have any undesired side-effects." [vagrant] - 10https://gerrit.wikimedia.org/r/200624 (https://phabricator.wikimedia.org/T89919) (owner: 10Dduvall) [01:21:37] (03PS4) 10PleaseStand: update.php: Correctly report durations >= 1 hour [core] - 10https://gerrit.wikimedia.org/r/135433 (https://bugzilla.wikimedia.org/65772) [01:21:44] (03PS1) 10Mattflaschen: Add missing import for WikitextContent. [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202643 [01:23:03] (03PS2) 10Mattflaschen: Add missing import for WikitextContent [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202643 [01:24:10] (03CR) 10PleaseStand: "Rebased. Removed the release note since as far as I know this is not a user-reported issue or a change that is likely to affect users, and" [core] - 10https://gerrit.wikimedia.org/r/135433 (https://bugzilla.wikimedia.org/65772) (owner: 10PleaseStand) [01:24:12] (03CR) 10Sbisson: [C: 032 V: 031] Add missing import for WikitextContent [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202643 (owner: 10Mattflaschen) [01:24:32] (03PS5) 10PleaseStand: update.php: Correctly report durations >= 1 hour [core] - 10https://gerrit.wikimedia.org/r/135433 (https://phabricator.wikimedia.org/T67772) [01:26:05] (03PS9) 10PleaseStand: Remove $wgBrowserBlackList and associated EditPage code [core] - 10https://gerrit.wikimedia.org/r/138840 (https://phabricator.wikimedia.org/T67297) [01:26:17] 10MediaWiki-extensions-PageImages: Add support for finding a freely licensed page image - https://phabricator.wikimedia.org/T95378#1188268 (10Deskana) [01:26:50] (03CR) 10Mattflaschen: "This was introduced in https://gerrit.wikimedia.org/r/#/c/190476/ , and didn't get caught until now because apparently no one used --redir" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202643 (owner: 10Mattflaschen) [01:26:59] (03CR) 10PleaseStand: [C: 04-1] "This just adds the Phabricator task number to the commit message and release note. I am still considering whether to add a Unicode check f" [core] - 10https://gerrit.wikimedia.org/r/138840 (https://phabricator.wikimedia.org/T67297) (owner: 10PleaseStand) [01:27:55] (03Merged) 10jenkins-bot: Add missing import for WikitextContent [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202643 (owner: 10Mattflaschen) [01:28:00] 10MediaWiki-extensions-PageImages: Add support for finding a freely licensed page image - https://phabricator.wikimedia.org/T95378#1188204 (10Deskana) [01:28:18] 10UI-Standardization: mw-ui-buttons should have a max width - https://phabricator.wikimedia.org/T95367#1188271 (10Prtksxna) @jdlrobson, do you think changing the width of `.anonwarning` would make sense. If we add a `max-width` to the buttons they'll float on the left and I am not sure if that looks nice. | {F... [01:33:39] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Need-volunteer: Tag protected items with a padlock - https://phabricator.wikimedia.org/T64554#1188283 (10Ricordisamoa) [01:37:11] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Need-volunteer: Tag protected items with a padlock - https://phabricator.wikimedia.org/T64554#1188288 (10Ricordisamoa) Any reason why this isn't a dup of T92384 or T12347? [01:38:45] 10UI-Standardization: mw-ui-buttons should have a max width - https://phabricator.wikimedia.org/T95367#1188295 (10Jdlrobson) Mm. Basically the issue happens when you make a button display block (e.g. stacked buttons) we could add a margin auto to circumvent this issue in MobileFrontend so I don't think it is a b... [01:39:58] (03PS1) 10Manybubbles: Make sure property edits show up in recent changes [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 [01:41:28] (03PS2) 10Manybubbles: Make sure property edits show up in recent changes [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 [01:58:15] 6MediaWiki-Core-Team, 6Security, 5Patch-For-Review: Stored XSS in SVG via embedded SVG - https://phabricator.wikimedia.org/T85850#1188304 (10csteipp) Asasigned CVE-2015-2931 > Use CVE-2015-2931 for this issue involving an incomplete list of disallowed MIME types for data: URIs (the application/xml type wasn... [01:58:29] 6MediaWiki-Core-Team, 6Security, 5Patch-For-Review: Stored XSS in SVG via embedded SVG - https://phabricator.wikimedia.org/T85850#1188305 (10csteipp) [01:58:50] 6MediaWiki-Core-Team, 6Security, 5Patch-For-Review, 7Vuln-XSS: Fix SVG blacklist for animate - https://phabricator.wikimedia.org/T86711#1188306 (10csteipp) [01:59:09] 6MediaWiki-Core-Team, 6Security, 5Patch-For-Review, 7Vuln-XSS: Fix SVG blacklist for animate - https://phabricator.wikimedia.org/T86711#974689 (10csteipp) > Use CVE-2015-2932 for this issue involving an incomplete list of dangerous parts of HTML5. (The list is supposed to include all uses of 'animate attri... [01:59:33] 6MediaWiki-Core-Team, 6Security, 5Patch-For-Review, 7Vuln-XSS: XSS in language converter when used with Html class's tricky escaping - https://phabricator.wikimedia.org/T73394#1188308 (10csteipp) [02:12:33] 6MediaWiki-Core-Team, 6Availability-Team: Look into Maria 10 parallel-replication - https://phabricator.wikimedia.org/T85266#1188318 (10aaron) Tempted to at least partly blame the misc::maintenance::purge_abusefilter job. It runs at 1AM daily and uses 'LIMIT' with UPDATE, which we don't support so the LIMIT is... [02:17:34] (03CR) 10GOIII: [C: 031] Try to unbreak WikiEditor modules [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/199387 (https://phabricator.wikimedia.org/T93384) (owner: 10Alex Monk) [02:19:11] 4(no projects): Investigate 1AM enwiki slave lag spikes - https://phabricator.wikimedia.org/T95382#1188327 (10aaron) 3NEW a:3aaron [02:24:11] 10Hovercards, 10Wikimedia-Extension-setup, 3Roadmap, 7notice: Turn off 2 month test release of Hovercards on Chinese, Catalan, and Greek Wikipedias on 2015-06-18 - https://phabricator.wikimedia.org/T92555#1188341 (10Quiddity) One of my CL colleagues pointed out that it would be ideal to Not Turn Off hoverc... [02:24:51] 6Community-Liaison, 10Hovercards: Contact the 3 Hovercard trial communities, to assess their wishes for post-trial configuration (May 15) - https://phabricator.wikimedia.org/T95383#1188343 (10Quiddity) 3NEW a:3Quiddity [02:27:32] (03PS1) 10Aaron Schulz: Fixed broken batching in –PurgeOldLogIPData [extensions/AbuseFilter] - 10https://gerrit.wikimedia.org/r/202657 (https://phabricator.wikimedia.org/T95382) [02:31:18] 5Patch-For-Review: Investigate 1AM enwiki slave lag spikes - https://phabricator.wikimedia.org/T95382#1188356 (10aaron) [02:33:00] (03Abandoned) 10Aaron Schulz: [WIP] Made slave-lag induced read-only mode more robust [core] - 10https://gerrit.wikimedia.org/r/202358 (owner: 10Aaron Schulz) [02:33:47] (03PS2) 10Aaron Schulz: Added CAS logic to User::addAutopromoteOnceGroups [core] - 10https://gerrit.wikimedia.org/r/202589 (https://phabricator.wikimedia.org/T48834) [02:38:31] (03CR) 10jenkins-bot: [V: 04-1] Added CAS logic to User::addAutopromoteOnceGroups [core] - 10https://gerrit.wikimedia.org/r/202589 (https://phabricator.wikimedia.org/T48834) (owner: 10Aaron Schulz) [02:43:48] 10MediaWiki-extensions-Newsletter, 6Engineering-Community, 10Possible-Tech-Projects, 10Wikimedia-Hackathon-2015, and 2 others: Possible Project for a Newsletter MediaWiki extension - https://phabricator.wikimedia.org/T76199#1188365 (10Quiddity) goodreads.com is my current ideal model - just look at this de... [02:51:10] (03PS3) 10Aaron Schulz: Added CAS logic to User::addAutopromoteOnceGroups [core] - 10https://gerrit.wikimedia.org/r/202589 (https://phabricator.wikimedia.org/T48834) [02:51:33] (03PS1) 10Manybubbles: Switch clear from DELETE to CLEAR [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202658 [03:04:36] YuviPanda: No idea how jenkin's jobs work so I don't know how effective I'll be at solving https://gerrit.wikimedia.org/r/#/c/199545/ [03:05:04] Negative24: legoktm or krinkle or mutante should be able to help, I guess [03:05:07] * YuviPanda has no idea either [03:05:32] (03PS5) 10Manybubbles: Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 [03:06:43] (03PS1) 10Aaron Schulz: Made User::invalidateCache() use touch() instead of the DB query [core] - 10https://gerrit.wikimedia.org/r/202659 [03:09:39] (03PS1) 10Manybubbles: Cleanup docs on Ontology [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202660 [03:09:48] YuviPanda: I will see about getting a bug report filed on that. I'll CC them on it [03:15:16] (03PS4) 10Aaron Schulz: Added CAS logic to User::addAutopromoteOnceGroups [core] - 10https://gerrit.wikimedia.org/r/202589 (https://phabricator.wikimedia.org/T48834) [03:54:47] (03CR) 10BryanDavis: "Good start. I made some notes inline." (035 comments) [wikimedia/iegreview] - 10https://gerrit.wikimedia.org/r/202090 (https://phabricator.wikimedia.org/T87099) (owner: 10Niharika29) [04:25:32] (03PS1) 10Manybubbles: Compress Entity and EntityData into WikibaseUris [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 [04:32:03] 6WMF-NDA-Requests: Allow Valhallasw to sign an NDA - https://phabricator.wikimedia.org/T93644#1188432 (10Eloquence) Per Mark's endorsement, this NDA request is approved. [04:32:46] (03CR) 10Smalyshev: [C: 032] Cleanup docs on Ontology [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202660 (owner: 10Manybubbles) [04:33:44] (03CR) 10Smalyshev: [C: 032] Switch clear from DELETE to CLEAR [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202658 (owner: 10Manybubbles) [04:34:28] (03Merged) 10jenkins-bot: Cleanup docs on Ontology [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202660 (owner: 10Manybubbles) [04:36:09] (03Merged) 10jenkins-bot: Switch clear from DELETE to CLEAR [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202658 (owner: 10Manybubbles) [04:46:56] 10MediaWiki-Extension-Requests, 10Possible-Tech-Projects, 6Wiktionary: Extension to provide access via the dict protocol - https://phabricator.wikimedia.org/T31229#1188458 (10Ricordisamoa) >>! In T31229#1185939, @Lydia_Pintscher wrote: > Agreed. With whom? On what? [04:47:46] 6Phabricator: make gerritbot remove the "patch-for-review" tag once a patch is merged (or abandoned) - https://phabricator.wikimedia.org/T95309#1188460 (10MZMcBride) >>! In T95309#1186316, @mmodell wrote: > Is there even a project in phabricator for the gerritbot? Don't think so. I think most people use #Wikime... [04:52:45] (03CR) 10Smalyshev: [C: 032] Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [04:52:47] (03CR) 10jenkins-bot: [V: 04-1] Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [04:55:03] (03PS6) 10Smalyshev: Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [04:55:17] (03CR) 10Smalyshev: "rebasing" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [05:00:59] (03PS2) 10MarkAHershberger: WIP: Initial work on HitCounters extension. [extensions/HitCounters] - 10https://gerrit.wikimedia.org/r/202629 (https://phabricator.wikimedia.org/T74420) [05:03:32] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5: [citation needed] is not adapted. - https://phabricator.wikimedia.org/T95391#1188466 (10KartikMistry) 3NEW [05:06:39] (03CR) 10Smalyshev: [C: 032] Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [05:08:10] (03Merged) 10jenkins-bot: Process dump files [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202405 (owner: 10Manybubbles) [05:09:51] 3Google-Summer-of-Code-2015, 3Outreachy-Round-10: Select participants for GSoC 2015 and Outreachy round 10 - https://phabricator.wikimedia.org/T91374#1188480 (10NiharikaKohli) Hey @Qgil. Hope your vacation was fun. :) Not really. When you have a bit of spare time look at the number of slots I asked for in Mel... [05:21:35] 10MediaWiki-extensions-SecurePoll: Update SecurePoll bv2013 scripts for the 2015 WMF elections - https://phabricator.wikimedia.org/T95262#1188481 (10tstarling) >>! In T95262#1184798, @Jalexander wrote: > I need the list to be CHECKED globally That's an architecture statement, not a product statement. > For exa... [05:57:06] (03CR) 10Niharika29: [WIP] Add a report by reviewer for a campaign (031 comment) [wikimedia/iegreview] - 10https://gerrit.wikimedia.org/r/202090 (https://phabricator.wikimedia.org/T87099) (owner: 10Niharika29) [05:58:35] (03PS5) 1001tonythomas: Removed hard coded punctuations from [[Special:oauthlistconsumers]] [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/185611 (https://phabricator.wikimedia.org/T68832) [06:01:09] (03PS1) 1001tonythomas: Removed unused variable $time from OAuthListConsumers [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/202678 [06:02:34] (03PS2) 1001tonythomas: Removed unused variable $time from OAuthListConsumers [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/202678 [06:08:15] 10MediaWiki-extensions-ContentTranslation, 3LE-Sprint-85: Segment highlighting getting saved as part of drafts - https://phabricator.wikimedia.org/T95394#1188503 (10santhosh) 3NEW a:3santhosh [06:14:20] Anyone know where the HTML for https://doc.wikimedia.org/ lives? [06:14:50] The first link on that page (MediaWiki PHP) is broken, it needs the trailing /html removed from it [06:15:57] tto: i'll teach you a trick Krinkle showed me once [06:16:04] 10MediaWiki-extensions-ContentTranslation, 3LE-Sprint-85, 5Patch-For-Review: Segment highlighting getting saved as part of drafts - https://phabricator.wikimedia.org/T95394#1188516 (10Amire80) I wasn't crazy: T90202, T87511 :) [06:16:20] pick a unique phrase from the page (i went with "MediaWiki-Vagrant puppet") [06:16:36] then go to github.com/search , and search for: @wikimedia "MediaWiki-Vagrant puppet" [06:16:42] the @wikimedia scopes it to wikimedia repos [06:16:51] one result: https://github.com/wikimedia/integration-docroot/blob/9d6e10c77aa801aafa8fe36fc1e5de2e3bb561f8/org/wikimedia/doc/default.html [06:17:00] ori: Hm, nice trick! [06:17:12] yeah comes in handy all the time [06:17:31] I'll keep it in mind [06:17:34] Thanks [06:17:39] np [06:18:50] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 3§ Wikidata-Sprint-2015-03-24, 7Documentation, and 2 others: update documentation of high-level data model - https://phabricator.wikimedia.org/T75603#1188519 (10Tobi_WMDE_SW) [06:22:39] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: History and diff pages should show labels of properties and items in edit summaries - https://phabricator.wikimedia.org/T55247#1188521 (10Tobi_WMDE_SW) a:3hoo [06:26:02] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 3§ Wikidata-Sprint-2015-03-24, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: Add language fallback for badge labels - https://phabricator.wikimedia.org/T94552#1188524 (10Tobi_WMDE_SW) a:3hoo [06:26:28] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: allow arbitrary data access on Wikidata (parser function) - https://phabricator.wikimedia.org/T70029#1188526 (10Tobi_WMDE_SW) a:3hoo [06:28:12] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 3§ Wikidata-Sprint-2015-03-24, 5Patch-For-Review, and 2 others: Allow simultaneous editing of mainsnak and references - https://phabricator.wikimedia.org/T87759#1188531 (10Tobi_WMDE_SW) a:3thiemowmde [06:34:31] 10MediaWiki-RfCs, 10Wikimedia-General-or-Unknown: cleaner URLs for editing, previewing skins, etc. - https://phabricator.wikimedia.org/T14619#1188540 (10Nemo_bis) [06:40:31] 10Wikidata-Query-Service: Validate how the updater determines where it left off - https://phabricator.wikimedia.org/T95194#1188544 (10Smalyshev) There's another peculiar problem here. The [[ http://en.wikipedia.org/w/api.php?action=help&modules=query%2Brecentchanges | API ]] is driven by timestamps, however seve... [06:45:16] (03PS3) 10Robert Vogel: [WIP] examples/BoilerPlate [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/202396 [06:47:26] 10MediaWiki-extensions-SecurePoll: Update SecurePoll bv2013 scripts for the 2015 WMF elections - https://phabricator.wikimedia.org/T95262#1188555 (10Jalexander) >>! In T95262#1188481, @tstarling wrote: >>>! In T95262#1184798, @Jalexander wrote: >> I need the list to be CHECKED globally > > That's an architectur... [06:50:35] (03CR) 10Springle: [C: 031] Set a query group on SpecialRecentChanges query [core] - 10https://gerrit.wikimedia.org/r/202632 (owner: 10Aaron Schulz) [06:52:11] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-cxserver, 3LE-Sprint-85, 5Patch-For-Review: cxserver fails to load pages with question marks in URL - https://phabricator.wikimedia.org/T94667#1169410 (10santhosh) [06:52:21] 10MediaWiki-extensions-ContentTranslation, 3LE-Sprint-85, 5Patch-For-Review: Segment highlighting getting saved as part of drafts - https://phabricator.wikimedia.org/T95394#1188503 (10santhosh) [06:52:28] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-Deployments, 5ContentTranslation-Release5, 3LE-Sprint-85, and 4 others: Enable Swedish as target language for ContentTranslation - https://phabricator.wikimedia.org/T95108#1188579 (10santhosh) [06:52:39] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-Deployments, 5ContentTranslation-Release5, 3LE-Sprint-85, 5Patch-For-Review: Enable 'newarticle' campaign in all CX enabled wikis - https://phabricator.wikimedia.org/T95147#1188580 (10santhosh) [06:52:50] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5, 7Easy, 3LE-Sprint-85, 5Patch-For-Review: Red interlanguage links entry point to CX should not be shown when it does not make sense - https://phabricator.wikimedia.org/T94845#1188581 (10santhosh) [06:54:34] (03PS1) 10Mjbmr: Update to CLDR 27.0.1 [extensions/cldr] - 10https://gerrit.wikimedia.org/r/202684 [06:55:33] (03PS5) 10Aaron Schulz: Added CAS logic to User::addAutopromoteOnceGroups [core] - 10https://gerrit.wikimedia.org/r/202589 (https://phabricator.wikimedia.org/T48834) [06:57:33] 6WMF-Design-Research, 10MobileFrontend-Feature-requests, 5Patch-For-Review: Provide VisualEditor regardless of device size - https://phabricator.wikimedia.org/T85630#1188597 (10Jdforrester-WMF) [06:58:58] (03CR) 10Matthias Mullie: "I originally started that way, but it seemed like that would be much more confusing:" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202339 (https://phabricator.wikimedia.org/T90681) (owner: 10Matthias Mullie) [06:59:36] (03PS1) 10Robert Vogel: Code improvements [skins/BlueSpiceSkin] - 10https://gerrit.wikimedia.org/r/202685 [07:00:27] 7Design, 6Mobile-Web, 10MobileFrontend, 5Patch-For-Review: Remove mobile editing "call to registration" CTA - https://phabricator.wikimedia.org/T87508#1188627 (10Nemo_bis) [07:00:30] 10MediaWiki-extensions-Collection, 6Mobile-Web, 10MobileFrontend, 6Wikisource, 7Mobile: download a book from mobile should be possible - https://phabricator.wikimedia.org/T62250#1188631 (10Nemo_bis) [07:00:34] 7Design, 6Mobile-Web, 10MobileFrontend, 7Browser-Support-Firefox: Editing window text area indistinguishable from uneditable pages - https://phabricator.wikimedia.org/T71093#1188629 (10Nemo_bis) [07:00:37] 10MediaWiki-extensions-Collection, 6Mobile-Web, 10MobileFrontend, 6Wikisource, 7Mobile: A link should be provided to create a book from mobile - https://phabricator.wikimedia.org/T62249#1188633 (10Nemo_bis) [07:02:13] 10MediaWiki-Extension-Requests, 10MediaWiki-Special-pages, 6Mobile-Web, 10MobileFrontend, and 3 others: Move MobileFrontend extension's Special:UserProfile.php and related code to a separate MediaWiki extension - https://phabricator.wikimedia.org/T85929#1188638 (10Nemo_bis) [07:06:04] (03PS2) 10Aaron Schulz: Log huge write queries in CLI scripts [core] - 10https://gerrit.wikimedia.org/r/202640 (https://phabricator.wikimedia.org/T95382) [07:06:15] 10Wikimedia-General-or-Unknown: Use GPS location to deliver locally relevant messages about copyright (for people uploading pictures) - https://phabricator.wikimedia.org/T70770#1188655 (10Nemo_bis) [07:06:22] 6Availability-Team, 5Patch-For-Review: Investigate 1AM enwiki slave lag spikes - https://phabricator.wikimedia.org/T95382#1188656 (10aaron) [07:07:14] 10Wikimedia-General-or-Unknown: Use GPS location to deliver locally relevant messages about copyright (for people uploading pictures) - https://phabricator.wikimedia.org/T70770#721123 (10Nemo_bis) MobileFrontend's special upload features have been disabled and GeoIP is available for all features in Wikimedia pro... [07:07:40] (03CR) 10Springle: [C: 031] Made mergeActiveUsers use "recentchanges" query group [core] - 10https://gerrit.wikimedia.org/r/202634 (owner: 10Aaron Schulz) [07:08:49] 10MediaWiki-extensions-AbuseFilter, 6Availability-Team, 5Patch-For-Review: Investigate 1AM enwiki slave lag spikes - https://phabricator.wikimedia.org/T95382#1188659 (10Se4598) [07:12:11] 6Phabricator: make gerritbot remove the "patch-for-review" tag once a patch is merged (or abandoned) - https://phabricator.wikimedia.org/T95309#1188660 (10mmodell) @MZMcBride: There is still desire to transition to differential but it seems like there is a lot of FUD as well. Personally I've just been assigned w... [07:12:53] (03CR) 10Awight: Helper to get a page's revision ID given an approximate time (031 comment) [core] - 10https://gerrit.wikimedia.org/r/201929 (https://phabricator.wikimedia.org/T58849) (owner: 10Awight) [07:14:22] (03CR) 10Awight: Helper to get a page's revision ID given an approximate time (031 comment) [core] - 10https://gerrit.wikimedia.org/r/201929 (https://phabricator.wikimedia.org/T58849) (owner: 10Awight) [07:17:04] 3Google-Summer-of-Code-2015: Allow contributors to update their own details in tech metrics directly - https://phabricator.wikimedia.org/T93943#1188671 (10NiharikaKohli) Have you completed any microtasks for this project? Or are you working on any? Gerrit links welcome. [07:17:47] 7Design, 10UI-Standardization, 6Mobile-Web, 10MobileFrontend, and 2 others: Identify the best diff style and use it everywhere - https://phabricator.wikimedia.org/T90948#1188675 (10Nemo_bis) [07:19:20] 10MediaWiki-extensions-ContentTranslation, 10Beta-Cluster, 10ContentTranslation-Deployments, 5ContentTranslation-Release5, 3LE-Sprint-85: Setup new wikis in Beta Cluster for Content Translation - https://phabricator.wikimedia.org/T90683#1188766 (10Arrbee) See also T93213 [07:22:10] (03PS1) 10Tweichart: small change for hook result [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/202687 [07:26:48] (03CR) 10Pwirth: [C: 031] small change for hook result [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/202687 (owner: 10Tweichart) [07:28:54] (03CR) 10Tweichart: [C: 032] small change for hook result [extensions/BlueSpiceExtensions] - 10https://gerrit.wikimedia.org/r/202687 (owner: 10Tweichart) [07:29:02] (03PS1) 10Tweichart: small change for hook result [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/202688 [07:29:10] (03CR) 10Tweichart: [C: 032] small change for hook result [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/202688 (owner: 10Tweichart) [07:30:12] (03Merged) 10jenkins-bot: small change for hook result [extensions/BlueSpiceExtensions] (REL1_23) - 10https://gerrit.wikimedia.org/r/202688 (owner: 10Tweichart) [07:38:38] (03PS2) 10Mjbmr: Update to CLDR 27.0.1 [extensions/cldr] - 10https://gerrit.wikimedia.org/r/202684 [07:39:27] 10MediaWiki-extensions-ContentTranslation, 6Collaboration-Team, 10Flow, 10VisualEditor, and 3 others: HTML entities in CommentNode must be manually escaped/unescaped. - https://phabricator.wikimedia.org/T95040#1188778 (10Jdforrester-WMF) [07:39:45] 10MediaWiki-extensions-ContentTranslation, 6Collaboration-Team, 10Flow, 10VisualEditor, and 3 others: HTML entities in CommentNode must be manually escaped/unescaped - https://phabricator.wikimedia.org/T95040#1188779 (10Jdforrester-WMF) 5Open>3Resolved [07:40:51] 7Design, 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: Preview interface for link dialogue - https://phabricator.wikimedia.org/T88316#1188783 (10Jdforrester-WMF) [07:41:01] 7Design, 10MediaWiki-extensions-GuidedTour, 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: Change Welcome dialogue content to more helpful message to new users based on the GuidedTour - https://phabricator.wikimedia.org/T89072#1188785 (10Jdforrester-WMF) [07:41:54] 6MediaWiki-Core-Team, 6Availability-Team: Look into Maria 10 parallel-replication - https://phabricator.wikimedia.org/T85266#1188788 (10Springle) db1071 and 72 QPS loads look high, but they aren't really struggling even at peak. I'd tend to agree that most replag spikes are due to slow writes, and not individu... [07:49:26] (03PS3) 10Mjbmr: Update to CLDR 27.0.1 [extensions/cldr] - 10https://gerrit.wikimedia.org/r/202684 [07:53:05] 7Design, 6Mobile-Web, 10MobileFrontend: MobileFrontend "last edited" strapline should not include authorship information - https://phabricator.wikimedia.org/T66921#1188790 (10Nemo_bis) 5declined>3Open p:5Triage>3Normal [07:54:11] 7Design, 6Mobile-Web, 10MobileFrontend: MobileFrontend "last edited" strapline should not include authorship information - https://phabricator.wikimedia.org/T66921#1188796 (10Nemo_bis) [07:58:35] Hey, anybody an admin on testwiki? [07:58:43] well, bureaucrat [07:59:11] 10MediaWiki-extensions-DonationInterface, 10Fundraising Tech Backlog: Move DonationInterface test fixtures into pure data files. - https://phabricator.wikimedia.org/T95397#1188800 (10awight) 3NEW [08:00:47] 6MediaWiki-Core-Team, 6Availability-Team: Look into Maria 10 parallel-replication - https://phabricator.wikimedia.org/T85266#1188808 (10Springle) Re MariaDB 10 benefits: There isn't any obvious leap in performance to point to compared to 5.5. Some stuff is faster, some isn't. The biggest benefits have been:... [08:13:43] werdna: yes [08:13:55] never mind, I sorted it out [08:13:59] just did something minorly naughty [08:13:59] ok [08:14:17] werdna: you didn't flag yourself via DB change, did you [08:14:26] no, using my volunteer account ;-) [08:14:29] ;) [08:21:42] (03PS4) 10Awight: Helper to get a page's revision ID given an approximate time [core] - 10https://gerrit.wikimedia.org/r/201929 (https://phabricator.wikimedia.org/T58849) [08:29:34] 10MediaWiki-Page-editing, 7Epic, 5Patch-For-Review: Edit conflict detection suffers a race condition - https://phabricator.wikimedia.org/T58849#1188826 (10Nemo_bis) [08:36:02] (03PS4) 10Mjbmr: Update to CLDR 27.0.1 [extensions/cldr] - 10https://gerrit.wikimedia.org/r/202684 [08:37:14] 7Design, 6Collaboration-Team, 10Flow, 7Community-consensus-needed: preference to ignore comments from specific users ("killfile") - https://phabricator.wikimedia.org/T71404#1188846 (10Liuxinyu970226) [08:46:12] (03PS7) 10Awight: Add tests to exercise a language bug in DerivativeContext [core] - 10https://gerrit.wikimedia.org/r/57018 (https://bugzilla.wikimedia.org/46766) [08:52:20] (03CR) 10jenkins-bot: [V: 04-1] Add tests to exercise a language bug in DerivativeContext [core] - 10https://gerrit.wikimedia.org/r/57018 (https://bugzilla.wikimedia.org/46766) (owner: 10Awight) [08:53:45] (03PS5) 10Mjbmr: Update to CLDR 27.0.1 [extensions/cldr] - 10https://gerrit.wikimedia.org/r/202684 [08:53:51] 10MediaWiki-Vagrant, 10MediaWiki-extensions-Graph, 6Services: Update Graphoid Puppet role/module in MW-Vagrant - https://phabricator.wikimedia.org/T95399#1188880 (10mobrovac) 3NEW a:3Yurik [08:57:28] (03CR) 10Yurik: [C: 04-1] "Minor fixes, and 2 refactoring suggestions" (036 comments) [vagrant] - 10https://gerrit.wikimedia.org/r/195106 (https://phabricator.wikimedia.org/T91661) (owner: 10Mobrovac) [08:57:43] (03PS6) 10Mjbmr: Update to CLDR 27.0.1 [extensions/cldr] - 10https://gerrit.wikimedia.org/r/202684 [08:59:30] (03PS8) 10Awight: WIP Add tests to exercise a language bug in DerivativeContext [core] - 10https://gerrit.wikimedia.org/r/57018 (https://bugzilla.wikimedia.org/46766) [09:01:52] (03CR) 10Mobrovac: Add the RESTBase and Cassandra roles to MW-Vagrant (032 comments) [vagrant] - 10https://gerrit.wikimedia.org/r/195106 (https://phabricator.wikimedia.org/T91661) (owner: 10Mobrovac) [09:05:41] (03CR) 10jenkins-bot: [V: 04-1] WIP Add tests to exercise a language bug in DerivativeContext [core] - 10https://gerrit.wikimedia.org/r/57018 (https://bugzilla.wikimedia.org/46766) (owner: 10Awight) [09:14:01] (03PS1) 10UltrasonicNXT: trim() the thread title [extensions/WikiForum] - 10https://gerrit.wikimedia.org/r/202693 [09:14:58] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 6Services: Improve Graphoid code - https://phabricator.wikimedia.org/T93881#1188903 (10mobrovac) [09:14:59] 10MediaWiki-extensions-Graph, 6Services: Write Graphoid-specific tests - https://phabricator.wikimedia.org/T93921#1188901 (10mobrovac) 5Resolved>3Open [09:15:08] (03PS2) 10UltrasonicNXT: trim() the thread title [extensions/WikiForum] - 10https://gerrit.wikimedia.org/r/202693 [09:15:30] 10MediaWiki-extensions-Graph, 6Services: Write Graphoid-specific tests - https://phabricator.wikimedia.org/T93921#1149737 (10mobrovac) Reopening as this needs more work [09:15:42] (03CR) 10UltrasonicNXT: [C: 032 V: 032] trim() the thread title [extensions/WikiForum] - 10https://gerrit.wikimedia.org/r/202693 (owner: 10UltrasonicNXT) [09:16:07] 10MediaWiki-extensions-Graph, 6Services, 6operations, 10service-template-node, 7service-runner: Deploy graphoid service into production - https://phabricator.wikimedia.org/T90487#1188907 (10mobrovac) [09:16:10] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 6Services: Improve Graphoid code - https://phabricator.wikimedia.org/T93881#1188905 (10mobrovac) 5Resolved>3Open Reopening due to unmet dependencies [09:20:53] 10MediaWiki-extensions-Graph, 6Services: Write Graphoid-specific tests - https://phabricator.wikimedia.org/T93921#1188910 (10Yurik) @mobrovac, please document this in a on-wiki guide - it has never been mentioned before. [09:42:03] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 6Services: Improve Graphoid code - https://phabricator.wikimedia.org/T93881#1188926 (10Yurik) [09:42:05] 10MediaWiki-extensions-Graph, 6Services: Write Graphoid-specific tests - https://phabricator.wikimedia.org/T93921#1188924 (10Yurik) 5Open>3Resolved @mobrovac, The code coverage for v1.js already passes your requirement: ``` Statements: 80.58% (166 / 206) Branches: 61.54% (56 / 91) Functions: 78.1... [09:58:32] (03PS2) 10Werdna: Make Parsoid work with multiwiki. [vagrant] - 10https://gerrit.wikimedia.org/r/199591 [09:58:35] (03CR) 10jenkins-bot: [V: 04-1] Make Parsoid work with multiwiki. [vagrant] - 10https://gerrit.wikimedia.org/r/199591 (owner: 10Werdna) [09:59:27] (03PS3) 10Werdna: Make Parsoid work with multiwiki. [vagrant] - 10https://gerrit.wikimedia.org/r/199591 [10:00:10] 10MediaWiki-Installer, 10MediaWiki-Tarball-Backports, 5MW-1.24-release, 5Patch-For-Review: Missing defaults in GUI installer - https://phabricator.wikimedia.org/T71281#1188945 (10Umherirrender) 5Open>3Resolved [10:05:56] 10MediaWiki-extensions-Graph, 10MediaWiki-extensions-WikiHiero, 10Citoid, 10Mathoid-General-or-Unknown, and 3 others: Services need to share the same domain name - https://phabricator.wikimedia.org/T95402#1188961 (10Yurik) 3NEW [10:08:11] (03PS1) 10Prtksxna: add bootstrap [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202698 [10:09:17] 10MediaWiki-Search, 7I18n: A Hebrew article title with an apostrophe cannot be found when searching without an apostrophe - https://phabricator.wikimedia.org/T95404#1188979 (10Amire80) 3NEW [10:12:57] (03PS1) 10Umherirrender: Add test for IRC line for type protect/move_prot [core] - 10https://gerrit.wikimedia.org/r/202699 [10:14:45] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: edit links are displayed on old revisions - https://phabricator.wikimedia.org/T95405#1188988 (10aude) 3NEW a:3aude [10:16:25] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: browser tests for old revisions of Wikibase item pages to check there are no edit links - https://phabricator.wikimedia.org/T95406#1188999 (10aude) 3NEW [10:17:44] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1189007 (10aude) 3NEW [10:18:02] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1189007 (10aude) [10:18:36] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: edit links are displayed on old revisions - https://phabricator.wikimedia.org/T95405#1189017 (10aude) [10:26:11] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5, 5Patch-For-Review: Polish the CX Dashboard - https://phabricator.wikimedia.org/T88344#1189021 (10santhosh) @nirzar, I see you remove the relative timestamps from UI(3 days ago etc). Any reason for that? Also, the design shows the whole... [10:27:07] 10Wikidata-Query-Service: Validate how the updater determines where it left off - https://phabricator.wikimedia.org/T95194#1189023 (10JanZerebecki) As you pointed out its not nice, but if we use timestamp we only may ever skip last seen timestamp - 1. Otherwise one might accidentally skip changes that were made... [10:27:29] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5, 5Patch-For-Review: Polish the CX Dashboard - https://phabricator.wikimedia.org/T88344#1189024 (10santhosh) @Nirzar, Do you have a prototype for this? If not please give the color codes for important elements, also the hover interaction... [10:27:37] 6Phabricator, 10Wikimedia-Bugzilla, 6operations: Sanitise a Bugzilla database dump - https://phabricator.wikimedia.org/T85141#1189025 (10Aklapper) >>! In T85141#1187711, @Dzahn wrote: > Addtionally, @aklapper provided me with a list of IDs (not sure how he created that) Query in old-bz for all tasks in the... [10:27:43] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5, 5Patch-For-Review: Polish the CX Dashboard - https://phabricator.wikimedia.org/T88344#1189026 (10Pginer-WMF) @Amire80, no. Linking to Magnus tool is a workaround for the lack of translation suggestions. Magnus tool is a great tool but i... [10:38:00] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5, 5Patch-For-Review: Polish the CX Dashboard - https://phabricator.wikimedia.org/T88344#1189034 (10Amire80) >>! In T88344#1189026, @Pginer-WMF wrote: > @Amire80, no. Linking to Magnus tool is a workaround for the lack of translation sugge... [10:39:02] 10Wikimedia-Site-requests: Liangent's user page and user subpages on zhwiki - https://phabricator.wikimedia.org/T95408#1189035 (10Jianhui67) 3NEW a:3hoo [10:39:43] 10MediaWiki-extensions-Renameuser: Userpage revisions destroyed after global rename / strange pagemoves - https://phabricator.wikimedia.org/T95029#1189047 (10Aklapper) p:5Triage>3Normal [10:40:02] 10Wikimedia-Site-requests: Liangent's user page and user subpages on zhwiki - https://phabricator.wikimedia.org/T95408#1189048 (10hoo) [10:40:29] 6Learning-and-Evaluation: Programs Evaluation Subproject - https://phabricator.wikimedia.org/T91045#1189051 (10Aklapper) Ok, no problem, but how was this [[ https://www.mediawiki.org/wiki/Bug_management/Bug_report_life_cycle | resolved ]]? [10:41:39] 10Wikimedia-Site-requests: Lift IP rate limit - Santiago Editatón - 2015-04-11 and 2015-04-12 - https://phabricator.wikimedia.org/T95302#1189053 (10Aklapper) p:5Normal>3High [10:53:17] 7Design, 10VisualEditor, 10VisualEditor-EditingTools: Inspectors don't have close buttons but dialogs do, and some users find this confusing - https://phabricator.wikimedia.org/T93749#1189078 (10Jdforrester-WMF) p:5Triage>3Lowest [11:01:11] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Failed asserting that 'Could not find such an entity id (The serialization "noANid" is not recognized by the configured id builders)' contains "is not valid". - https://phabricator.wikimedia.org/T94672#1189104 (10aude) [11:01:13] 10MediaWiki-ContentHandler, 10MediaWiki-Internationalization, 10VisualEditor: Default content model should apply to non-customised messages in the MediaWiki: namespace - https://phabricator.wikimedia.org/T92899#1189106 (10Jdforrester-WMF) [11:01:25] 10MediaWiki-ContentHandler, 10MediaWiki-Internationalization, 10VisualEditor: Default content model should apply to non-customised messages in the MediaWiki: namespace - https://phabricator.wikimedia.org/T92899#1123086 (10Jdforrester-WMF) p:5Triage>3Normal [11:02:33] 6Phabricator, 3ECT-April-2015: Show percentage of teams migrated to Phabricator for project management - https://phabricator.wikimedia.org/T434#1189109 (10Qgil) [11:07:46] 10MediaWiki-extensions-ContentTranslation, 5ContentTranslation-Release5, 5Patch-For-Review: Polish the CX Dashboard - https://phabricator.wikimedia.org/T88344#1189116 (10Pginer-WMF) > here is the possibility that we'll have this redesigned/polished dashboard without suggestions for some time If there are no... [11:07:49] 6Phabricator: Obsolete attachments were not migrated from bugzilla - https://phabricator.wikimedia.org/T78747#1189117 (10Aklapper) We won't migrate obsolete attachments from Bugzilla to Phabricator. That's how I read the task summary and hence proposing "declined" status here. [11:08:36] 10MediaWiki-General-or-Unknown, 10VisualEditor: Typing "{{" does not show the template dialog - https://phabricator.wikimedia.org/T95410#1189118 (10Nnemo) 3NEW [11:09:38] 6Phabricator, 3ECT-April-2015: Show percentage of teams migrated to Phabricator for project management - https://phabricator.wikimedia.org/T434#1189129 (10Qgil) There was some progress on March. The Mobile Apps team is using Phabricator for their current sprints and general backlog, but still has the legacy... [11:11:55] 6Phabricator, 6Mobile-Apps: Migration of Mobile Apps team to Phabricator - https://phabricator.wikimedia.org/T851#1189145 (10Qgil) Thank you! Your progress has been reported in T434. If you want help migrating your Trello legacy cards, please contact @spage. [11:13:48] (03PS1) 10Aude: Update Wikidata - fix issue with diff view [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202702 [11:14:04] 6Phabricator: Migration of Mobile Web team to Phabricator - https://phabricator.wikimedia.org/T830#1189158 (10Qgil) For some reason we counted Gather as an own team, and now I don't want to touch those numbers now... It is very good to know that the move is getting closer. If you want help migrating your Trello... [11:14:47] 6Phabricator, 6WMF-Design, 6WMF-Design-Research, 6Team-Practices: Migration of the Design team to Phabricator - https://phabricator.wikimedia.org/T832#1189160 (10Qgil) OK, let's say 0.75, then. Your progress has been reported in T434. [11:16:40] 10Wikimedia Phabricator RfC: Identify features Bugzilla users would miss in Phabricator - https://phabricator.wikimedia.org/T22#1189178 (10Qgil) [11:16:41] 10Phabricator-Upstream: RSS feeds for saved bug searches - https://phabricator.wikimedia.org/T91#1189180 (10Qgil) [11:16:43] 10Wikimedia-Site-requests: Allow embedding RSS feeds from Phabricator into mediawiki.org - https://phabricator.wikimedia.org/T58287#1189179 (10Qgil) [11:16:45] 10Wikimedia Phabricator RfC: Identify features Mingle users would miss in Phabricator - https://phabricator.wikimedia.org/T20#1189181 (10Qgil) [11:16:47] 10Phabricator-Upstream: RSS/Atom feeds - https://phabricator.wikimedia.org/T67#1189175 (10Qgil) 5Open>3declined a:3Qgil Upstream has closed the related task as wontfix, see the explanation at https://secure.phabricator.com/T1928#103907 Resolving here accordingly. [11:20:37] 10MediaWiki-extensions-ContentTranslation, 3LE-Sprint-85: ULS IME does not work in Content Translation - https://phabricator.wikimedia.org/T88279#1189194 (10santhosh) [11:21:31] 10MediaWiki-General-or-Unknown, 10VisualEditor: The tool bar plays hide-and-and-seek-and-greyed, so the user cannot insert a special character - https://phabricator.wikimedia.org/T95414#1189199 (10Nnemo) 3NEW [11:21:52] 10Wikimedia-Site-requests: Allow embedding RSS feeds from Phabricator into mediawiki.org - https://phabricator.wikimedia.org/T58287#1189207 (10Qgil) As per T67 we cannot expect RSS feeds from Phabricator, and therefore this task as formulated must be declined as well. {T90432} seeks the same goal, but with a di... [11:22:34] 10Wikimedia-Site-requests: Allow embedding RSS feeds from Phabricator into mediawiki.org - https://phabricator.wikimedia.org/T58287#1189210 (10Qgil) 5Open>3declined a:3Qgil [11:23:30] 6Phabricator, 10Wikimania-Hackathon-2015, 10Wikimedia-Hackathon-2015: MediaWiki extension to embed Phabricator content in wiki pages - https://phabricator.wikimedia.org/T90432#1058674 (10Qgil) [11:24:16] (03CR) 10Aude: [C: 032] Update Wikidata - fix issue with diff view [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202702 (owner: 10Aude) [11:26:28] (03CR) 10Daniel Kinzler: API: Overhaul ApiResult, make format=xml not throw, and add json formatversion (0324 comments) [core] - 10https://gerrit.wikimedia.org/r/182858 (https://phabricator.wikimedia.org/T76728) (owner: 10Anomie) [11:27:22] 10MediaWiki-Search, 7I18n: A Hebrew article title with an apostrophe cannot be found when searching without an apostrophe - https://phabricator.wikimedia.org/T95404#1189229 (10Aklapper) @Amire80: Did you intentionally file this against [[ https://phabricator.wikimedia.org/project/profile/146/ | MediaWiki-Searc... [11:32:00] 10MediaWiki-Search, 7I18n: A Hebrew article title with an apostrophe cannot be found when searching without an apostrophe - https://phabricator.wikimedia.org/T95404#1189252 (10Amire80) >>! In T95404#1189229, @Aklapper wrote: > @Amire80: Did you intentionally file this against [[ https://phabricator.wikimedia.o... [11:32:11] (03CR) 10Aude: API: Overhaul ApiResult, make format=xml not throw, and add json formatversion (033 comments) [core] - 10https://gerrit.wikimedia.org/r/182858 (https://phabricator.wikimedia.org/T76728) (owner: 10Anomie) [11:36:43] (03PS2) 10Matthias Mullie: Stop doing pointless Parsoid roundtrips [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202335 (https://phabricator.wikimedia.org/T90681) [11:36:46] (03PS1) 10Matthias Mullie: Refactor switchEditor [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202706 [11:36:49] (03PS1) 10Matthias Mullie: Remove ext.flow.parsoid.js [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202707 [11:36:56] (03CR) 10jenkins-bot: [V: 04-1] Remove ext.flow.parsoid.js [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202707 (owner: 10Matthias Mullie) [11:37:16] (03Merged) 10jenkins-bot: Update Wikidata - fix issue with diff view [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202702 (owner: 10Aude) [11:39:19] (03PS2) 10Matthias Mullie: Refactor switchEditor [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202706 [11:39:22] (03PS2) 10Matthias Mullie: Remove ext.flow.parsoid.js [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202707 [11:39:25] (03PS3) 10Matthias Mullie: Stop doing pointless Parsoid roundtrips [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202335 (https://phabricator.wikimedia.org/T90681) [11:39:29] 10MediaWiki-extensions-PhpTags: Merge PhpTags Wiki, PhpTags Functions and PhpTags Widgets to PhpTags - https://phabricator.wikimedia.org/T94879#1189258 (10JoelKP) I've thought more about PhpTags Functions, which is the one extension among those you are considering that I didn't mention. PhpTags Functions seems... [11:40:38] 10MediaWiki-extensions-ContentTranslation, 6Collaboration-Team, 10Flow, 10VisualEditor, and 3 others: HTML entities in CommentNode must be manually escaped/unescaped - https://phabricator.wikimedia.org/T95040#1189259 (10Esanders) @cscott you mention that is invalid XML (which is annoyin... [11:44:12] 10Phabricator-Upstream, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1189264 (10Aklapper) Thank you for correcting me and reopening this! Appreciated. I don't think this necessarily... [11:44:31] 10Phabricator-Upstream, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1189265 (10Aklapper) p:5Low>3Normal [11:48:05] (03CR) 10Matthias Mullie: "I'm not sure I would call chaining promises less error prone in all cases (I find it easier to mess up when changing callback return value" (032 comments) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202335 (https://phabricator.wikimedia.org/T90681) (owner: 10Matthias Mullie) [11:48:30] 10MediaWiki-extensions-ContentTranslation: translate all links inside tables - https://phabricator.wikimedia.org/T89951#1189266 (10CennoxX) [11:49:15] 10MediaWiki-Database, 7Schema-change: Some tables lack unique or primary keys, may allow confusing duplicate data - https://phabricator.wikimedia.org/T17441#1189268 (10daniel) [11:49:18] 10MediaWiki-API: API uses non-unique value for paging for some modules - https://phabricator.wikimedia.org/T26782#1189267 (10daniel) [11:49:34] 10MediaWiki-API: Query-continue for imageinfo returns timestamp which is ambiguous if multiple files were uploaded within the same second under the same file name - https://phabricator.wikimedia.org/T67251#1189270 (10daniel) [11:49:35] 10MediaWiki-API: API uses non-unique value for paging for some modules - https://phabricator.wikimedia.org/T26782#285936 (10daniel) [11:51:29] 10Wikidata-Query-Service: Validate how the updater determines where it left off - https://phabricator.wikimedia.org/T95194#1189273 (10daniel) This is a very general and old issue. Perhaps consider re-opening T26782 [11:52:45] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: edit links are displayed on old revisions - https://phabricator.wikimedia.org/T95405#1189275 (10thiemowmde) @WMDE-Fisch, can you try to create a browser test for this? I suggest to leave this ticket open... [11:55:59] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: edit links are displayed on old revisions - https://phabricator.wikimedia.org/T95405#1189277 (10aude) 5Open>3Resolved [11:56:41] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: edit links are displayed on old revisions - https://phabricator.wikimedia.org/T95405#1188988 (10aude) @thiemowmde T95406 [11:57:09] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: edit links are displayed on old revisions - https://phabricator.wikimedia.org/T95405#1189283 (10aude) obviously I would like T95406 in the sprint [11:57:26] 10Living-Style-Guide, 10MediaWiki-skins-Blueprint, 5Patch-For-Review: Stop using bootstrap on the blueprint skin - https://phabricator.wikimedia.org/T91610#1189291 (10Prtksxna) We are going to bring a limited bootstrap back. Details coming soon! [11:58:05] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: browser tests for old revisions of Wikibase item pages to check there are no edit links - https://phabricator.wikimedia.org/T95406#1188999 (10aude) [11:59:24] 10MediaWiki-extensions-ContentTranslation, 6Collaboration-Team, 10Flow, 10VisualEditor, and 3 others: HTML entities in CommentNode must be manually escaped/unescaped - https://phabricator.wikimedia.org/T95040#1189296 (10Esanders) From what I can see single hyphens are valid so I have provided the above fix. [12:01:04] 10MediaWiki-extensions-Graph, 6Services: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1189297 (10mobrovac) 3NEW a:3Yurik [12:01:38] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 6Services: Improve Graphoid code - https://phabricator.wikimedia.org/T93881#1189305 (10mobrovac) [12:01:40] 10MediaWiki-extensions-Graph, 6Services: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1189306 (10mobrovac) [12:02:50] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Browsertests for quantity value input - https://phabricator.wikimedia.org/T88547#1189307 (10WMDE-Fisch) a:3WMDE-Fisch [12:03:17] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1189310 (10mobrovac) Moving the discussion regarding Vagrant to T95399 so that we don't clutter the security review :) [12:06:28] (03PS1) 10Matthias Mullie: Get rid of timeouts in ve init [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202713 [12:07:05] (03Abandoned) 10Matthias Mullie: [WIP] VE plugins [extensions/Flow] - 10https://gerrit.wikimedia.org/r/194346 (https://phabricator.wikimedia.org/T90760) (owner: 10Matthias Mullie) [12:10:01] (03Abandoned) 10Jforrester: Update VisualEditor for Id3ee4dfb for the Collaboration team [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/201588 (owner: 10Jforrester) [12:15:18] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Implement Unit Selector widget - https://phabricator.wikimedia.org/T77982#1189316 (10thiemowmde) * The widget **must** support "custom" values, similar to what currently happens with the globe coordinate precision. If there... [12:18:18] 6Phabricator, 10Wikimania-Hackathon-2015, 10Wikimedia-Hackathon-2015: MediaWiki extension to embed Phabricator content in wiki pages - https://phabricator.wikimedia.org/T90432#1189317 (10scfc) While developing an extension that connects to Phabricator feels like the obvious (and proven) way to solve this, I... [12:19:01] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests: Whitelisting an online Museum Collection for the GWToolset - https://phabricator.wikimedia.org/T95418#1189318 (10PatHadley) 3NEW [12:21:42] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Save button inactive when changing language on monolingual text statements - https://phabricator.wikimedia.org/T95419#1189326 (10WMDE-Fisch) 3NEW [12:26:37] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Thin out scenario outlines on data value browsertest - https://phabricator.wikimedia.org/T95420#1189333 (10WMDE-Fisch) 3NEW [12:26:49] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Thin out scenario outlines on data value browsertest - https://phabricator.wikimedia.org/T95420#1189333 (10WMDE-Fisch) p:5Triage>3Low [12:27:44] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Thin out scenario outlines on data value browsertest - https://phabricator.wikimedia.org/T95420#1189333 (10WMDE-Fisch) [12:27:47] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Tracking: Browsertests for currently used datavalues (tracking) - https://phabricator.wikimedia.org/T55845#1189340 (10WMDE-Fisch) [12:28:04] (03CR) 10Matthias Mullie: "Ebernhardson: actually, that error happens when on the noJS page, but with JS enabled. JS will take over, submitting over API & replacing " [extensions/Flow] - 10https://gerrit.wikimedia.org/r/197027 (https://phabricator.wikimedia.org/T95362) (owner: 10Matthias Mullie) [12:29:28] 7database: Database error when trying to connect to NL wikipedia - https://phabricator.wikimedia.org/T95421#1189342 (10Mbch331) 3NEW [12:30:06] (03PS4) 10Matthias Mullie: Remove preview related code [extensions/Flow] - 10https://gerrit.wikimedia.org/r/199541 (owner: 10EBernhardson) [12:33:33] (03CR) 10Matthias Mullie: [C: 031] Remove preview related code [extensions/Flow] - 10https://gerrit.wikimedia.org/r/199541 (owner: 10EBernhardson) [12:35:10] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Browsertests for quantity value input - https://phabricator.wikimedia.org/T88547#1189353 (10WMDE-Fisch) https://github.com/wmde/WikidataBrowserTests/pull/66 [12:35:31] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: Browsertests for quantity value input - https://phabricator.wikimedia.org/T88547#1189354 (10WMDE-Fisch) [12:36:15] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Browsertests for monolingual text value input - https://phabricator.wikimedia.org/T88546#1189365 (10WMDE-Fisch) a:3WMDE-Fisch [12:38:09] 7database: Database error when trying to connect to NL wikipedia - https://phabricator.wikimedia.org/T95421#1189372 (10Mbch331) 5Open>3Resolved a:3Mbch331 It's working again. [12:42:59] (03CR) 10Hashar: "recheck" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/199541 (owner: 10EBernhardson) [12:48:51] (03PS2) 10Prtksxna: add bootstrap [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202698 [12:48:53] (03PS1) 10Prtksxna: composer update [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202717 [12:48:55] (03PS1) 10Prtksxna: fix indentation [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202718 [12:48:57] (03PS1) 10Prtksxna: remove styles that would override bootstrap [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202719 [12:48:59] (03PS1) 10Prtksxna: update the navbar [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202720 [12:49:01] (03PS1) 10Prtksxna: move toc icon [skins/Blueprint] - 10https://gerrit.wikimedia.org/r/202721 [12:57:05] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Implement Unit Selector widget - https://phabricator.wikimedia.org/T77982#1189408 (10daniel) >>! In T77982#1189316, @thiemowmde wrote: > * The widget **must** support "custom" values, similar to what currently happens with t... [13:04:26] 10Wikimedia-Site-requests: Please add http://www.padil.gov.au/ to the wgCopyUploadsDomains whitelist of Wikimedia Commons - https://phabricator.wikimedia.org/T95328#1189411 (10Dereckson) a:3Dereckson [13:05:56] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Wikidata-Sprint-2015-04-07: Implement Unit Selector widget - https://phabricator.wikimedia.org/T77982#1189412 (10Snipre) @thiemowmde. The list will be very long: we have to think a way to avoid scrolling dozen of units to find the correct one. One poss... [13:09:47] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1189413 (10Tobi_WMDE_SW) p:5Triage>3High [13:11:00] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: browser tests for old revisions of Wikibase item pages to check there are no edit links - https://phabricator.wikimedia.org/T95406#1189417 (10Tobi_WMDE_SW) [13:11:03] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1189007 (10Tobi_WMDE_SW) [13:13:31] 10Wikimedia-Hackathon-2015: Transportation during Hackathon - https://phabricator.wikimedia.org/T94543#1189427 (10AlexCella) We'll give many explainations regarding how to come to Valpré from many places. I think we'll even film how to get there from the main train station with a GoPro. :) Uber looks to us the... [13:19:50] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1189439 (10mobrovac) @csteipp Need more info/help for doing the review? [13:22:25] 10MediaWiki-API, 6Multimedia: Provide an API to rotate an image - https://phabricator.wikimedia.org/T35186#1189445 (10Anomie) [13:24:59] 10MediaWiki-API: Insufficient param validation - https://phabricator.wikimedia.org/T41830#1189447 (10Anomie) [13:25:31] 10MediaWiki-Database: SiteStatsInit::edits and SiteStatsInit::articles query storm - https://phabricator.wikimedia.org/T95426#1189449 (10Springle) 3NEW [13:26:22] 10MediaWiki-API: Insufficient param validation - https://phabricator.wikimedia.org/T41830#440148 (10Anomie) >>! In T41830#440226, @Aklapper wrote: > Still there is a patch in gerrit that somebody could pick up, hence restoring patch-in-gerrit, keyword. While there was a patch, it's not something someone could... [13:29:13] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests: Whitelisting ymt.adlibhosting.com for the GWToolset - https://phabricator.wikimedia.org/T95418#1189462 (10Aklapper) [13:36:51] 7database: Database error when trying to connect to NL wikipedia - https://phabricator.wikimedia.org/T95421#1189464 (10Aklapper) [13:36:53] 10MediaWiki-Database: SiteStatsInit::edits and SiteStatsInit::articles query storm - https://phabricator.wikimedia.org/T95426#1189465 (10Aklapper) [13:37:47] 7database: Database error when trying to connect to NL wikipedia - https://phabricator.wikimedia.org/T95421#1189342 (10Aklapper) Thanks a lot for reporting this! Indeed there was an outage of s2 wikis. T95426 has some more technical information so I have closed this task as a duplicate of T95426. T95426 will g... [13:39:25] (03CR) 10Anomie: [C: 032] Removed unused variable $time from OAuthListConsumers [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/202678 (owner: 1001tonythomas) [13:40:36] (03CR) 10Matthias Mullie: [C: 04-1] Allow moving flow boards (034 comments) [extensions/Flow] - 10https://gerrit.wikimedia.org/r/201596 (https://phabricator.wikimedia.org/T90063) (owner: 10EBernhardson) [13:40:47] (03Merged) 10jenkins-bot: Removed unused variable $time from OAuthListConsumers [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/202678 (owner: 1001tonythomas) [13:41:06] (03CR) 10Anomie: [C: 031] "Seems sane. Haven't tested." [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/185611 (https://phabricator.wikimedia.org/T68832) (owner: 1001tonythomas) [13:45:10] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests: Whitelisting ymt.adlibhosting.com for the GWToolset - https://phabricator.wikimedia.org/T95418#1189473 (10Dereckson) [13:45:28] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests: Whitelisting ymt.adlibhosting.com for the GWToolset - https://phabricator.wikimedia.org/T95418#1189474 (10Dereckson) a:3Dereckson [13:53:45] 10MediaWiki-Database: SiteStatsInit::edits and SiteStatsInit::articles query storm - https://phabricator.wikimedia.org/T95426#1189475 (10Aklapper) p:5Triage>3High [13:54:08] (03PS1) 10MaxSem: Use PoolCounter for SiteStats updates [core] - 10https://gerrit.wikimedia.org/r/202732 (https://phabricator.wikimedia.org/T95426) [13:55:39] hoo: time to quick review ( and maybe +2 ) https://gerrit.wikimedia.org/r/#/c/185611/ ? [13:56:07] (03PS4) 10Amire80: Removed obsolete HTML tags and replaced with CSS [extensions/FlaggedRevs] - 10https://gerrit.wikimedia.org/r/196478 (owner: 10Kyle0692) [13:56:20] tonythomas: In a short bit, yes [13:56:37] \o/ [13:58:31] (03PS9) 10Physikerwelt: Render math tags in search results [extensions/Math] - 10https://gerrit.wikimedia.org/r/197737 (https://phabricator.wikimedia.org/T93075) [14:00:56] tonythomas: Did you test it? [14:02:28] hoo: yup - I could check the 'parentheses' part neatly - I remember checking the 'brackets' part too - let me test again though :) [14:02:41] Because I doubt it works on PHP5.3 [14:03:50] manybubbles: MaxSem heads up,i just emailed about doing the merge on https://gerrit.wikimedia.org/r/#/c/199178/ and https://gerrit.wikimedia.org/r/#/c/199179/ this morning. i also asked haha if he's on the operations/mediawiki-config part of this, not sure if you guys have a preference for which files in that repo he should use to enforce the global [14:04:21] hoo: yeah - it works for me here http://pasteboard.co/2pG8HcHN.jpg [14:04:37] Which PHP version do you run? [14:04:46] let me check [14:04:58] 5.4.36 [14:05:29] (03PS10) 10Physikerwelt: Render math tags in search results [extensions/Math] - 10https://gerrit.wikimedia.org/r/197737 (https://phabricator.wikimedia.org/T93075) [14:05:40] (03CR) 10Hoo man: [C: 04-1] Removed hard coded punctuations from [[Special:oauthlistconsumers]] (031 comment) [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/185611 (https://phabricator.wikimedia.org/T68832) (owner: 1001tonythomas) [14:05:50] PHP 5.3 is "fun" [14:07:24] ha - I thought I was running the 'oldest' version :P let me check for a work-around [14:07:45] 10MediaWiki-extensions-ContentTranslation, 6Language-Engineering, 5ContentTranslation-Release5: Improve language selection when creating a translation - https://phabricator.wikimedia.org/T76196#1189489 (10Amire80) 5Open>3Resolved [14:08:06] (03CR) 10Physikerwelt: "I implemented all proposed changes." [extensions/Math] - 10https://gerrit.wikimedia.org/r/197737 (https://phabricator.wikimedia.org/T93075) (owner: 10Physikerwelt) [14:11:25] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2: Placeholder: Q4 plan investigation - https://phabricator.wikimedia.org/T94447#1189497 (10phuedx) The "site speed report card" epic and subtasks are here: T94752 [14:13:16] (03PS6) 1001tonythomas: Removed hard coded punctuations from [[Special:oauthlistconsumers]] [extensions/OAuth] - 10https://gerrit.wikimedia.org/r/185611 (https://phabricator.wikimedia.org/T68832) [14:14:42] (03PS1) 10Tinaj1234: Trigger Echo notification when registered email address becomes unverified [extensions/BounceHandler] - 10https://gerrit.wikimedia.org/r/202735 (https://phabricator.wikimedia.org/T94436) [14:16:07] 10MediaWiki-extensions-WikibaseClient, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Tracking: language fallback - https://phabricator.wikimedia.org/T76216#1189510 (10Tobi_WMDE_SW) [14:16:09] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 3§ Wikidata-Sprint-2015-03-24, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: Add language fallback for badge labels - https://phabricator.wikimedia.org/T94552#1189508 (10Tobi_WMDE_SW) 5Open>3Resolved [14:20:51] 10MediaWiki-extensions-Newsletter, 6Engineering-Community, 10Possible-Tech-Projects, 3Google-Summer-of-Code-2015, 3Outreachy-Round-10: GSoC'15 Proposal for Newsletter Extension Project - https://phabricator.wikimedia.org/T93583#1189518 (10Tinaj1234) [14:24:53] 10Wikimedia-Site-requests: Liangent's user page and user subpages on zhwiki - https://phabricator.wikimedia.org/T95408#1189527 (10Legoktm) I suppose we need a javascript confirm button on the rename queue like Special:GlobalRenameUser has? [14:24:57] 6MediaWiki-API-Team, 10MediaWiki-extensions-OAuth: OAuth: Authorisation should not fail because you don't have an account on central wiki - https://phabricator.wikimedia.org/T74469#1189528 (10Tgr) [14:25:00] 10SUL-Finalization, 7Epic, 3Roadmap, 7Tracking, and 2 others: Complete unification of all accounts to SUL - https://phabricator.wikimedia.org/T37707#1189529 (10Tgr) [14:25:05] (03CR) 1001tonythomas: [C: 04-1] "Maybe add which email address too got un-subscribed in the flyout message ? @Lego, @hoo: Do you think it is necessary ?" (036 comments) [extensions/BounceHandler] - 10https://gerrit.wikimedia.org/r/202735 (https://phabricator.wikimedia.org/T94436) (owner: 10Tinaj1234) [14:25:57] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Link item error details in my repo language instead of client language - https://phabricator.wikimedia.org/T95427#1189535 (10aude) 3NEW [14:29:37] ha ! https://www.mediawiki.org/wiki/Echo_(Notifications)/Developer_guide#Hook_the_notification_into_the_Echo_extension miss adding the hooks declaration to extension.php file :\ [14:29:48] * tonythomas is adding that now [14:31:19] 10MediaWiki-extensions-WikibaseClient, 10Wikidata: Link item error details in my repo language instead of client language - https://phabricator.wikimedia.org/T95427#1189553 (10aude) now the error is correctly displayed in english: {F110057} this might be a non-bug :) and was only caused by another bug (Wikib... [14:33:50] 10Wikimedia-Site-requests: Lift IP rate limit - Santiago Editatón - 2015-04-11 and 2015-04-12 - https://phabricator.wikimedia.org/T95302#1189563 (10Dereckson) a:3Dereckson [14:36:22] hoo: are you planning on fixing https://phabricator.wikimedia.org/T95408 since it's assigned to you? [14:36:59] legoktm: Only if no one else comes around to it [14:37:03] 10MediaWiki-extensions-CentralAuth: Add javascript confirmation button to Special:GlobalRenameQueue - https://phabricator.wikimedia.org/T95428#1189564 (10Legoktm) 3NEW [14:38:21] (03PS1) 10Legoktm: Log promote to global renames in the global rename log [extensions/CentralAuth] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202739 (https://phabricator.wikimedia.org/T93235) [14:41:04] (03CR) 10MaxSem: [C: 032] Install non-vendor services to /vagrant/srv [vagrant] - 10https://gerrit.wikimedia.org/r/200624 (https://phabricator.wikimedia.org/T89919) (owner: 10Dduvall) [14:41:36] (03Merged) 10jenkins-bot: Install non-vendor services to /vagrant/srv [vagrant] - 10https://gerrit.wikimedia.org/r/200624 (https://phabricator.wikimedia.org/T89919) (owner: 10Dduvall) [14:41:42] (03Abandoned) 10MaxSem: Create the arcanist dir, otherwise it fails [vagrant] - 10https://gerrit.wikimedia.org/r/202246 (owner: 10MaxSem) [14:42:49] 10Wikimedia-Site-requests, 5Patch-For-Review: Lift IP rate limit - Santiago Editatón - 2015-04-11 and 2015-04-12 - https://phabricator.wikimedia.org/T95302#1189583 (103BRBS) Thanks Dereckson! [14:42:49] (03CR) 10Legoktm: [C: 032] Log promote to global renames in the global rename log [extensions/CentralAuth] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202739 (https://phabricator.wikimedia.org/T93235) (owner: 10Legoktm) [14:44:50] (03Merged) 10jenkins-bot: Log promote to global renames in the global rename log [extensions/CentralAuth] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202739 (https://phabricator.wikimedia.org/T93235) (owner: 10Legoktm) [14:45:35] (03PS1) 10Legoktm: Update CentralAuth to 1.25wmf24 HEAD [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202742 [14:56:01] (03CR) 10Legoktm: [C: 032] Add test for IRC line for type protect/move_prot [core] - 10https://gerrit.wikimedia.org/r/202699 (owner: 10Umherirrender) [15:05:07] (03Merged) 10jenkins-bot: Add test for IRC line for type protect/move_prot [core] - 10https://gerrit.wikimedia.org/r/202699 (owner: 10Umherirrender) [15:09:13] 10Wikimedia-Site-requests, 5Patch-For-Review: Lift IP rate limit - Santiago Editatón - 2015-04-11 and 2015-04-12 - https://phabricator.wikimedia.org/T95302#1189637 (10Dereckson) 5Open>3Resolved Deployed during Morning SWAT. [15:10:18] (03PS2) 10Anomie: Report correct rev_id in missing-revision message [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202602 (https://phabricator.wikimedia.org/T92046) [15:10:26] (03CR) 10Anomie: [C: 032] "SWAT" [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202602 (https://phabricator.wikimedia.org/T92046) (owner: 10Anomie) [15:10:38] (03PS2) 10Anomie: AdHocDebug: Get stack traces for failures in Article::fetchContentObject [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202603 (https://phabricator.wikimedia.org/T92046) [15:10:45] (03CR) 10Anomie: [C: 032] "SWAT" [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202603 (https://phabricator.wikimedia.org/T92046) (owner: 10Anomie) [15:11:48] 6MediaWiki-Core-Team, 10MediaWiki-Debug-Logging, 6Release-Engineering, 10Wikimedia-Logstash, and 2 others: Log php fatals with full backtraces again (fatal.log on fluorine) - https://phabricator.wikimedia.org/T89169#1189640 (10Legoktm) Progress! We now have logs that look like: ``` 2015-04-08 15:08:56 mw12... [15:14:31] 10MediaWiki-extensions-Renameuser: Userpage revisions destroyed after global rename / strange pagemoves - https://phabricator.wikimedia.org/T95029#1189643 (10Ajraddatz) Yes, thanks Krenair. The global renameuser. [15:17:45] 7Design, 6Collaboration-Team, 10Flow: Editing toolbar(s) for Flow (VE and/or wikitext toolbar) - https://phabricator.wikimedia.org/T78346#1189648 (10Jdforrester-WMF) [15:20:44] 10Phabricator-Upstream, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1189655 (10chasemp) I'm confused as to why we are adding things here? AFAIK legal basically [[ https://phabrica... [15:22:27] 10WMF-Product-Strategy: Perform a Wikidata gap analysis - https://phabricator.wikimedia.org/T91423#1189665 (10Lydia_Pintscher) @Ironholds: ou saw https://www.wikidata.org/wiki/Wikidata:Data_access I assume? [15:24:16] (03Merged) 10jenkins-bot: Report correct rev_id in missing-revision message [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202602 (https://phabricator.wikimedia.org/T92046) (owner: 10Anomie) [15:24:20] (03Merged) 10jenkins-bot: AdHocDebug: Get stack traces for failures in Article::fetchContentObject [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202603 (https://phabricator.wikimedia.org/T92046) (owner: 10Anomie) [15:24:42] 10WMF-Product-Strategy: Perform a Wikidata gap analysis - https://phabricator.wikimedia.org/T91423#1189687 (10Ironholds) Yep; see "not minified": streaming it line-by-line after it's on disc breaks since it seems to be formatted as one big JSON blob. I've worked out a way around this (and probably didn't need th... [15:25:55] (03CR) 10Legoktm: [C: 032] Update CentralAuth to 1.25wmf24 HEAD [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202742 (owner: 10Legoktm) [15:28:14] 10Wikimedia-General-or-Unknown: English Wikipedia user talk page shown as blue link but displays: revision #0 does not exist - https://phabricator.wikimedia.org/T92046#1189718 (10Anomie) [15:28:56] 10Wikimedia-General-or-Unknown: English Wikipedia user talk page shown as blue link but displays: revision #0 does not exist - https://phabricator.wikimedia.org/T92046#1102261 (10Anomie) Ok, we've merged some more debugging code to wmf24. If you see an instance of the problem on a non-Wikipedia for an edit after... [15:29:09] 7service-runner: Decide whether to install heapdump by default, or install npm & install on demand - https://phabricator.wikimedia.org/T95431#1189722 (10GWicke) 3NEW [15:33:37] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Need-volunteer: Store the badges of sitelinks in a queryable format - https://phabricator.wikimedia.org/T72229#1189746 (10Bene) >>! In T72229#1143170, @Bene wrote: >>>! In T72229#1109613, @aude wrote: >> I don't think there is good solution to this othe... [15:34:24] 10Wikimedia-Site-requests: Add namespace aliases in Latvian Wikipedia - https://phabricator.wikimedia.org/T95106#1189747 (10Dereckson) a:3Dereckson [15:34:48] (03Merged) 10jenkins-bot: Update CentralAuth to 1.25wmf24 HEAD [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202742 (owner: 10Legoktm) [15:37:06] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Tracking: Determine list of available languages in a uniform way - https://phabricator.wikimedia.org/T78006#1189768 (10jeblad) [15:42:00] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1189783 (10WMDE-Fisch) a:3WMDE-Fisch [15:44:36] (03PS1) 10devunt: Fix some codes to fit the style guide [extensions/Josa] - 10https://gerrit.wikimedia.org/r/202754 [15:46:38] 10Wikimedia-Site-requests, 5Patch-For-Review: Add namespace aliases in Latvian Wikipedia - https://phabricator.wikimedia.org/T95106#1189852 (10Dereckson) p:5Triage>3Normal [15:47:18] 6WMF-Design-Research, 10MobileFrontend-Feature-requests, 10VisualEditor, 10VisualEditor-MediaWiki-Mobile, and 2 others: Provide VisualEditor regardless of device size - https://phabricator.wikimedia.org/T85630#1189855 (10Jdforrester-WMF) [15:47:34] 6WMF-Design, 10VisualEditor, 10VisualEditor-MediaWiki-Mobile, 3Editing Department 2014/15 Q4 blockers: Provide design guidance for minimal VisualEditor header on phones - https://phabricator.wikimedia.org/T93325#1189858 (10Jdforrester-WMF) [15:48:53] (03CR) 10Dereckson: "I abandon this one." [core] - 10https://gerrit.wikimedia.org/r/198208 (owner: 10Dereckson) [15:49:35] (03Abandoned) 10Dereckson: Add .arcconfig to .gitignore [core] - 10https://gerrit.wikimedia.org/r/198208 (owner: 10Dereckson) [15:51:53] 10Phabricator-Upstream, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1189876 (10Josve05a) IANL, but AFAIK per the quoted paragraph in the above mentioned legal text for the licese (... [15:53:24] 6WMF-NDA-Requests: Allow Valhallasw to sign an NDA - https://phabricator.wikimedia.org/T93644#1189880 (10chasemp) >>! In T93644#1142903, @valhallasw wrote: > That's exactly what this task is, right? I'm not sure what the 'specific permissions' are, but I guess "Access to private information on Tool Labs", and th... [15:53:45] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: I cannot edit or add statements - https://phabricator.wikimedia.org/T77904#1189882 (10Bene) 5stalled>3Invalid a:3Bene [[ https://www.wikidata.org/wiki/Special:Contributions/GerardM | Apperently ]] this user can edit again. If the bug appears again fe... [15:55:36] 10Phabricator-Upstream, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1189889 (10chasemp) >>! In T94946#1189876, @Josve05a wrote: > IANL, but AFAIK per the quoted paragraph in the ab... [15:55:52] (03PS4) 10Niharika29: [WIP] Add a report by reviewer for a campaign [wikimedia/iegreview] - 10https://gerrit.wikimedia.org/r/202090 (https://phabricator.wikimedia.org/T87099) [15:56:19] (03PS10) 10Dr0ptp4kt: Track search events [core] - 10https://gerrit.wikimedia.org/r/199178 (owner: 10Bmansurov) [15:56:26] (03PS16) 10Dr0ptp4kt: Add support for event logging [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/199179 (owner: 10Bmansurov) [15:56:39] 10Wikimedia-Site-requests: Liangent's user page and user subpages on zhwiki - https://phabricator.wikimedia.org/T95408#1189896 (10Trijnstel) >>! In T95408#1189527, @Legoktm wrote: > I suppose we need a javascript confirm button on the rename queue like Special:GlobalRenameUser has? Good suggestion! [15:57:26] (03CR) 10Dr0ptp4kt: [C: 032] Add support for event logging [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/199179 (owner: 10Bmansurov) [15:57:35] 6WMF-Design-Research, 10MobileFrontend-Feature-requests, 10VisualEditor, 10VisualEditor-MediaWiki-Mobile, and 2 others: Provide VisualEditor regardless of device size - https://phabricator.wikimedia.org/T85630#1189897 (10Jdforrester-WMF) p:5Triage>3Normal [15:57:38] (03CR) 10Dr0ptp4kt: [V: 032] Add support for event logging [extensions/CirrusSearch] - 10https://gerrit.wikimedia.org/r/199179 (owner: 10Bmansurov) [15:57:51] (03CR) 10Dr0ptp4kt: [C: 032] Track search events [core] - 10https://gerrit.wikimedia.org/r/199178 (owner: 10Bmansurov) [15:58:02] (03CR) 10Dr0ptp4kt: [V: 032] Track search events [core] - 10https://gerrit.wikimedia.org/r/199178 (owner: 10Bmansurov) [15:58:59] 10Wikimedia-Site-requests: add Reports and Press-releases namespace to chapter wiki ru.wikimedia.org - https://phabricator.wikimedia.org/T95110#1189901 (10Dereckson) a:3Dereckson [16:00:22] 10Phabricator-Upstream, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1189904 (10Josve05a) >>! In T94946#1189889, @chasemp wrote: > T But all of us are IANAL. Or at least IANYL ;).... [16:00:41] (03CR) 10BryanDavis: [WIP] Add a report by reviewer for a campaign (031 comment) [wikimedia/iegreview] - 10https://gerrit.wikimedia.org/r/202090 (https://phabricator.wikimedia.org/T87099) (owner: 10Niharika29) [16:04:41] hiiii anomie :) [16:04:59] ottomata: Hi [16:05:16] was about to post again in that ticket, but then I figured it'd be easier to talk with ya here, if you got a sec [16:06:16] ottomata: Which ticket? [16:06:23] https://phabricator.wikimedia.org/T92875 [16:06:28] x analytics in api with page_id thing [16:06:40] i'm looking at code now and am sloowwwly understanding :) [16:07:30] does MobileFrontend have access to hooks defined in WikimediaEvents extension? [16:07:34] or, is that the issue? [16:07:43] (there is a lot I do not know about Mediawiki) [16:08:19] I see a onBeforePageDisplay hook defined for MobileFrontend [16:08:24] that is also touching the x analytics header [16:08:58] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Tracking: Determine list of available languages in a uniform way - https://phabricator.wikimedia.org/T78006#1189946 (10Snipre) Just find this list for variation of metalanguage: http://www.iana.org/assignments/language-subtag-registry/language-subtag-re... [16:09:32] $wgHooks['BeforePageDisplay'][] = 'MobileFrontendHooks::onBeforePageDisplay'; [16:10:03] so, i *think* i should do two things [16:10:27] ottomata: MobileFrontend is setting the header based on criteria that I don't know anything about. The XAnalytics extension hooks into MediaWiki just before MediaWiki starts producing output, grabs the current value of the header, calls the XAnalyticsSetHeader hook, and then re-sets the header. The problem is that "just before MediaWiki starts producing output" is using the BeforePageDisplay hook, but that hook isn't used by the API. [16:10:27] "APIAfterExecute" is probably the most-equivalent hook for XAnalytics to use as "just before MediaWiki starts producing output" for API requests. [16:10:46] 10MediaWiki-Interface: A simple XML "skin" to be used with XSLT - https://phabricator.wikimedia.org/T19829#1189954 (10Mvolz) [16:10:48] 10MediaWiki-General-or-Unknown: Options for Skins - https://phabricator.wikimedia.org/T20365#1189955 (10Mvolz) [16:10:50] 10MediaWiki-General-or-Unknown: Merge all skins' PHP files - https://phabricator.wikimedia.org/T8884#1189952 (10Mvolz) 5declined>3Resolved [16:10:54] 10MediaWiki-extensions-ContentTranslation, 10ContentTranslation-cxserver, 3LE-Sprint-85, 5Patch-For-Review: cxserver fails to load pages with question marks in URL - https://phabricator.wikimedia.org/T94667#1189956 (10santhosh) 5Open>3Resolved [16:11:29] ah, so, should I make the X-analytics extension function also be added APIAfterExecute hooks? [16:11:40] $wgHooks['APIAfterExecute'][] = function [16:11:42] ... [16:11:42] ? [16:11:49] in mediawiki-extensions-XAnalytics? [16:12:08] ottomata: Exactly. But you'll run into a problem with trying to call the same XAnalyticsSetHeader hook functions, because you won't have an OutputPage object to pass. [16:12:17] note that currently, MobileFrontend is mangling x-analytics in BeforePageDisplay [16:12:26] hm [16:13:04] So MobileFrontend isn't doing its mangling for API requests either. *shrug* That's another thing to fix, if you want. [16:13:08] https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/includes/MobileFrontend.hooks.php#L685 [16:13:18] oh. [16:13:19] ? [16:13:27] confused now [16:13:36] ottomata: For your specific issue in T92875, you may run into another problem: some API requests are actually acting on up to 5000 different titles. [16:14:05] what requests would those be? [16:14:28] i don't need every request tagged by this, we just want ones that would be the main page content request for a page display [16:14:48] if someone is bulk requesting articles, then that doesn't count [16:14:49] 10MediaWiki-Extension-Requests, 10Possible-Tech-Projects, 6Wiktionary: Extension to provide access via the dict protocol - https://phabricator.wikimedia.org/T31229#1190033 (10Lydia_Pintscher) Sorry. Should have been more verbose. I agree that we currently don't plan to provide a dict server. And that it woul... [16:14:51] as a page view request [16:15:08] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: With uselang=ar, redirected to special page instead of edit mode for labels etc. - https://phabricator.wikimedia.org/T94686#1190034 (10aude) also seems fixed for me :) [16:15:13] 10MediaWiki-General-or-Unknown: Options for Skins - https://phabricator.wikimedia.org/T20365#1190037 (10matmarex) [16:15:15] 10MediaWiki-Interface: A simple XML "skin" to be used with XSLT - https://phabricator.wikimedia.org/T19829#1190038 (10matmarex) [16:15:16] ottomata: action=query, action=purge, action=setnotificationtimestamp, action=watch. Possibly others in extensions, you can tell by whether they have "titles", "pageids", and "revids" parameters. [16:15:17] 10MediaWiki-General-or-Unknown: Merge all skins' PHP files - https://phabricator.wikimedia.org/T8884#1190036 (10matmarex) 5Resolved>3declined [16:15:23] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 7Need-volunteer: Store the badges of sitelinks in a queryable format - https://phabricator.wikimedia.org/T72229#1190040 (10Superyetkin) What else needs to be done to have this feature implemented? [16:15:26] ottomata: Or "generator" [16:17:56] but some might not have "generator" while having the other three. [16:17:57] ottomata: Even if it's something that operates on only one page, I don't think there's any general mechanism to identify that page. [16:17:59] 10MediaWiki-General-or-Unknown: Merge all skins' PHP files - https://phabricator.wikimedia.org/T8884#113125 (10matmarex) [16:18:06] 7Design, 10Wikimania-Hackathon-2015, 10Wikimedia-Hackathon-2015, 10Wikipedia-Android-App: Add an easier and faster way to change the search language - https://phabricator.wikimedia.org/T87154#1190045 (10Mdann52) [16:24:16] anomie: what is the onBeforePageDisplay function doing? [16:24:17] ottomata: In which extension? [16:24:17] is that not called part of an API request? [16:24:17] MobileFrontend [16:24:19] i think I am still confused [16:24:19] is API not in MobileFrontend? [16:24:21] ottomata: BeforePageDisplay is (generally) not called for API requests. With respect to the X-Analytics header, MobileFrontend's onBeforePageDisplay is basically doing what the XAnalytics extension does. [16:24:21] 10MediaWiki-Extension-Requests, 10Possible-Tech-Projects, 6Wiktionary: Extension to provide access via the dict protocol - https://phabricator.wikimedia.org/T31229#1190056 (10Ricordisamoa) >>! In T31229#1190033, @Lydia_Pintscher wrote: > Sorry. Should have been more verbose. I agree that we currently don't p... [16:24:21] 6Scrum-of-Scrums, 10Wikimedia-Search: Scope a spike around implementing https://meta.wikimedia.org/wiki/Schema:Search for Desktop - https://phabricator.wikimedia.org/T90518#1190057 (10dr0ptp4kt) I merged the JS patches. I've asked @bmansurov about the operations/mediawiki-config global change. [16:24:21] ottomata: MobileFrontend implements an API module, but "the API" isn't in MobileFrontend. [16:24:21] aye [16:24:21] ok [16:24:22] hm [16:24:22] ok, anomie, in ApiParseExtender.php, I'm looking at the onAPIAfterExecute function [16:24:22] i see there is a $data object [16:24:22] i don't know what that is [16:24:22] but, it does seem to have page info in it [16:24:22] ottomata: ApiParseExtender is rather evil, BTW. [16:24:22] $title = Title::newFromText( $data['parse']['title'] ); [16:24:22] (03PS1) 10Aaron Schulz: Use wfWaitForSlaves in upload stash cleanup script [core] - 10https://gerrit.wikimedia.org/r/202763 [16:24:22] haha, well, that is the hook :p [16:24:22] Had I code-reviewed that, I'd probably have -1ed with a suggestion to do something more sane. [16:24:31] (03PS2) 10Aaron Schulz: Use wfWaitForSlaves in upload stash cleanup script [core] - 10https://gerrit.wikimedia.org/r/202763 (https://phabricator.wikimedia.org/T95382) [16:25:57] bwwuhhhh, where is ApiBase defined? [16:25:57] not in MobileFrontend [16:25:57] ottomata: ApiParseExtender is taking the result data after the core ApiParse module runs, munging it, and replacing it. [16:25:57] ottomata: core, includes/api/ApiBase.php [16:25:57] HM. [16:25:58] As I said, "the API" isn't in MobileFrontend. [16:26:01] (03PS1) 10Legoktm: Add javascript confirmation to Special:GlobalRenameQueue [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/202764 (https://phabricator.wikimedia.org/T95428) [16:31:34] ok, but action=mobileview is responded to by ApiMobileView [16:31:34] ja? [16:31:35] ottomata: Correct. That's configured by https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/MobileFrontend.php#L109, BTW [16:31:36] aye, ok [16:31:36] so, hm. [16:31:36] anomie: this seems like a bad idea, but technically could I just set the header in the execute() method there? [16:31:38] ottomata: If you want action=mobileview to set the header, then yes you could just set the header in the execute() method there. But XAnalyticsSetHeader hook functions still won't be called, unless you also make the changes to that extension to use APIAfterExecute. [16:31:38] ok, starting to understand [16:31:41] 10Wikimedia-Site-requests: Liangent's user page and user subpages on zhwiki - https://phabricator.wikimedia.org/T95408#1190066 (10Legoktm) >>! In T95408#1189896, @Trijnstel wrote: >>>! In T95408#1189527, @Legoktm wrote: >> I suppose we need a javascript confirm button on the rename queue like Special:GlobalRenam... [16:31:43] the extension should be able to hook into API related stuff too via APIAfterExecute [16:31:44] 6MediaWiki-API-Team, 10MediaWiki-extensions-CentralAuth, 5Patch-For-Review: Add javascript confirmation button to Special:GlobalRenameQueue - https://phabricator.wikimedia.org/T95428#1190070 (10Legoktm) a:3Legoktm [16:31:44] ok. [16:32:01] but. hm. APIAfterExecute does not have a Page type object I can get info from, right? [16:32:07] unless [16:32:09] it is in $data = $module->getResultData(); [16:32:27] guess I need to look at what ApiBase -getResultData is... [16:32:41] 10MediaWiki-extensions-SandboxLink, 10Wikimedia-Extension-setup: Write a script to disable local "My sandbox" gadgets, to be run simultaneously with the SandboxLink extension being deployed - https://phabricator.wikimedia.org/T92771#1190096 (10matmarex) p:5Normal>3High [16:32:43] 10MediaWiki-extensions-SyntaxHighlight-(GeSHi): Combine infrequently-used language modules into a single module - https://phabricator.wikimedia.org/T93025#1190099 (10matmarex) p:5Triage>3Lowest [16:32:48] 10MediaWiki-extensions-SyntaxHighlight-(GeSHi): Combine infrequently-used language modules into a single module - https://phabricator.wikimedia.org/T93025#1126893 (10matmarex) p:5Lowest>3Low [16:33:07] 6Phabricator, 5Patch-For-Review: MediaWiki login for Phabricator produces broken links for Wikimedia profiles with URL encoded characters - https://phabricator.wikimedia.org/T547#1190240 (10mmodell) [16:33:10] 6MediaWiki-API-Team, 10Librarization, 7Documentation, 5MW-1.25-release: Update [[mw:Composer]] to include information about usage with libraries - https://phabricator.wikimedia.org/T85172#1190242 (10Legoktm) [16:33:15] 10MediaWiki-extensions-SandboxLink, 10Wikimedia-Extension-setup, 5Patch-For-Review, 3Roadmap, and 2 others: Deploy SandboxLink extension to cluster and enable on wikis where it currently is a gadget - https://phabricator.wikimedia.org/T72499#1190245 (10Legoktm) >>! In T72499#1176738, @Eloquence wrote: > @L... [16:36:16] (03PS1) 10Aaron Schulz: Prevent SiteStats::loadAndLazyInit from reloading in $wgMiserMode [core] - 10https://gerrit.wikimedia.org/r/202766 (https://phabricator.wikimedia.org/T95426) [16:36:56] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1190355 (10WMDE-Fisch) https://github.com/wmde/WikidataBrowserTests/pull/67 [16:37:11] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 5Patch-For-Review: Link item error details in my repo language instead of client language - https://phabricator.wikimedia.org/T95427#1190360 (10aude) i can't reproduce this now, even in incognito mode. if it happens again, i'll reopen and investigate more. [16:37:14] 10MediaWiki-extensions-WikibaseClient, 10Wikidata, 5Patch-For-Review: Link item error details in my repo language instead of client language - https://phabricator.wikimedia.org/T95427#1190362 (10aude) 5Open>3Invalid [16:37:17] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1190368 (10WMDE-Fisch) [16:37:36] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review, 10Wikidata-Sprint-2015-04-07: create browser tests to check editing and display of terms in EntityTermView - https://phabricator.wikimedia.org/T95407#1189007 (10WMDE-Fisch) [16:42:41] ottomata: $module->getResultData() is an array, often with many subarrays, containing basically the PHP representation of what will eventually be json-encoded or php-serialized or XML-encoded, or whatever and sent to the client. Gerrit change 182858 will make that slightly less true, but still mostly accurate. [16:42:59] 10MediaWiki-extensions-Graph, 10MediaWiki-extensions-WikiHiero, 10Citoid, 10Mathoid-General-or-Unknown, and 3 others: Services need to share the same domain name - https://phabricator.wikimedia.org/T95402#1190382 (10mobrovac) RESTBase can easily be set up to proxy requests to back-end services without chan... [16:43:30] 10MediaWiki-Database, 5Patch-For-Review: SiteStatsInit::edits and SiteStatsInit::articles query storm - https://phabricator.wikimedia.org/T95426#1190386 (10aaron) Sample backtrace: #0 /srv/mediawiki/php-1.25wmf23/includes/db/Database.php(1189): DatabaseBase->reportQueryError() #1 /srv/mediawiki/php-1.25wmf23/... [16:44:40] 10Wikimedia-Site-requests, 7Tracking: Maintenance scripts waiting to be run on Wikimedia wikis (tracking) - https://phabricator.wikimedia.org/T31782#1190390 (10MarkAHershberger) ----- Original Message ----- > Dereckson added a blocking task: T95110: add Reports and Press-releases > namespace to chapter wiki r... [16:45:09] 10SUL-Finalization, 7Epic, 3Roadmap, 7Tracking, and 2 others: Complete unification of all accounts to SUL - https://phabricator.wikimedia.org/T37707#1190397 (10MarkAHershberger) ----- Original Message ----- > Tgr added a blocked task: T74469: OAuth: Authorisation should not fail > because you don't have a... [16:47:17] ^ wth [16:47:42] 6Phabricator, 5Patch-For-Review: MediaWiki login for Phabricator produces broken links for Wikimedia profiles with URL encoded characters - https://phabricator.wikimedia.org/T547#1190402 (10mmodell) [16:49:25] Glaisher, email bugs [16:49:28] presumably [16:49:43] not particularly surprising [16:49:46] yeah, I figured. It says "via email" [16:49:47] 10Programs-Evaluation: GLAM data pulls - https://phabricator.wikimedia.org/T95439#1190412 (10Abit) 3NEW a:3Abit [16:54:18] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Diff of a qualifier shows the value of the new property instead of the old property - https://phabricator.wikimedia.org/T87096#1190435 (10Ricordisamoa) [16:55:39] (03PS1) 10Aaron Schulz: Lowered the updatePageTriageQueue batch size to 100 [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/202769 [16:58:08] 10MediaWiki-extensions-Graph, 10Parsoid, 10Possible-Tech-Projects, 6Services, and 3 others: Graph editing in VisualEditor - https://phabricator.wikimedia.org/T89287#1190457 (10Jdforrester-WMF) [17:01:49] 3Mobile Web Design, 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2, 5Patch-For-Review: Stop serving menu in the page HTML - https://phabricator.wikimedia.org/T95332#1190469 (10phuedx) a:5phuedx>3None [17:02:05] (03CR) 10BryanDavis: [C: 04-2] "Obsoleted by I769c8da0509c8cd6d30955fe18228b19ffa9d216" [vagrant] - 10https://gerrit.wikimedia.org/r/200850 (https://phabricator.wikimedia.org/T94469) (owner: 10Mobrovac) [17:02:39] (03CR) 10BryanDavis: [C: 04-2] "Obsoleted by I769c8da0509c8cd6d30955fe18228b19ffa9d216" [vagrant] - 10https://gerrit.wikimedia.org/r/202579 (owner: 10Sbisson) [17:03:24] (03CR) 10Legoktm: [C: 032] Prevent SiteStats::loadAndLazyInit from reloading in $wgMiserMode [core] - 10https://gerrit.wikimedia.org/r/202766 (https://phabricator.wikimedia.org/T95426) (owner: 10Aaron Schulz) [17:04:40] 6Scrum-of-Scrums, 10Wikimedia-Search, 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2: Scope a spike around implementing https://meta.wikimedia.org/wiki/Schema:Search for Desktop - https://phabricator.wikimedia.org/T90518#1190483 (10bmansurov) [17:07:17] 10MediaWiki-RfCs, 10Wikimedia-General-or-Unknown: cleaner URLs for editing, previewing skins, etc. - https://phabricator.wikimedia.org/T14619#1190488 (10GWicke) @spage, my focus is currently more on the API portion, which is discussed in {T95229}. I'm still supportive of the effort to clean up edit paths etc a... [17:08:41] (03CR) 10Mobrovac: "Now that the actual directory is in hiera, should be make run-git-update a template and set the dir to that fact?" [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [17:09:37] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2: Mobile menu improvements (alpha) - Remove ® from wordmark - https://phabricator.wikimedia.org/T95007#1190492 (10JKatzWMF) [17:09:40] 10MediaWiki-Database: Postgres SQL error comparing text and integer in getRangeCond() from includes/Block.php - https://phabricator.wikimedia.org/T89171#1190493 (10Jjanes) 5stalled>3Invalid [17:09:43] 10MediaWiki-Database, 7Tracking: PostgreSQL/pgsql support (tracking) - https://phabricator.wikimedia.org/T2384#1190494 (10Jjanes) [17:10:16] 10Programs-Evaluation: GLAM Data Checks - https://phabricator.wikimedia.org/T95440#1190495 (10Abit) 3NEW [17:12:25] 10FR-Campaigns: Fundraise in Ukraine: April 20ish - https://phabricator.wikimedia.org/T94285#1190511 (10atgo) [17:15:33] (03CR) 10Legoktm: [C: 032] Made User::invalidateCache() use touch() instead of the DB query [core] - 10https://gerrit.wikimedia.org/r/202659 (owner: 10Aaron Schulz) [17:16:38] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 5Patch-For-Review: localize display of time values - https://phabricator.wikimedia.org/T50962#1190518 (10hoo) IMO this bug is solved, as these can be localized now... but we should probably create some bug about having a better (more flexible) localizati... [17:17:35] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Use JSON and n-triples files to test RdfBuilder. - https://phabricator.wikimedia.org/T90616#1190521 (10Smalyshev) 5Open>3Resolved a:3Smalyshev I think we are already doing it now. [17:17:42] (03Merged) 10jenkins-bot: Prevent SiteStats::loadAndLazyInit from reloading in $wgMiserMode [core] - 10https://gerrit.wikimedia.org/r/202766 (https://phabricator.wikimedia.org/T95426) (owner: 10Aaron Schulz) [17:17:50] (03CR) 10MaxSem: [C: 032 V: 032] Disable preferences save spam [services/hierator] - 10https://gerrit.wikimedia.org/r/201091 (owner: 10MaxSem) [17:18:52] (03PS1) 10Manybubbles: Handle expanded values [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202775 [17:20:13] 3Mobile Web Design, 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2: Mobile menu improvements (alpha) - https://phabricator.wikimedia.org/T94458#1163900 (10phuedx) [17:20:50] (03PS1) 10Glaisher: Allow to set a limit on global blocks on IP ranges [extensions/GlobalBlocking] - 10https://gerrit.wikimedia.org/r/202776 (https://phabricator.wikimedia.org/T46836) [17:20:54] 10MediaWiki-extensions-GlobalBlocking, 7Ipv6, 5Patch-For-Review: Allow to set limits on global blocks on IP ranges - https://phabricator.wikimedia.org/T46836#1190556 (10Glaisher) a:3Glaisher [17:21:22] 10MediaWiki-extensions-GlobalBlocking, 5Patch-For-Review: Allow to set limits on global blocks on IP ranges - https://phabricator.wikimedia.org/T46836#1190563 (10Glaisher) [17:22:06] (03CR) 10BryanDavis: Make Parsoid work with multiwiki. (032 comments) [vagrant] - 10https://gerrit.wikimedia.org/r/199591 (owner: 10Werdna) [17:25:15] 10MediaWiki-Vagrant, 3Fundraising Sprint House of Pain, 10Wikimedia-Fundraising, 3Fundraising Sprint Grandmaster Flash, 5Patch-For-Review: Vagrant Fundraising role needs to be able to run a specific MediaWiki branch - https://phabricator.wikimedia.org/T78739#1190568 (10awight) [17:25:42] (03CR) 1001tonythomas: "Nah. we wont need separate alert for CA user un-subscribe. Passing the mail id too to the flyout-message would be the only addition needed" (031 comment) [extensions/BounceHandler] - 10https://gerrit.wikimedia.org/r/202735 (https://phabricator.wikimedia.org/T94436) (owner: 10Tinaj1234) [17:27:30] (03PS2) 10Manybubbles: Handle expanded values [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202775 [17:28:05] (03PS4) 10Alex Monk: Collect only 25% of WikiEditor events [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/199132 (https://phabricator.wikimedia.org/T93201) [17:28:55] 6Phabricator, 6Engineering-Community, 6Project-Creators, 7Analytics-Volunteering, and 2 others: Analytics-Volunteering and Wikidata's Need-Volunteer tags; "New contributors" vs "volunteers" terms - https://phabricator.wikimedia.org/T88266#1190591 (10kevinator) @Aklapper Yes, I have discussed with my team a... [17:29:18] 3Team-Practices-This-Week: Document team roles - https://phabricator.wikimedia.org/T88334#1190593 (10ksmith) I would love to have Joel's fresh eyes look the doc over before we call it final. [17:29:48] 6MediaWiki-API-Team, 10MediaWiki-extensions-OAuth: OAuth: Authorisation should not fail because you don't have an account on central wiki - https://phabricator.wikimedia.org/T74469#1190598 (10Nemo_bis) >>! In T74469#752694, @csteipp wrote: > Trying to force an autocreation on the central wiki as part of the au... [17:31:12] (03Merged) 10jenkins-bot: Made User::invalidateCache() use touch() instead of the DB query [core] - 10https://gerrit.wikimedia.org/r/202659 (owner: 10Aaron Schulz) [17:33:14] (03PS1) 10Aaron Schulz: Made WikiPage::doPurge() avoid calling commit() [core] - 10https://gerrit.wikimedia.org/r/202778 [17:34:50] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1190608 (10csteipp) Just a couple things so far, 1) A service like this should be setting the http headers, ``` X-XSS-Pro... [17:37:20] 10MediaWiki-Vagrant, 3Fundraising Sprint House of Pain, 10Wikimedia-Fundraising, 3Fundraising Sprint Grandmaster Flash, 5Patch-For-Review: Vagrant Fundraising role needs to be able to run a specific MediaWiki branch - https://phabricator.wikimedia.org/T78739#1190616 (10awight) [17:37:23] 10Hovercards, 10Wikimedia-Extension-setup: Enable Hovercards on se.wikimedia.org (Swedish chapter wiki) - https://phabricator.wikimedia.org/T68374#1190614 (10Vibhabamba) @jaredzimmerman @quiddity Did we respond to this community about switching it on for the 2 month trial? It seems like its large in terms of '... [17:37:39] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1190617 (10GWicke) @csteipp, do you see any reasons for not setting those headers in all RB responses? [17:38:52] 10Programs-Evaluation: GLAM data checks - https://phabricator.wikimedia.org/T95440#1190619 (10Abit) [17:39:54] (03CR) 10BryanDavis: "The hiera variable isn't attached to a Puppet class so there is no way to get it. :( Using scope[] returns an empty string." [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [17:45:57] (03CR) 10Chad: Begin 1.26 development cycle (031 comment) [core] - 10https://gerrit.wikimedia.org/r/202587 (owner: 10Chad) [17:46:26] (03CR) 10Nikerabbit: "Not sure I understand. Are you saying that MediaWiki prevents account creation if $wgEmailConfirmToEdit is set to true?" [extensions/InviteSignup] - 10https://gerrit.wikimedia.org/r/202478 (owner: 10JoelKP) [17:46:54] (03CR) 10Daniel Kinzler: [C: 04-1] Disable action=parse web API (031 comment) [extensions/Lockdown] - 10https://gerrit.wikimedia.org/r/193563 (owner: 10Nemo bis) [17:47:10] (03PS1) 10Chad: Fix I9e63789e: Release notes should refer to old versions in HISTORY [core] - 10https://gerrit.wikimedia.org/r/202782 [17:47:18] <^d> legoktm: Trivial ^ [17:47:35] (03CR) 10Legoktm: [C: 032] Fix I9e63789e: Release notes should refer to old versions in HISTORY [core] - 10https://gerrit.wikimedia.org/r/202782 (owner: 10Chad) [17:48:01] 4(no projects): Unable to use links when explaining why we are hiding a Flow topic - https://phabricator.wikimedia.org/T95445#1190670 (10He7d3r) 3NEW [17:48:06] <^d> thx [17:49:22] (03CR) 10Chad: [C: 032] Removed unused wgDBAvgStatusPoll var [core] - 10https://gerrit.wikimedia.org/r/202604 (owner: 10Aaron Schulz) [17:49:27] 10MediaWiki-Search, 6Collaboration-Team, 10Collaboration-Team-Sprint-T-2015-04-08, 10Flow, and 2 others: Search indexed Flow data - https://phabricator.wikimedia.org/T78789#1190691 (10DannyH) [17:50:10] (03CR) 10Chad: [C: 032] Set a query group on SpecialRecentChanges query [core] - 10https://gerrit.wikimedia.org/r/202632 (owner: 10Aaron Schulz) [17:50:48] (03CR) 10Chad: [C: 032] Made mergeActiveUsers use "recentchanges" query group [core] - 10https://gerrit.wikimedia.org/r/202634 (owner: 10Aaron Schulz) [17:51:28] (03CR) 10Chad: [C: 032] Log huge write queries in CLI scripts [core] - 10https://gerrit.wikimedia.org/r/202640 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [17:51:49] (03CR) 10Chad: [C: 032] Use wfWaitForSlaves in upload stash cleanup script [core] - 10https://gerrit.wikimedia.org/r/202763 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [17:52:06] (03CR) 10Chad: [C: 032] Lowered the updatePageTriageQueue batch size to 100 [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/202769 (owner: 10Aaron Schulz) [17:52:24] 4(no projects): Spike: how can we create diagnostic that VE team built? - https://phabricator.wikimedia.org/T95447#1190698 (10JKatzWMF) 3NEW [17:53:16] (03CR) 10Chad: [C: 032] Made WikiPage::doPurge() avoid calling commit() [core] - 10https://gerrit.wikimedia.org/r/202778 (owner: 10Aaron Schulz) [17:54:02] (03CR) 10Smalyshev: Handle expanded values (031 comment) [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202775 (owner: 10Manybubbles) [17:54:38] 10MediaWiki-extensions-Graph, 10MediaWiki-extensions-WikiHiero, 10Citoid, 10Mathoid-General-or-Unknown, and 4 others: Services need to share the same domain name - https://phabricator.wikimedia.org/T95402#1190711 (10Yurik) [17:54:52] (03CR) 10Chad: [C: 032] Make RedisBagOStuff support tag name hashing [core] - 10https://gerrit.wikimedia.org/r/202329 (https://phabricator.wikimedia.org/T95126) (owner: 10Aaron Schulz) [17:55:06] (03CR) 10Smalyshev: [C: 032] Compress Entity and EntityData into WikibaseUris [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 (owner: 10Manybubbles) [17:55:08] (03CR) 10jenkins-bot: [V: 04-1] Compress Entity and EntityData into WikibaseUris [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 (owner: 10Manybubbles) [17:56:37] (03Abandoned) 10Nemo bis: Disable action=parse web API [extensions/Lockdown] - 10https://gerrit.wikimedia.org/r/193563 (owner: 10Nemo bis) [17:57:50] (03CR) 10Smalyshev: "oops, needs a rebase now" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 (owner: 10Manybubbles) [17:58:41] 10Programs-Evaluation: GLAM data pulls - https://phabricator.wikimedia.org/T95439#1190736 (10Abit) p:5Triage>3High [18:00:13] 10Hovercards, 10Wikimedia-Extension-setup: Enable Hovercards on se.wikimedia.org (Swedish chapter wiki) - https://phabricator.wikimedia.org/T68374#1190740 (10Vibhabamba) I did some testing for the different layouts - {F110125} {F110126} {F110129} {F110131} {F110133} [18:02:49] ah, anomie, I think I can get page_id and namespace from the TItle object [18:02:58] that onAPIAfterExecute has [18:02:59] 7Design, 10MediaWiki-Interface, 10MediaWiki-Page-editing: Add section edit link for 0th section of a page/article - https://phabricator.wikimedia.org/T2156#1190748 (10SamB) [18:03:38] (03CR) 10Legoktm: [C: 032] Fixed broken batching in –PurgeOldLogIPData [extensions/AbuseFilter] - 10https://gerrit.wikimedia.org/r/202657 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [18:03:48] ottomata: You must be referring to something specific in MobileFrontend, becuase the APIAfterExecute hook doesn't have a title. [18:04:05] naw it does [18:04:07] oh well [18:04:10] it gets it from data [18:04:25] but yes [18:04:37] 10MediaWiki-API: Many false positive warnings about missing continuation parameter - https://phabricator.wikimedia.org/T74415#1190754 (10Umherirrender) FYI: https://gerrit.wikimedia.org/r/#/c/201193/ [18:04:37] anomie: https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/includes/api/ApiParseExtender.php#L78 [18:05:06] so, since MobileFrontend already has logic to deal with X-Analytics header, i think i will try to just add page_id and namespace in this already existent hook, unless you think i should do something different [18:05:13] ottomata: In that "parse extender"? $data['parse'] is only going to be set for action=parse. If that's the only API action you care about then that's good, but if not then you haven't solved your whole problem. [18:05:42] hm [18:05:47] ph [18:05:48] oh [18:06:24] hm, so maybe a new hook that checks [18:06:27] action=mobileview [18:06:28] ? [18:06:40] YuviPanda: how do I do that? (that was your suggestion) [18:06:57] $module->getModuleName() == 'mobileview' ? [18:07:05] or [18:07:09] check if $data['mobileview'] ? [18:10:43] ottomata: legoktm might be able to help? I dont know the API methods off hand and am not at a computer yet [18:10:59] ah ok, thank you [18:11:56] legoktm: , yt? [18:12:02] hey [18:12:04] what's up? [18:12:37] mm, X-Analytics [18:13:13] ottomata: is there a bug for this? [18:13:21] (03PS1) 10BearND: Add more HTML structure [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202784 [18:13:23] (03PS1) 10BearND: Serve js bundle from service [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202785 [18:13:25] (03PS1) 10BearND: Fix DOM structure [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202786 [18:13:27] (03PS1) 10BearND: Move first JS metadata block up again. [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202787 [18:13:36] legoktm: yews [18:13:37] yes [18:13:51] https://phabricator.wikimedia.org/T92875 [18:14:26] (03PS2) 10Matthias Mullie: Upgrade to Handlebars 3.0.0 [extensions/Flow] - 10https://gerrit.wikimedia.org/r/198709 (https://phabricator.wikimedia.org/T93851) [18:14:42] (03CR) 10BearND: [C: 032] Add more HTML structure [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202784 (owner: 10BearND) [18:14:53] (03CR) 10BearND: [C: 032] Serve js bundle from service [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202785 (owner: 10BearND) [18:14:56] 10Programs-Evaluation: GLAM charts - https://phabricator.wikimedia.org/T95451#1190792 (10Abit) a:3Abit [18:14:56] we want page_id and namespace in X-Analytics from mobile app requests [18:14:56] so, on pageview from mobile app [18:14:56] add page_id to X-analytics [18:14:56] i'm not exactly sure how to identify a 'pageview' in API [18:14:56] and i don't know very much about the mediawiki codebase either! :) [18:15:02] right now i'm thinking another APIAfterExecute hook [18:15:05] that checks somewow if the request is for a page [18:15:07] 10Programs-Evaluation: GLAM charts - https://phabricator.wikimedia.org/T95451#1190782 (10Abit) p:5Triage>3Normal [18:15:14] (03CR) 10BearND: [V: 032] Serve js bundle from service [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202785 (owner: 10BearND) [18:15:26] or, just add it as another conditional to the existant onAPIAfterExecute in MobileFrontend [18:15:51] I think we should have mobile frontend call something like XAnalytics::addHeader( 'page_id', ###); and then the extension takes care of outputting it in the APIAfterExecute hook [18:16:21] hmm so like [18:16:27] https://phabricator.wikimedia.org/rEXAN64e43ad61363ff69d2bad173a6693ff311876ec0#C1546702NL26 [18:16:54] yeah so right now it has a BeforePageDisplay hook [18:17:05] we'd also add a APIAfterExecute hook [18:17:44] 4(no projects): GLAM narrative draft - https://phabricator.wikimedia.org/T95453#1190877 (10Abit) 3NEW a:3Abit [18:17:56] (03CR) 10BearND: [V: 032] Add more HTML structure [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202784 (owner: 10BearND) [18:18:10] but with APIAfterExecute [18:18:10] ? [18:18:10] makes sense, that is what anomie was saying too [18:18:10] aye [18:18:11] hm [18:18:11] so [18:18:12] MobileFrontend's MobileContext.php has functions for dealing with X-Analytics [18:18:16] (03CR) 10BearND: [C: 032 V: 032] Fix DOM structure [services/mobileapps] - 10https://gerrit.wikimedia.org/r/202786 (owner: 10BearND) [18:18:19] that were made before this extension was. [18:18:21] (03CR) 10Hoo man: [C: 04-1] "After having a short look at the code I have to agree with Daniel: It would be nice if we could move in a direction that greatly improves " [core] - 10https://gerrit.wikimedia.org/r/182858 (https://phabricator.wikimedia.org/T76728) (owner: 10Anomie) [18:19:21] onBeforePageDisplay actually outputs it in MobileFrontend [18:19:35] (03CR) 10JoelKP: "With InviteSignup, yes. There is a check in MediaWiki's specials/SpecialUserlogin.php, which disallows creating an account with a blank em" [extensions/InviteSignup] - 10https://gerrit.wikimedia.org/r/202478 (owner: 10JoelKP) [18:19:48] but, i think that doesn't happen because most API requets don't trigger BeforePageDisplay, right? [18:19:59] yeah [18:20:40] do you want me to submit a patch of what I'm thinking about? [18:21:06] would love that legoktm, I am still faliing around between extensions and core trying to understand this [18:21:08] (03Merged) 10jenkins-bot: Fix I9e63789e: Release notes should refer to old versions in HISTORY [core] - 10https://gerrit.wikimedia.org/r/202782 (owner: 10Chad) [18:21:29] flailing * [18:22:14] (03Merged) 10jenkins-bot: Removed unused wgDBAvgStatusPoll var [core] - 10https://gerrit.wikimedia.org/r/202604 (owner: 10Aaron Schulz) [18:31:33] (03CR) 10Manybubbles: Handle expanded values (031 comment) [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202775 (owner: 10Manybubbles) [18:32:24] (03CR) 10jenkins-bot: [V: 04-1] Set a query group on SpecialRecentChanges query [core] - 10https://gerrit.wikimedia.org/r/202632 (owner: 10Aaron Schulz) [18:33:36] (03PS1) 10Smalyshev: Check updates on properties too [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202789 [18:34:00] (03CR) 10Aaron Schulz: "Recheck" [core] - 10https://gerrit.wikimedia.org/r/202632 (owner: 10Aaron Schulz) [18:36:19] 10MediaWiki-Internationalization, 7Easy, 5MW-1.23-release: "Show"/"Hide" links in recent changes / watchlist options box should be localizable per item - https://phabricator.wikimedia.org/T60449#1191049 (10Umar) Why not fix it and close the query remains a little? [18:37:40] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191059 (10csteipp) Correct me if I got this wrong, but it looks like it's using https://github.com/Automattic/node-canvas... [18:38:21] 10Community-Wikimetrics: "Create Report" button does not appear when uploading a new cohort - https://phabricator.wikimedia.org/T95456#1191060 (10Abit) 3NEW [18:38:50] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191076 (10mobrovac) Thank you for the review @csteipp! Very helpful. Regarding CSP, probably something like this would wo... [18:38:53] (03PS1) 10Aaron Schulz: Fixed broken batching in –PurgeOldLogIPData [extensions/AbuseFilter] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202792 (https://phabricator.wikimedia.org/T95382) [18:41:34] 10MediaWiki-extensions-CentralAuth: Auto-login is creating accounts - https://phabricator.wikimedia.org/T18864#1191083 (10Tgr) See T74469#1172639 for an example (the task has reproduction steps). [18:42:02] (03PS2) 10Ori.livneh: Made wfReadOnly() more robust at handling the lagged-slave case [core] - 10https://gerrit.wikimedia.org/r/202457 (owner: 10Aaron Schulz) [18:42:07] (03CR) 10Ori.livneh: [C: 032] Made wfReadOnly() more robust at handling the lagged-slave case [core] - 10https://gerrit.wikimedia.org/r/202457 (owner: 10Aaron Schulz) [18:43:28] 10Phabricator-Upstream, 6WMF-Legal, 5Patch-For-Review: In the footer, make "Creative Commons Attribution-ShareAlike 3.0 License" and "GNU General Public License" links - https://phabricator.wikimedia.org/T94946#1191086 (10Josve05a) [18:44:15] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191087 (10csteipp) >>! In T93126#1190617, @GWicke wrote: > @csteipp, do you see any reasons for not setting those headers... [18:44:42] (03Merged) 10jenkins-bot: Made mergeActiveUsers use "recentchanges" query group [core] - 10https://gerrit.wikimedia.org/r/202634 (owner: 10Aaron Schulz) [18:44:48] (03Merged) 10jenkins-bot: Log huge write queries in CLI scripts [core] - 10https://gerrit.wikimedia.org/r/202640 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [18:45:21] 10Community-Wikimetrics: Email Wikimetrics list re. vanishing "Create Report" button - https://phabricator.wikimedia.org/T94642#1191090 (10Abit) Oop, sorry Jonathan, I just created a task as I emailed the list--feel free to update or change it as you like! https://phabricator.wikimedia.org/T95456 The only way... [18:45:33] 10Community-Wikimetrics: Email Wikimetrics list re. vanishing "Create Report" button - https://phabricator.wikimedia.org/T94642#1191091 (10Abit) 5Open>3Resolved [18:45:49] (03CR) 10Manybubbles: "Dupe of Iccd2f8a597439f3c9f3002cf0bcebef101c781df ?" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202789 (owner: 10Smalyshev) [18:46:33] (03CR) 10Manybubbles: [C: 031] Render math tags in search results [extensions/Math] - 10https://gerrit.wikimedia.org/r/197737 (https://phabricator.wikimedia.org/T93075) (owner: 10Physikerwelt) [18:47:15] (03CR) 10Smalyshev: "Sure, that works too :) haven;t seen it." [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202789 (owner: 10Smalyshev) [18:48:04] 10Programs-Evaluation: GLAM chart checks - https://phabricator.wikimedia.org/T95458#1191135 (10Abit) 3NEW [18:48:04] (03CR) 10Smalyshev: [C: 031] Make sure property edits show up in recent changes [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 (owner: 10Manybubbles) [18:48:23] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191144 (10csteipp) >>! In T93126#1191076, @mobrovac wrote: > Thank you for the review @csteipp! Very helpful. > > Regardi... [18:48:25] 10Programs-Evaluation: GLAM chart checks - https://phabricator.wikimedia.org/T95458#1191145 (10Abit) p:5Triage>3Normal [18:48:46] (03CR) 10Manybubbles: "What you need for +2? :)" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 (owner: 10Manybubbles) [18:49:07] 10Programs-Evaluation: GLAM narrative draft - https://phabricator.wikimedia.org/T95453#1191146 (10Abit) [18:49:32] 10Programs-Evaluation: GLAM chart checks - https://phabricator.wikimedia.org/T95458#1191135 (10Abit) [18:49:34] 4(no projects): GLAM chart checks - https://phabricator.wikimedia.org/T95452#1191148 (10Abit) [18:49:55] (03PS1) 10Legoktm: Move BeforePageDisplay hook to separate class [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202799 [18:49:57] (03PS1) 10Legoktm: Add X-Analytics header for API requests too [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202800 (https://phabricator.wikimedia.org/T92875) [18:49:59] (03PS1) 10Legoktm: Allow extensions to add items at runtime with XAnalytics::addItem() [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202801 [18:50:53] ottomata: what API query do the mobile apps make? [18:51:22] haha, uhhh [18:51:56] in the ticket, YuviPanda says action=mobileview [18:52:05] does that make sense? [18:54:40] 6Phabricator, 6Team-Practices: Show tags in Phab Sprint and Workboard UI - https://phabricator.wikimedia.org/T95343#1191166 (10Fjalapeno) [18:55:19] (03CR) 10Dduvall: "That's something I didn't think about when defining those root hiera variables. Maybe we should reintroduce a `vagrant` module to encapsul" [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [18:55:27] (03CR) 10Smalyshev: [C: 032] Make sure property edits show up in recent changes [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 (owner: 10Manybubbles) [18:55:43] marxarelli: I'm working on a patch for that now :) [18:55:46] (03CR) 10Smalyshev: "Sorry, wasn't sure it's ready since you haven't put it for review :)" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 (owner: 10Manybubbles) [18:55:59] marxarelli: ::mwv is the class I'm going to introduce [18:55:59] 10Hovercards, 10Wikimedia-Extension-setup: Enable Hovercards on se.wikimedia.org (Swedish chapter wiki) - https://phabricator.wikimedia.org/T68374#1191167 (10Ricordisamoa) >>! In T68374#1190614, @Vibhabamba wrote: > @jaredzimmerman @quiddity Did we respond to this community about switching it on for the 2 mont... [18:56:15] bd808: oh, nice [18:56:40] (03CR) 10Manybubbles: "Ah! That was just me forgetting. I'll always put WIP before commits I don't want to merge. Or -1 my own patch." [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 (owner: 10Manybubbles) [18:57:29] (03Merged) 10jenkins-bot: Make sure property edits show up in recent changes [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202644 (owner: 10Manybubbles) [18:57:59] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191168 (10mobrovac) >>! In T93126#1191059, @csteipp wrote: > Correct me if I got this wrong, but it looks like it's using... [18:58:47] ottomata: ok, https://gerrit.wikimedia.org/r/#/c/202801/ and https://gerrit.wikimedia.org/r/202802 should do it I think [19:01:24] 10MediaWiki-API, 6MediaWiki-API-Team, 6Analytics-Engineering, 10Wikipedia-Android-App, and 2 others: Add page_id and namespace to X-Analytics header in App / api requests - https://phabricator.wikimedia.org/T92875#1191182 (10Legoktm) [19:05:15] 10MediaWiki-extensions-ContentTranslation, 6Collaboration-Team, 10Flow, 10VisualEditor, and 2 others: HTML entities in CommentNode must be manually escaped/unescaped - https://phabricator.wikimedia.org/T95040#1191200 (10Jdforrester-WMF) [19:05:46] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, 7Blocked-on-Security: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191203 (10mobrovac) >>! In T93126#1191087, @csteipp wrote: >>>! In T93126#1190617, @GWicke wrote: >> @csteipp, do you see... [19:10:46] Ooo, legoktm looking.. [19:12:24] (03PS6) 10Cenarium: Single hook to register tags, ChangeTag class and perf improvements [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) [19:12:55] 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: Make label/description available for plain item/property links - https://phabricator.wikimedia.org/T95463#1191221 (10SamB) 3NEW [19:14:13] (03PS8) 10Tinaj1234: Display warning message in Special:ListFiles when invalid username entered [core] - 10https://gerrit.wikimedia.org/r/197923 (https://phabricator.wikimedia.org/T72784) [19:15:20] (03CR) 10jenkins-bot: [V: 04-1] Single hook to register tags, ChangeTag class and perf improvements [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) (owner: 10Cenarium) [19:15:28] 6Phabricator, 10Wikimedia-Bugzilla, 6operations: Sanitise a Bugzilla database dump - https://phabricator.wikimedia.org/T85141#1191236 (10Eloquence) @Slaporte is the tech liaison from legal now that Luis runs community. Daniel and/or Andre, my recommendation would be that you set up a quick time with him to w... [19:15:51] hmm, so legoktm, it is possible in API that one of the two hooks might be called [19:15:52] ? [19:17:05] (03CR) 10Legoktm: Maintenance script for deleting empty global accounts (032 comments) [extensions/CentralAuth] - 10https://gerrit.wikimedia.org/r/202588 (https://phabricator.wikimedia.org/T93167) (owner: 10Anomie) [19:18:03] 6MediaWiki-API-Team, 10SUL-Finalization, 5Patch-For-Review: Check for empty global accounts - https://phabricator.wikimedia.org/T93167#1191262 (10Legoktm) p:5Triage>3High a:3Anomie [19:18:56] ottomata: so the APIAfterExecute hook will always be called on API requests, and some will also trigger the BeforePageDisplay hook [19:19:21] (03PS2) 10Ottomata: Allow extensions to add items at runtime with XAnalytics::addItem() [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202801 (https://phabricator.wikimedia.org/T92875) (owner: 10Legoktm) [19:19:30] (03PS2) 10Ottomata: Move BeforePageDisplay hook to separate class [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202799 (https://phabricator.wikimedia.org/T92875) (owner: 10Legoktm) [19:19:51] aye, and so you just guard to make sure it isn't output more than once. [19:20:10] too bad there isn't a BeforeOutputStarts hook [19:20:22] (03Merged) 10jenkins-bot: Use wfWaitForSlaves in upload stash cleanup script [core] - 10https://gerrit.wikimedia.org/r/202763 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [19:20:25] (03Merged) 10jenkins-bot: Lowered the updatePageTriageQueue batch size to 100 [extensions/PageTriage] - 10https://gerrit.wikimedia.org/r/202769 (owner: 10Aaron Schulz) [19:20:32] (03Merged) 10jenkins-bot: Made WikiPage::doPurge() avoid calling commit() [core] - 10https://gerrit.wikimedia.org/r/202778 (owner: 10Aaron Schulz) [19:20:40] (03Merged) 10jenkins-bot: Make RedisBagOStuff support tag name hashing [core] - 10https://gerrit.wikimedia.org/r/202329 (https://phabricator.wikimedia.org/T95126) (owner: 10Aaron Schulz) [19:20:47] (03Merged) 10jenkins-bot: Fixed broken batching in –PurgeOldLogIPData [extensions/AbuseFilter] - 10https://gerrit.wikimedia.org/r/202657 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [19:22:14] 10Labs-Vagrant, 5Patch-For-Review: Labs-vagrant broken or docs out of sync ? - https://phabricator.wikimedia.org/T94469#1191333 (10mobrovac) 5Open>3Resolved Fixed with the merge of https://gerrit.wikimedia.org/r/#/c/200624/ [19:23:00] (03PS7) 10Cenarium: Single hook to register tags, ChangeTag class and perf improvements [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) [19:23:08] 10MediaWiki-Vagrant, 5Patch-For-Review: Error: Cannot create /vagrant/phab/libphutil; parent directory /vagrant/phab does not exist - https://phabricator.wikimedia.org/T94553#1191347 (10mobrovac) 5Open>3Resolved a:3mobrovac Fixed in https://gerrit.wikimedia.org/r/#/c/200624/ , resolving. [19:24:36] (03CR) 10Smalyshev: "Replaced by https://gerrit.wikimedia.org/r/#/c/202644/" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202789 (owner: 10Smalyshev) [19:24:42] (03Abandoned) 10Smalyshev: Check updates on properties too [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202789 (owner: 10Smalyshev) [19:25:14] 7Design, 10MediaWiki-extensions-GuidedTour, 10VisualEditor, 10VisualEditor-MediaWiki, 3Editing Department 2014/15 Q4 blockers: Change Welcome dialogue content to more helpful message to new users based on the GuidedTour - https://phabricator.wikimedia.org/T89072#1026973 (10Jdforrester-WMF) [19:27:17] 7Design, 10VisualEditor, 10VisualEditor-MediaWiki-Links, 3Editing Department 2014/15 Q4 blockers: Preview interface for link dialogue - https://phabricator.wikimedia.org/T88316#1009312 (10Jdforrester-WMF) [19:27:50] 7Design, 10OOjs-UI, 10VisualEditor, 3Editing Department 2014/15 Q4 blockers: MediaWiki theme: Alter the toolbar and dropdown menu design - https://phabricator.wikimedia.org/T86693#1191415 (10Jdforrester-WMF) a:3Nirzar [19:29:24] (03PS1) 10Dduvall: Use default `git::clone` permissions for Parsoid [vagrant] - 10https://gerrit.wikimedia.org/r/202827 [19:38:11] (03Merged) 10jenkins-bot: Made wfReadOnly() more robust at handling the lagged-slave case [core] - 10https://gerrit.wikimedia.org/r/202457 (owner: 10Aaron Schulz) [19:40:08] (03CR) 10Dduvall: [C: 032] Cleanup .gitignore [vagrant] - 10https://gerrit.wikimedia.org/r/202638 (owner: 10BryanDavis) [19:40:34] (03Merged) 10jenkins-bot: Cleanup .gitignore [vagrant] - 10https://gerrit.wikimedia.org/r/202638 (owner: 10BryanDavis) [19:43:59] (03CR) 10Cenarium: Single hook to register tags, ChangeTag class and perf improvements (031 comment) [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) (owner: 10Cenarium) [19:46:09] 10WMF-Product-Strategy: Perform a Wikidata gap analysis - https://phabricator.wikimedia.org/T91423#1191480 (10Ironholds) Update: 1m-item retrieval done! [19:47:04] legoktm: all looks good to me, did you see my comment about namespace here:? https://gerrit.wikimedia.org/r/#/c/202802/ [19:49:44] 7Design, 10UI-Standardization, 6Mobile-Web, 10MobileFrontend, and 2 others: Identify the best diff style and use it everywhere - https://phabricator.wikimedia.org/T90948#1191490 (10TheDJ) Slightly related is T94292, where we note that we are actually missing a considerable amount of languages. [19:53:18] (03CR) 10Bartosz Dziewoński: [C: 032] Partially revert increased wikitable padding [core] - 10https://gerrit.wikimedia.org/r/198557 (https://phabricator.wikimedia.org/T91890) (owner: 10Gerrit Patch Uploader) [19:54:00] 10MediaWiki-extensions-SyntaxHighlight-(GeSHi): Combine infrequently-used language modules into a single module - https://phabricator.wikimedia.org/T93025#1191497 (10TheDJ) Slightly related is T94292, where i figured out, we are actually missing a considerable amount of languages, including a few new ones that m... [19:54:59] 10MediaWiki-extensions-SyntaxHighlight-(GeSHi): Combine infrequently-used language modules into a single module - https://phabricator.wikimedia.org/T93025#1191499 (10TheDJ) https://gerrit.wikimedia.org/r/197449 [19:56:33] 6Phabricator: Next Phabricator upgrade on YYYY-MM-DD - https://phabricator.wikimedia.org/T89830#1191501 (10chasemp) @christopher I am looking at 6f371a5a799bcbfb3b0dc95da11ba2ebc771b125 for sprint against d24b3dcb7de3c77caa299694d31f5f7f9bd1a2ff for phabricator as a next deployment here. Do you see a problem wi... [19:56:44] (03PS1) 10BryanDavis: Attach directory hiera config to a Puppet class [vagrant] - 10https://gerrit.wikimedia.org/r/202906 [19:56:56] 6Phabricator: Next Phabricator upgrade on YYYY-MM-DD - https://phabricator.wikimedia.org/T89830#1191504 (10chasemp) a:3chasemp [19:56:59] 10MediaWiki-extensions-DonationInterface, 3Fundraising Sprint House of Pain, 10Fundraising Tech Backlog, 3Fundraising Sprint Grandmaster Flash: Use the new queue wrapper for adding messages to the queues - https://phabricator.wikimedia.org/T92921#1191505 (10awight) Unblocked. [19:57:41] (03Abandoned) 10Bartosz Dziewoński: sectionAnchor:Move § from HTML to CSS [core] - 10https://gerrit.wikimedia.org/r/194377 (https://phabricator.wikimedia.org/T18691) (owner: 10Gerrit Patch Uploader) [19:57:43] (03Abandoned) 10Bartosz Dziewoński: mediawiki.sectionAnchor: Decrease contrast for section anchor symbol [core] - 10https://gerrit.wikimedia.org/r/195032 (https://phabricator.wikimedia.org/T18691) (owner: 10Gerrit Patch Uploader) [19:58:14] (03CR) 10Umherirrender: [C: 032] "Resubmit to give jenkins a second try" [core] - 10https://gerrit.wikimedia.org/r/202632 (owner: 10Aaron Schulz) [19:58:15] 10MediaWiki-RfCs: MediaWiki RFC: Exposure of user IP addresses - https://phabricator.wikimedia.org/T95144#1191517 (10daniel) a:3MZMcBride Tagging this as "draft" for now, assigning to MZMcBride for lack of better knowledge. [19:58:20] (03CR) 10Bartosz Dziewoński: "So, is this safe to merge?" [core] - 10https://gerrit.wikimedia.org/r/107009 (https://phabricator.wikimedia.org/T27319) (owner: 10Jackmcbarn) [19:59:07] burritocat: Can you help me with a master/slave type problem? [19:59:23] * burritocat needs to change his nick ;) [19:59:40] burritocat: ApiVisualEditorEdit.php uses an internal API request to save a new revision of a page, then another internal API request to action=parse to parse the page [19:59:55] Except the latter uses the slave so we often get an old version back [20:00:08] Or a missingrev error because of split-brained-ness [20:00:53] 3Google-Summer-of-Code-2015: Allow contributors to update their own details in tech metrics directly - https://phabricator.wikimedia.org/T93943#1191520 (10Sarvesh.onlyme) @NiharikaKohli, I've implemented microtasks, though they uses a bit different technology as needed for this project. But source code of implem... [20:01:28] burritocat: Is there any way I can tell ApiParse to use DB_MASTER, or can we use DB_MASTER_IF_WRITE there, or is there a way to wait for slaves to sync or .... how would you recommend I solve this problem? [20:01:32] 10MediaWiki-RfCs, 6ArchCom: RfC: image and oldimage tables - https://phabricator.wikimedia.org/T589#1191521 (10daniel) a:3daniel Assigning to me; I'll contact Gilles and perhaps Erik about this. [20:02:36] (03CR) 10Bartosz Dziewoński: [V: 032] Partially revert increased wikitable padding [core] - 10https://gerrit.wikimedia.org/r/198557 (https://phabricator.wikimedia.org/T91890) (owner: 10Gerrit Patch Uploader) [20:02:50] RoanKattouw: curious, do you pass all the cookie stuff through? [20:02:55] 7Design, 10MediaWiki-Interface: Wikitable padding - https://phabricator.wikimedia.org/T91890#1191524 (10matmarex) 5Open>3Resolved a:3MZMcBride [20:03:00] 7Design, 10MediaWiki-Interface: Wikitable padding - https://phabricator.wikimedia.org/T91890#1098253 (10matmarex) Edokter's latest patch hopefully achieves the highest possible total happiness level. [20:03:08] e.g. does ChronProtector apply? [20:03:39] burritocat: $api = new ApiMain( new DerivativeRequest( $this->getRequest(), $apiParams, false ), true ); [20:03:41] (03PS1) 10Bartosz Dziewoński: Partially revert increased wikitable padding [core] (REL1_25) - 10https://gerrit.wikimedia.org/r/202907 (https://phabricator.wikimedia.org/T91890) [20:03:49] (03CR) 10Bartosz Dziewoński: [C: 032] Partially revert increased wikitable padding [core] (REL1_25) - 10https://gerrit.wikimedia.org/r/202907 (https://phabricator.wikimedia.org/T91890) (owner: 10Bartosz Dziewoński) [20:04:16] Where $apiRequest = array( 'action' => 'parse', 'page' => 'Foo', 'prop' => 'text|revid|categorieshtml|displaytitle' ); [20:04:25] I suppose I could add a revid there [20:04:37] and then add some logic in ApiParse to try master after slave if it doesn't find that revid [20:05:16] * burritocat doesn't like that kind of logic :) [20:05:45] so the request that did the save didn't finish yet, so I suppose no master positions would be in the session [20:06:23] 4(no projects): Use 'modules' and 'moduleStyles' keys in FlowActions to stop loading javascript on pages that are supposed to be no-javascript only - https://phabricator.wikimedia.org/T95468#1191539 (10EBernhardson) 3NEW [20:06:44] (03CR) 10EBernhardson: "fixing the nojs should be relatively trivial now, i added 'modules' and 'moduleStyles' keys to FlowActions which define which modules are " [extensions/Flow] - 10https://gerrit.wikimedia.org/r/197027 (https://phabricator.wikimedia.org/T95362) (owner: 10Matthias Mullie) [20:07:04] RoanKattouw: maybe the API could support giving it master position coordinates (e.g. db1034-bin.000976/843431247) as a param [20:07:26] though that would also slow it down if the stuff happens all at once [20:07:27] haha [20:07:41] it could be faster to just force the master [20:07:42] 6MediaWiki-API-Team, 10Librarization, 7Documentation, 5MW-1.25-release: Update [[mw:Composer]] to include information about usage with libraries - https://phabricator.wikimedia.org/T85172#1191556 (10greg) p:5Triage>3Normal [20:09:20] 6Phabricator: Next Phabricator upgrade on YYYY-MM-DD - https://phabricator.wikimedia.org/T89830#1191560 (10chasemp) spoke with @Awjrichards who is going to put phab-01 through it's paces for some QA by the 22nd of april [20:09:49] burritocat: How do I "force the master"? [20:10:08] well there is no way atm, I'm just thinking about how to handle that in theory [20:10:35] we don't have many cases of internal api requests like that, so it's a bit novel [20:10:41] Yeah [20:11:00] I mean especially since I have two of them that are in different conceptual requests but in the same real request, and they depend on each other [20:11:53] RoanKattouw: better yet, we can have an hmaced+timestamp (no reply) API key to trip the hasOrMadeRecentMasterChanges() method in LoadBalancer [20:12:11] that would cause the Revision classes to use DB_MASTER if stuff isn't found [20:12:20] *no replay [20:12:39] ebernhardson: Please add tasks to https://phabricator.wikimedia.org/T95468#1191539 [20:12:50] burritocat: Ooh that sounds interesting [20:13:06] it avoids using the master for everything, so it seems optimal-ish [20:13:09] 6WMF-NDA-Requests: Allow Valhallasw to sign an NDA - https://phabricator.wikimedia.org/T93644#1191577 (10valhallasw) OK, done so! [20:13:51] I guess the key could do it for all loadbalancers or we could be smart and pack a list of master names in there [20:14:10] But, hold on [20:14:18] These API requests are fake requests, rght [20:14:20] I mean, they're internal [20:14:41] So presumably hasOrMadeRecentMasterChanges() should already be true? [20:14:46] if they are fake, then I don't see why this happens, unless the rev ID is not qualified [20:14:57] Hmm maybe that's my problem [20:15:00] I don't currently qualify the revid [20:15:13] Does Revision.php try the master after the slave if you give it a revid and the slave can't find that revid? [20:15:25] the fallback logic only applies when a caller seems to be wanting a revision that isn't found at first [20:15:29] OK good [20:15:35] Then I'll pass the revid explicitly [20:15:47] when you just do newFromTitle() for the "latest", it's not clear if that logic should be applied [20:16:03] you might want to just always use slave data or stay in the same repeatable-read snapshot [20:16:46] Makes sense [20:17:48] since revisions are fairly idempotent, newFromId() and to a lesser extent newFromTitle/newFromPageId with rev ID (assuming moves are rare) basically don't have to care so much about snapshots and lag [20:18:16] of course this also assumes deletes are rare (though if we redid deletion like I want, the revision would still be there in the same table) [20:18:41] 6WMF-NDA-Requests: Allow Valhallasw to sign an NDA - https://phabricator.wikimedia.org/T93644#1191597 (10chasemp) >>! In T93644#1191577, @valhallasw wrote: > OK, done so! I added you to #wmf-nda and I'm leaving this to yuvi to handle the tool-labs things. [20:20:14] 10UI-Standardization, 10OOjs-UI, 3Editing Department 2014/15 Q4 blockers, 5OOjs-UI-0.9.5, 5WMF-deploy-2015-04-08_(1.26wmf1): Flags can only be applied to buttons - https://phabricator.wikimedia.org/T93890#1191605 (10Jdforrester-WMF) [20:22:14] (03PS2) 10BryanDavis: Update run-git-update for new /vagrant/srv directory [vagrant] - 10https://gerrit.wikimedia.org/r/202639 [20:24:00] (03CR) 10BryanDavis: "New and improved to use the ::mwv class variables from I6f2416e4a7ad9f6db07e335d8250b0dcbe1083c6 to template the script." [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [20:25:00] burritocat: OK so I did this https://gerrit.wikimedia.org/r/202911 [20:25:20] burritocat: Which is basically just passing the new revid (which I already get from ApiEdit) to ApiParse [20:25:25] You said that should work, right? [20:25:26] heh, I suppose newFromTitle() could do the fallback too, but that would only help with new pages, not edits [20:25:32] * burritocat is just wondering random stuff [20:25:37] (03CR) 10BryanDavis: "The ::mwv Puppet class is just a shell for now but there are probably things in ::mediawiki and our site.pp that could be moved there." [vagrant] - 10https://gerrit.wikimedia.org/r/202906 (owner: 10BryanDavis) [20:25:49] (03Merged) 10jenkins-bot: Set a query group on SpecialRecentChanges query [core] - 10https://gerrit.wikimedia.org/r/202632 (owner: 10Aaron Schulz) [20:25:52] (03Merged) 10jenkins-bot: Partially revert increased wikitable padding [core] (REL1_25) - 10https://gerrit.wikimedia.org/r/202907 (https://phabricator.wikimedia.org/T91890) (owner: 10Bartosz Dziewoński) [20:26:26] (03PS3) 10Umherirrender: Add array type hints to minor methods in the Html class [core] - 10https://gerrit.wikimedia.org/r/199849 (owner: 10Thiemo Mättig (WMDE)) [20:26:33] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2: Mobile menu improvements (alpha) - Remove ® from wordmark - https://phabricator.wikimedia.org/T95007#1191650 (10kaldari) a:3kaldari [20:26:33] (03CR) 10Umherirrender: [C: 032] Add array type hints to minor methods in the Html class [core] - 10https://gerrit.wikimedia.org/r/199849 (owner: 10Thiemo Mättig (WMDE)) [20:26:44] 10MediaWiki-extensions-Graph, 6Services, 5Patch-For-Review: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1191651 (10Yurik) I have fixed most of the issues, but I am hesitant to migrate to HTTPError just yet - it provides a number of good features, but it also misses things I am using, like... [20:27:42] 10MediaWiki-Categories, 10MediaWiki-RfCs, 10MediaWiki-Watchlist: Notify on additions/removals in watched categories - https://phabricator.wikimedia.org/T94414#1191659 (10daniel) Scheduling this for an IRC meeting on April 15., 23:00 CEST. [20:28:06] (03PS2) 10BryanDavis: Use default `git::clone` permissions for Parsoid [vagrant] - 10https://gerrit.wikimedia.org/r/202827 (owner: 10Dduvall) [20:28:09] RoanKattouw: looks like that triggers << $rev = Revision::newFromId( $oldid ); >> so it should work [20:28:12] (03CR) 10BryanDavis: [C: 032] Use default `git::clone` permissions for Parsoid [vagrant] - 10https://gerrit.wikimedia.org/r/202827 (owner: 10Dduvall) [20:29:07] Excellent [20:30:11] (03Merged) 10jenkins-bot: Use default `git::clone` permissions for Parsoid [vagrant] - 10https://gerrit.wikimedia.org/r/202827 (owner: 10Dduvall) [20:30:15] ottomata: yup, that should be doable [20:32:35] 10MediaWiki-RfCs, 10Wikimedia-General-or-Unknown: cleaner URLs for editing, previewing skins, etc. - https://phabricator.wikimedia.org/T14619#1191665 (10Spage) [20:34:14] RoanKattouw: so, ApiParse doesn't really check $page when $oldid is given, but I guess that doesn't matter if it's fast enough [20:35:59] since it's actually the same request, the revision would have the right rev_page...I wonder what happens if the revision is for a new page though [20:36:26] Revision::getTitle() just uses DB_SLAVE, which the API uses << $titleObj = $rev->getTitle(); >> [20:37:40] 10MediaWiki-extensions-Graph, 6Services, 5Patch-For-Review: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1191668 (10mobrovac) >>! In T95417#1191651, @Yurik wrote: > I have fixed most of the issues, but I am hesitant to migrate to HTTPError just yet - it provides a number of good features,... [20:39:56] (03CR) 10Dduvall: [C: 032] Attach directory hiera config to a Puppet class [vagrant] - 10https://gerrit.wikimedia.org/r/202906 (owner: 10BryanDavis) [20:40:37] (03Merged) 10jenkins-bot: Add array type hints to minor methods in the Html class [core] - 10https://gerrit.wikimedia.org/r/199849 (owner: 10Thiemo Mättig (WMDE)) [20:41:10] <<$newRevId = $title->getLatestRevId();>> [20:41:17] RoanKattouw: $title looks undefined too [20:41:31] (03Merged) 10jenkins-bot: Attach directory hiera config to a Puppet class [vagrant] - 10https://gerrit.wikimedia.org/r/202906 (owner: 10BryanDavis) [20:42:08] (03PS3) 10Mobrovac: Update run-git-update for new /vagrant/srv directory [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [20:42:20] 10MediaWiki-extensions-ContentTranslation, 6Collaboration-Team, 10Flow, 10VisualEditor, and 2 others: HTML entities in CommentNode must be manually escaped/unescaped - https://phabricator.wikimedia.org/T95040#1191673 (10cscott) T95039 is now deployed in Parsoid. If a few encoded characters in comments suf... [20:42:25] (03CR) 10Mobrovac: [C: 032] Update run-git-update for new /vagrant/srv directory [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [20:42:43] 10MediaWiki-extensions-Graph, 6Services, 5Patch-For-Review: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1191674 (10Yurik) I don't think bad user input should be logged as an error, but http code should be set to 4xx. Error is a bug in the service. As for endTiming - a bit too verbose, fo... [20:43:45] (03Merged) 10jenkins-bot: Update run-git-update for new /vagrant/srv directory [vagrant] - 10https://gerrit.wikimedia.org/r/202639 (owner: 10BryanDavis) [20:46:40] RoanKattouw: I guess I can fix the getTitle part in core [20:46:58] burritocat: Oh crap this code still uses $page for a Title [20:47:02] I cleaned that up in the other file but not in this one [20:49:08] burritocat: So you're saying that if it's a page creation, then things won't magically work the way they currently do for existing pages? [20:49:17] burritocat: (Fixed the $title -> $page thing in PS2) [20:49:49] 3Mobile-App-Sprint-54-iOS, 5Patch-For-Review: Add crash reporting to alpha channel - https://phabricator.wikimedia.org/T95318#1191681 (10Fjalapeno) [20:50:47] RoanKattouw: actually, nevermind that [20:51:02] newFromConds() loads with a JOIN on page and sets all the mTitle data [20:51:07] so it would all come from the master [20:51:12] OK cool [20:51:43] mTitle only lazy loads if you make the revision with an array or DIY select [20:52:51] 10MediaWiki-extensions-DonationInterface, 10Fundraising Tech Backlog: Failed iframe liberation can make it impossible to retry a donation - https://phabricator.wikimedia.org/T95475#1191695 (10awight) 3NEW [20:55:31] 10MediaWiki-extensions-Graph, 6Services, 5Patch-For-Review: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1191713 (10Yurik) Btw, re default protocol - the 'url' lib also treats the column as part of the protocol: ``` > require('url').parse('http://example.org').protocol 'http:' ``` [20:58:53] (03CR) 10EBernhardson: [C: 031] "I can agree with those reasons, this is prudent and less likely to cause issues." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202339 (https://phabricator.wikimedia.org/T90681) (owner: 10Matthias Mullie) [20:59:32] (03CR) 10Umherirrender: Display warning message in Special:ListFiles when invalid username entered (031 comment) [core] - 10https://gerrit.wikimedia.org/r/197923 (https://phabricator.wikimedia.org/T72784) (owner: 10Tinaj1234) [21:01:59] 3Team-Practices-This-Week: Respond to 'add support for task types' - https://phabricator.wikimedia.org/T93653#1191734 (10Awjrichards) 5Open>3Resolved [21:02:49] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 6Services: Improve Graphoid code - https://phabricator.wikimedia.org/T93881#1191740 (10mobrovac) [21:02:50] 10MediaWiki-extensions-Graph, 6Services, 5Patch-For-Review: Graphoid Code Review - https://phabricator.wikimedia.org/T95417#1191738 (10mobrovac) 5Open>3Resolved >>! In T95417#1191674, @Yurik wrote: > Not sure how to approach it yet, lets move it to a separate non-blocking discussion/service improvement.... [21:06:27] 10Co-op: HostBot matches learner to two mentors in quick succession - https://phabricator.wikimedia.org/T95477#1191754 (10I_JethroBT) 3NEW a:3Capt_Swing [21:07:10] 3Team-Practices-This-Week: Schedule Q3 health check followup meetings - https://phabricator.wikimedia.org/T91714#1191763 (10Awjrichards) 5Open>3Resolved [21:07:25] (03PS2) 10Manybubbles: Compress Entity and EntityData into WikibaseUris [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 [21:07:33] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, and 2 others: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191764 (10Yurik) @csteipp, addressed the graphoid specific issues - https by default, extra check for protocol, added Content Dispos... [21:07:40] 3Team-Practices-This-Week: Create initial Recommended reading/Team Practices bookshelf - https://phabricator.wikimedia.org/T91662#1191767 (10Awjrichards) 5Open>3Resolved [21:07:42] 3Team-Practices-This-Week: Draft TPG tech talk: Team Health Checks - https://phabricator.wikimedia.org/T94476#1191768 (10KLans_WMF) a:3KLans_WMF [21:07:50] 3Team-Practices-This-Week: Create definitions for remaining (4) starred terms - https://phabricator.wikimedia.org/T93654#1191769 (10Awjrichards) 5Open>3Resolved [21:10:42] 6MediaWiki-Core-Team, 10Wikimedia-General-or-Unknown, 5Patch-For-Review: "refreshLinks.php --dfn-only" can't reliably make it through a full enwiki run - https://phabricator.wikimedia.org/T38195#1191788 (10PleaseStand) The above change is in 1.25wmf24, which has been deployed to all Wikipedias. There may sti... [21:11:38] 3Team-Practices-This-Week: Document team roles - https://phabricator.wikimedia.org/T88334#1191798 (10ksmith) [21:11:57] 3Team-Practices-This-Week: Draft TPG tech talk: Team Health Checks - https://phabricator.wikimedia.org/T94476#1191799 (10KLans_WMF) @ggellerman @awjrichards @ksmith @jaufrecht please take a spin through the deck and note anything that looks like I'm NOT getting at that you think should be presented [21:12:52] 3Team-Practices-This-Week: Document team roles - https://phabricator.wikimedia.org/T88334#1191803 (10ksmith) a:5ksmith>3JAufrecht [21:15:04] (03PS1) 10Manybubbles: WIP: skip bad uris during load [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202922 [21:15:32] 10MediaWiki-ResourceLoader, 7JavaScript: Implement ability to create wiki-modules at a user level - https://phabricator.wikimedia.org/T36958#1191820 (10SamB) [21:16:00] 6MediaWiki-API-Team, 10MediaWiki-extensions-SecurePoll: Set up mini wikifarm in Labs which has SecurePoll on it - https://phabricator.wikimedia.org/T88725#1191823 (10bd808) [21:16:04] 6MediaWiki-API-Team, 10MediaWiki-Debug-Logging, 10Incident-20150205-SiteOutage, 10Wikimedia-Logstash, and 2 others: Decouple logging infrastructure failures from MediaWiki logging - https://phabricator.wikimedia.org/T88732#1191822 (10bd808) [21:16:15] 6MediaWiki-API-Team, 10Wikimedia-Wikimania-Scholarships: Operational support for Wikimania Scholarships 2015 cycle - https://phabricator.wikimedia.org/T90739#1191829 (10bd808) [21:19:52] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Platform teams - https://phabricator.wikimedia.org/T93904#1191839 (10bd808) [21:19:54] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Search team - https://phabricator.wikimedia.org/T94488#1191838 (10bd808) 5Open>3Resolved [21:20:32] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Availability team - https://phabricator.wikimedia.org/T93941#1191840 (10bd808) 5Open>3Resolved [21:20:34] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Platform teams - https://phabricator.wikimedia.org/T93904#1149339 (10bd808) [21:22:20] 3Team-Practices-This-Week: Create a document describing possible Phabricator practices - https://phabricator.wikimedia.org/T94484#1191842 (10ksmith) a:5ksmith>3ggellerman All thoughts (or edits) would be welcome, but if you can't look at anything else, please focus on the work boards section. I'm especially... [21:23:24] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, and 2 others: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191850 (10Yurik) @csteipp, I have updated http://graphoid.wmflabs.org to the above security patch, but now it seems like Content Dis... [21:24:30] 6MediaWiki-API-Team, 10MediaWiki-Authentication-and-authorization, 7Epic: Modernize MediaWiki authentication system - https://phabricator.wikimedia.org/T89459#1191855 (10bd808) [21:27:08] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, and 2 others: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191869 (10csteipp) @Yurik, if Content-Disposition gets in the way, then don't worry about setting it. [21:28:44] 10MediaWiki-extensions-Graph, 6Scrum-of-Scrums, 10Security-Reviews, 6Services, and 2 others: Perform a security review of graphoid - https://phabricator.wikimedia.org/T93126#1191878 (10Yurik) @csteipp, ok, removed [21:29:37] csteipp, want to +2 https://gerrit.wikimedia.org/r/#/c/202917/ ? [21:30:13] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Security team - https://phabricator.wikimedia.org/T95482#1191880 (10bd808) 3NEW a:3bd808 [21:31:36] 4(no projects): Mock up a variety of burnup chart options - https://phabricator.wikimedia.org/T95483#1191891 (10JAufrecht) 3NEW a:3JAufrecht [21:33:09] 3Team-Practices-This-Week: Tasks for Joel during onboarding period - https://phabricator.wikimedia.org/T94200#1191900 (10Awjrichards) 5Open>3Invalid [21:33:37] 3Team-Practices-This-Week: Tasks for Joel during onboarding period - https://phabricator.wikimedia.org/T94200#1157370 (10Awjrichards) Joel is creating a specific task for this. [21:34:42] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Security team - https://phabricator.wikimedia.org/T95482#1191914 (10bd808) [21:34:55] (03CR) 10Smalyshev: [C: 032] Compress Entity and EntityData into WikibaseUris [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 (owner: 10Manybubbles) [21:34:57] (03PS1) 10AndyRussG: WIP Send RL context to getDependencies [core] - 10https://gerrit.wikimedia.org/r/202925 [21:35:57] 10MediaWiki-RfCs: Introduce library for assertions for parameter validation and post-condition checking - https://phabricator.wikimedia.org/T91071#1191923 (10matmarex) [21:36:08] (03Merged) 10jenkins-bot: Compress Entity and EntityData into WikibaseUris [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202669 (owner: 10Manybubbles) [21:40:09] (03PS1) 10Mjbmr: Update special pages aliases for Westerm Baluchi (bgn) from translatewiki [core] - 10https://gerrit.wikimedia.org/r/202927 [21:41:04] (03CR) 10Cenarium: [C: 04-1] "I'll need a bit more time to wrap this up." [core] - 10https://gerrit.wikimedia.org/r/201905 (https://phabricator.wikimedia.org/T91535) (owner: 10Cenarium) [21:41:18] 6MediaWiki-Core-Team, 6Project-Creators, 15User-Bd808-Test: Create Security-Team project - https://phabricator.wikimedia.org/T95486#1191945 (10bd808) 3NEW a:3bd808 [21:41:27] burritocat: ARGH, so that code doesn't work and I think I've found why [21:41:31] (03CR) 10jenkins-bot: [V: 04-1] WIP Send RL context to getDependencies [core] - 10https://gerrit.wikimedia.org/r/202925 (owner: 10AndyRussG) [21:41:43] // If for some reason the "oldid" is actually the current revision, it may be cached [21:41:45] if ( $rev->isCurrent() ) { [21:41:46] // May get from/save to parser cache [21:41:48] $p_result = $this->getParsedContent( $pageObj, $popts, $pageid, isset( $prop['wikitext'] ) ); [21:41:55] (In ApiParse.php) [21:42:01] That seems wrong [21:42:37] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests, 5Patch-For-Review: Whitelisting ymt.adlibhosting.com for the GWToolset - https://phabricator.wikimedia.org/T95418#1191963 (10Multichill) Probably better to whitelist *.adlibhosting.com . They host collections of plenty of museums [21:43:34] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Security team - https://phabricator.wikimedia.org/T95482#1191970 (10greg) [21:43:36] 6MediaWiki-Core-Team, 6Project-Creators, 15User-Bd808-Test: Create Security-Team project - https://phabricator.wikimedia.org/T95486#1191968 (10greg) 5Open>3Resolved #security-team {{done}} [21:43:58] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2, 5Patch-For-Review: Mobile menu improvements (alpha) - Remove ® from wordmark - https://phabricator.wikimedia.org/T95007#1191972 (10Deskana) Did this get Legal signoff in the end? Just double-checking since that's not documented in this card. [21:44:21] 6MediaWiki-Core-Team, 15User-Bd808-Test: Setup communication infrastructure for new Platform teams - https://phabricator.wikimedia.org/T93904#1191974 (10bd808) [21:44:32] (03CR) 10Legoktm: "Can you explain your usecase? This is just going to trigger strict standards warnings for a bunch of extensions..." [core] - 10https://gerrit.wikimedia.org/r/202925 (owner: 10AndyRussG) [21:46:22] 10MediaWiki-RfCs, 6Security-Team, 7HTTPS: RFC: MediaWiki HTTPS policy - https://phabricator.wikimedia.org/T75953#1191979 (10bd808) [21:46:23] 10Security-Reviews, 6Security-Team, 10Wikidata-Query-Service: BlazeGraph Security Review - https://phabricator.wikimedia.org/T90115#1191980 (10bd808) [21:46:25] 6Security-Team: Refactor Title to make permission checking it's own class - https://phabricator.wikimedia.org/T75958#1191982 (10bd808) [21:46:27] 6Phabricator, 10Security-Reviews, 6Security-Team: Aphlict security review - https://phabricator.wikimedia.org/T1286#1191981 (10bd808) [21:47:01] legoktm: hi! thanks for the very speedy CR! :) I'll reply a bit later, meeting coming up in a shortly 8p [21:47:07] o/ alright [21:47:33] :) [21:48:36] (03PS1) 10Aaron Schulz: Moved ActiveUsers updates to recent changes jobs [core] - 10https://gerrit.wikimedia.org/r/202929 [21:49:00] RoanKattouw: yeah, that code is sloppy with rev IDs [21:50:14] 7Design, 10Possible-Tech-Projects, 3Google-Summer-of-Code-2015: An enhanced cross-wiki watchlist as an OAuth tool - https://phabricator.wikimedia.org/T92955#1191985 (10Sitic) [21:50:36] burritocat: Reading that code I can't really identify one place where it goes wrong, because it hits the ParserCache via a Content object and that seems like it should all be fine [21:50:49] burritocat: But I'm not pretty suspicious of that code, so maybe you could take a look? [21:50:56] *I'm NOW suspicious [21:51:13] 10MediaWiki-Debug-Logging, 10hardware-requests, 6operations, 5Patch-For-Review: Fluorine needs bigger disks - https://phabricator.wikimedia.org/T92417#1191989 (10bd808) [21:53:41] RoanKattouw: try changing the isCurrent() check to << if ( $rev->getId() == $pageObj->getLatest() ) { >> [21:53:54] OK.... [21:54:08] burritocat: That reads like it should be equivalent, but I guess the difference is that $pageObj->getLatest() is a cached value? [21:54:11] isCurrent() is from master, $pageObj from slave, so they will disagree on what is current/latest [21:54:28] Which will be already populated in the DerivativeRequest case [21:54:48] (from the master in this case) [21:55:05] Unfortunately I can't test this locally [21:55:22] So I'll submit that as a Gerrit change in core, and then if you merge it it'll appear in beta labs where we can test it :D [21:55:28] "Merge first, test later" :P [21:55:52] 3Team-Practices-This-Week: Formulate test cases for testing Phab sprint extenstion - https://phabricator.wikimedia.org/T95489#1192012 (10ggellerman) 3NEW a:3ggellerman [21:56:05] well even that's not a robust fix, I just wanted to see if that was enough to fix in testing [21:56:40] even with that all of the isCurrent() logic is a bit race (e.g. memcached is updated immediately but reads come from stale and snapshotted slaves) [21:57:06] you'd never really have a cache hit in your case except for a bug, right? [21:57:27] 3Team-Practices-This-Week: Test phabricator sprint extension updates - https://phabricator.wikimedia.org/T95469#1192028 (10Awjrichards) [21:57:40] 3Team-Practices-This-Week: Mock up a variety of burnup chart options - https://phabricator.wikimedia.org/T95483#1192030 (10Awjrichards) [21:59:38] (03CR) 10Smalyshev: [C: 032] "ok for now, should remove it as soon as T95461 fix lands." [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202922 (owner: 10Manybubbles) [21:59:54] 3Team-Practices-This-Week: TPG comment on "showing tags on issues" - https://phabricator.wikimedia.org/T95488#1192060 (10Awjrichards) a:3KLans_WMF [22:00:27] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests, 5Patch-For-Review: Whitelisting *.adlibhosting.com for the GWToolset - https://phabricator.wikimedia.org/T95418#1192064 (10Dereckson) [22:01:02] 3Team-Practices-This-Week: Review SoS retro notes - https://phabricator.wikimedia.org/T95492#1192068 (10ggellerman) 3NEW [22:01:29] 4(no projects): Edit content of the sidebar - https://phabricator.wikimedia.org/T95493#1192079 (10VictorPorton) 3NEW [22:01:32] 10MediaWiki-extensions-GWToolset, 6Multimedia, 10Wikimedia-Site-requests, 5Patch-For-Review: Whitelisting *.adlibhosting.com for the GWToolset - https://phabricator.wikimedia.org/T95418#1189318 (10Dereckson) >>! In T95418#1191963, @Multichill wrote: > Probably better to whitelist *.adlibhosting.com . They... [22:01:45] 3Team-Practices-This-Week: Review SoS retro notes - https://phabricator.wikimedia.org/T95492#1192087 (10ggellerman) a:3ggellerman [22:02:20] (03CR) 10Smalyshev: "needs rebase" [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202922 (owner: 10Manybubbles) [22:02:45] (03CR) 10Yurik: Add the RESTBase and Cassandra roles to MW-Vagrant (031 comment) [vagrant] - 10https://gerrit.wikimedia.org/r/195106 (https://phabricator.wikimedia.org/T91661) (owner: 10Mobrovac) [22:03:09] (03CR) 10Smalyshev: [C: 032] Handle expanded values [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202775 (owner: 10Manybubbles) [22:03:11] (03CR) 10jenkins-bot: [V: 04-1] Handle expanded values [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202775 (owner: 10Manybubbles) [22:03:13] (03CR) 10jenkins-bot: [V: 04-1] WIP: skip bad uris during load [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202922 (owner: 10Manybubbles) [22:04:55] (03PS1) 10Catrope: ApiParse: Replace isCurrent() check with getLatest() comparison [core] - 10https://gerrit.wikimedia.org/r/202931 [22:05:18] burritocat: https://gerrit.wikimedia.org/r/202931 [22:06:56] 3Team-Practices-This-Week: Propose new sequence for Team Health Check focus areas - https://phabricator.wikimedia.org/T95495#1192109 (10ksmith) 3NEW a:3ksmith [22:07:05] 10MediaWiki-General-or-Unknown: Edit content of the sidebar - https://phabricator.wikimedia.org/T95493#1192117 (10Catrope) [22:07:27] (03PS2) 10Catrope: ApiParse: Replace isCurrent() check with getLatest() comparison [core] - 10https://gerrit.wikimedia.org/r/202931 (https://phabricator.wikimedia.org/T95466) [22:07:46] RoanKattouw: I'll play around with this in vagrant [22:07:57] OK thanks [22:07:59] I can still repo with that change :/ [22:08:04] Ugh [22:08:22] I guess maybe I should bite the bullet and set up a vagrant install so I can test master/slave interactions [22:08:26] (says the JS developer :P ) [22:11:07] 6Availability-Team: Look into Maria 10 parallel-replication - https://phabricator.wikimedia.org/T85266#1192135 (10bd808) [22:12:07] (03PS2) 10Aaron Schulz: Moved ActiveUsers updates to recent changes jobs [core] - 10https://gerrit.wikimedia.org/r/202929 (https://phabricator.wikimedia.org/T92357) [22:12:18] 10MediaWiki-Search, 6Collaboration-Team, 10Collaboration-Team-Sprint-T-2015-04-08, 10Collaboration-Team-Sprint-U-2015-04-22, and 3 others: Search indexed Flow data - https://phabricator.wikimedia.org/T78789#1192142 (10DannyH) [22:12:32] 10Collaboration-Team-Sprint-T-2015-04-08, 10Collaboration-Team-Sprint-U-2015-04-22, 5Patch-For-Review: Bring Special:FlowSearch back to life - https://phabricator.wikimedia.org/T86257#1192144 (10DannyH) [22:12:38] 10UI-Standardization: Create primary button with indeterminate loader - https://phabricator.wikimedia.org/T72417#1192146 (10kaldari) In 99% of cases, all we need is an indeterminate pending state (with no checkmark stage). Once the process is complete, whatever page or dialog the button was in will normally be r... [22:12:57] 6Search-Team, 10Wikidata-Query-Service: Consume etherpad from wikidata query developer session - https://phabricator.wikimedia.org/T88341#1192147 (10bd808) [22:13:01] 6Search-Team, 10Wikidata-Query-Service: Wikidata Query: Secure BlazeGraph's SPARQL - https://phabricator.wikimedia.org/T91812#1192149 (10bd808) [22:13:05] 6Search-Team, 10Wikidata-Query-Service: Wikidata Query: Create readonly SPARQL - https://phabricator.wikimedia.org/T91817#1192151 (10bd808) [22:13:07] 6Search-Team, 10Wikidata-Query-Service: Wikidata Query: Secure BlazeGraph's SPARQL: Investigate HistoryServiceFactory - https://phabricator.wikimedia.org/T91824#1192150 (10bd808) [22:14:50] (03PS4) 10EBernhardson: Stop doing pointless Parsoid roundtrips [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202335 (https://phabricator.wikimedia.org/T90681) (owner: 10Matthias Mullie) [22:14:57] 6MediaWiki-API-Team, 10MediaWiki-Debug-Logging, 6Release-Engineering, 10Wikimedia-Logstash, and 2 others: Log php fatals with full backtraces again (fatal.log on fluorine) - https://phabricator.wikimedia.org/T89169#1192161 (10ksmith) [22:15:42] 6ArchCom, 6Availability-Team: Write design document for content/storage APIs - https://phabricator.wikimedia.org/T1124#1192162 (10RobLa-WMF) [22:17:11] 10MediaWiki-Page-editing, 6Availability-Team, 10Wikidata, 5Patch-For-Review: WikiPage: "Could not find text for current revision" - https://phabricator.wikimedia.org/T93976#1192173 (10ksmith) [22:18:38] (03PS2) 10EBernhardson: Get rid of redundant use's [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202336 (owner: 10Matthias Mullie) [22:18:40] 10MediaWiki-Extension-Requests, 6Community-Liaison, 6Engineering-Community, 10Wikimania-Hackathon-2015: System for reporting and flagging inappropriate user behavior - https://phabricator.wikimedia.org/T90312#1192187 (10Elena) [22:19:10] (03PS2) 10EBernhardson: Replace API param 'contentFormat' by 'format' [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202337 (owner: 10Matthias Mullie) [22:19:23] 10MediaWiki-Maintenance-scripts, 10MediaWiki-Redirects, 6Release-Engineering, 5Patch-For-Review: namespaceDupes not handling deleted namespace redirects as desired - https://phabricator.wikimedia.org/T91401#1192194 (10He7d3r) Any progress since last week? This regression is still causing confusion: https:/... [22:19:30] (03PS4) 10EBernhardson: Get rid of $requiresWikitext [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202338 (owner: 10Matthias Mullie) [22:19:50] 10MediaWiki-Debug-Logging, 6Release-Engineering, 6Security-Team, 6operations, 5Patch-For-Review: Store unsampled API and XFF logs - https://phabricator.wikimedia.org/T88393#1192197 (10RobLa-WMF) [22:20:05] thedj: I got a positive code review from manybubbles. Do you think https://gerrit.wikimedia.org/r/#/c/197737/ can be merged? [22:22:01] (03PS4) 10EBernhardson: Add content type: fixed-html [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202339 (https://phabricator.wikimedia.org/T90681) (owner: 10Matthias Mullie) [22:22:34] RoanKattouw: ok, that fix actually does work, I just was an idiot and forgot to git-pull the VE change [22:23:03] it at least needs some comment explaining the change though [22:25:03] 10MediaWiki-extensions-Gadgets, 7Crosswiki, 7Tracking: Implement Gadgets 2.0 (tracking) - https://phabricator.wikimedia.org/T31272#1192229 (10He7d3r) [22:25:45] 10MediaWiki-Authentication-and-authorization, 10MediaWiki-extensions-CentralAuth: UserLoadFromSession considered evil - https://phabricator.wikimedia.org/T43201#1192241 (10RobLa-WMF) [22:25:55] 10MediaWiki-Search, 6Collaboration-Team, 10Collaboration-Team-Sprint-U-2015-04-22, 10Flow, and 2 others: U9. Search indexed Flow data - https://phabricator.wikimedia.org/T78789#1192243 (10DannyH) [22:26:06] 10Collaboration-Team-Sprint-U-2015-04-22, 5Patch-For-Review: U11. Bring Special:FlowSearch back to life - https://phabricator.wikimedia.org/T86257#1192246 (10DannyH) [22:26:38] 10MediaWiki-Interface, 6Mobile-Web, 10Parsoid, 7Beta-Feature, and 2 others: Switch MobileFrontend over to using Parsoid for its read HTML - https://phabricator.wikimedia.org/T76970#1192248 (10bd808) [22:27:20] (03CR) 10Aaron Schulz: ApiParse: Replace isCurrent() check with getLatest() comparison (031 comment) [core] - 10https://gerrit.wikimedia.org/r/202931 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [22:28:18] 10MediaWiki-ResourceLoader, 7JavaScript: Implement ability to create wiki-modules at a user level - https://phabricator.wikimedia.org/T36958#1192252 (10He7d3r) [22:29:37] (03CR) 10Aaron Schulz: [C: 032] Fixed broken batching in –PurgeOldLogIPData [extensions/AbuseFilter] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202792 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [22:31:05] (03Merged) 10jenkins-bot: Fixed broken batching in –PurgeOldLogIPData [extensions/AbuseFilter] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202792 (https://phabricator.wikimedia.org/T95382) (owner: 10Aaron Schulz) [22:32:16] 7Performance: Fix profiling endpoints and format - https://phabricator.wikimedia.org/T90623#1192272 (10bd808) [22:32:21] 6Security, 10Security-Reviews: Security review of community extensions: Extension:AtomExporter, Extension:DownloadCounter, Extension:PasswordProtected - https://phabricator.wikimedia.org/T787#1192273 (10csteipp) [22:32:44] 7HHVM, 7Upstream: HHVM with FastCGI does not support streaming output - https://phabricator.wikimedia.org/T91468#1192274 (10bd808) [22:32:51] burritocat: OK will add [22:33:27] 10Wikimedia-General-or-Unknown, 5Patch-For-Review: Existed pages without ability to reach and obviously wrong namespace - https://phabricator.wikimedia.org/T87645#1192279 (10bd808) [22:34:00] (03PS3) 10Catrope: ApiParse: Replace isCurrent() check with getLatest() comparison [core] - 10https://gerrit.wikimedia.org/r/202931 [22:34:19] (03PS4) 10Catrope: ApiParse: Replace isCurrent() check with getLatest() comparison [core] - 10https://gerrit.wikimedia.org/r/202931 (https://phabricator.wikimedia.org/T95466) [22:34:35] burritocat: OK I've updated the patch. If it gets merged soon I can SWAT it at 4pm :) [22:36:18] (03CR) 10Aaron Schulz: [C: 032] ApiParse: Replace isCurrent() check with getLatest() comparison [core] - 10https://gerrit.wikimedia.org/r/202931 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [22:38:59] 10Wikimedia-Site-requests, 5Patch-For-Review: Additional page protection level on dewiki for "editor" - https://phabricator.wikimedia.org/T94368#1192301 (10Luke081515) OK, whats the next step now? [22:40:50] (03PS1) 10Aaron Schulz: Removed unusable maintenance script [core] - 10https://gerrit.wikimedia.org/r/202937 [22:43:58] (03PS2) 10Manybubbles: WIP: skip bad uris during load [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202922 [22:44:00] (03CR) 10jenkins-bot: [V: 04-1] WIP: skip bad uris during load [wikidata/query/rdf] - 10https://gerrit.wikimedia.org/r/202922 (owner: 10Manybubbles) [22:44:04] (03CR) 10Legoktm: [C: 032] Removed unusable maintenance script [core] - 10https://gerrit.wikimedia.org/r/202937 (owner: 10Aaron Schulz) [22:44:19] 10Wikimedia-General-or-Unknown, 5Patch-For-Review: "refreshLinks.php --dfn-only" can't reliably make it through a full enwiki run - https://phabricator.wikimedia.org/T38195#1192305 (10bd808) [22:44:22] 10Wikimedia-Site-requests, 5Patch-For-Review: InitializeSettings: decide upon 'wiki' vs. 'wikipedia' group usage - https://phabricator.wikimedia.org/T91340#1192309 (10bd808) [22:44:25] 10MediaWiki-JobQueue, 6operations, 7Monitoring: Establish monitoring thresholds for job queue - https://phabricator.wikimedia.org/T79687#1192310 (10bd808) [22:44:30] 10MediaWiki-Debug-Logging, 10MediaWiki-General-or-Unknown, 6Release-Engineering, 5MW-1.23-release, 15User-Bd808-Test: Create a minimal backport of PSR-3 logging to MediaWiki 1.23 LTS - https://phabricator.wikimedia.org/T91653#1192311 (10bd808) [22:44:46] * bd808 killed wikibugs [22:45:03] 10MediaWiki-Parser, 6Parsoid-Team: Mechanism to find usages of raw-html messages - https://phabricator.wikimedia.org/T21291#1192344 (10bd808) [22:45:06] 10MediaWiki-Internationalization, 7I18n, 7Performance: Add a {{USERLANGUAGE}}/{{USERLANG}} magic word - https://phabricator.wikimedia.org/T4085#1192345 (10bd808) [22:45:10] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2, 5Patch-For-Review: Mobile menu improvements (alpha) - Remove ® from wordmark - https://phabricator.wikimedia.org/T95007#1192354 (10kaldari) No, still waiting on sign-off from legal. [22:46:46] (03CR) 10Physikerwelt: "@thedj: Do you think this patch can be merged now?" [extensions/Math] - 10https://gerrit.wikimedia.org/r/197737 (https://phabricator.wikimedia.org/T93075) (owner: 10Physikerwelt) [22:47:10] (03PS1) 10Aaron Schulz: Warn when jobs do large DB writes at once [core] - 10https://gerrit.wikimedia.org/r/202939 [22:47:13] legoktm: ^ [22:48:18] burritocat: is it ok that that is before commitMasterChanges ? [22:48:33] Thanks for merging burritocat [22:48:47] Once Jenkins has finished taking its sweet time, I'll add that and my VE change to SWAT [22:48:49] legoktm: why would that matter? [22:49:01] (03Merged) 10jenkins-bot: ApiParse: Replace isCurrent() check with getLatest() comparison [core] - 10https://gerrit.wikimedia.org/r/202931 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [22:49:10] * burritocat waited an hour for a config change to merge earlier [22:49:22] 10MediaWiki-Internationalization, 7I18n, 7Performance: Add a {{USERLANGUAGE}}/{{USERLANG}} magic word - https://phabricator.wikimedia.org/T4085#1192371 (10He7d3r) [22:49:22] went through the whole metrics meeting [22:49:39] Oh was translate bot holding it up? [22:49:46] (03PS1) 10Catrope: ApiParse: Replace isCurrent() check with getLatest() comparison [core] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202940 (https://phabricator.wikimedia.org/T95466) [22:49:59] (03PS1) 10Catrope: ApiParse: Replace isCurrent() check with getLatest() comparison [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202941 (https://phabricator.wikimedia.org/T95466) [22:50:03] burritocat: couldn't that potentially run callbacks? or am I thinking of something else? [22:50:11] (03CR) 10Catrope: [C: 032] ApiParse: Replace isCurrent() check with getLatest() comparison [core] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202940 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [22:50:15] (03CR) 10Catrope: [C: 032] ApiParse: Replace isCurrent() check with getLatest() comparison [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202941 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [22:50:27] RoanKattouw: just jenkins+zuul were totally overloaded [22:50:46] zuul decided to stop running jobs on precise machines [22:51:00] legoktm: in theory, though nothing happens before then so it wont...even if it did, I'd still want to know :) [22:51:30] ok :P [22:51:55] (03CR) 10Legoktm: [C: 032] Warn when jobs do large DB writes at once [core] - 10https://gerrit.wikimedia.org/r/202939 (owner: 10Aaron Schulz) [22:53:32] (03Merged) 10jenkins-bot: Removed unusable maintenance script [core] - 10https://gerrit.wikimedia.org/r/202937 (owner: 10Aaron Schulz) [22:54:12] 6Availability-Team: Fix causes of slave lag and get it to less than ~5 seconds at peak - https://phabricator.wikimedia.org/T95501#1192404 (10aaron) 3NEW [22:54:32] 6Availability-Team: Fix causes of slave lag and get it to less than ~5 seconds at peak - https://phabricator.wikimedia.org/T95501#1192412 (10aaron) [22:55:14] 6Availability-Team: Fix causes of slave lag and get it to less than ~5 seconds at peak - https://phabricator.wikimedia.org/T95501#1192416 (10aaron) a:3aaron [22:55:32] 10Wikimedia-Site-requests, 5Patch-For-Review: Additional page protection level on dewiki for "editor" - https://phabricator.wikimedia.org/T94368#1192417 (10Se4598) >>! In T94368#1192301, @Luke081515 wrote: > OK, whats the next step now? still waiting on review/approve of https://gerrit.wikimedia.org/r/201940 [22:55:43] 6Availability-Team, 7Epic: MediaWiki multi-datacenter investigation and work - https://phabricator.wikimedia.org/T88445#1192419 (10aaron) [22:55:44] 6Availability-Team, 5Patch-For-Review: Make RedisBagOStuff support tag name hashing - https://phabricator.wikimedia.org/T95126#1192418 (10aaron) 5Open>3Resolved [22:56:10] (03PS1) 10Legoktm: Removed unusable maintenance script [core] (REL1_25) - 10https://gerrit.wikimedia.org/r/202946 [22:56:19] (03CR) 10Legoktm: [C: 032] Removed unusable maintenance script [core] (REL1_25) - 10https://gerrit.wikimedia.org/r/202946 (owner: 10Legoktm) [22:56:20] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2, 5Patch-For-Review: Mobile menu improvements (alpha) - Remove ® from wordmark - https://phabricator.wikimedia.org/T95007#1192421 (10Slaporte) >>! In T95007#1191972, @Deskana wrote: > Did this get Legal signoff in the end? Just double-checking since that's not docum... [23:01:11] 10MediaWiki-General-or-Unknown, 10MediaWiki-Recent-changes, 6Collaboration-Team, 10Flow, 7Technical-Debt: Use rc_source and drop RC_TYPE - https://phabricator.wikimedia.org/T74157#1192430 (10bd808) [23:02:36] (03Merged) 10jenkins-bot: ApiParse: Replace isCurrent() check with getLatest() comparison [core] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202940 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [23:08:24] (03Merged) 10jenkins-bot: ApiParse: Replace isCurrent() check with getLatest() comparison [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202941 (https://phabricator.wikimedia.org/T95466) (owner: 10Catrope) [23:08:28] (03Merged) 10jenkins-bot: Warn when jobs do large DB writes at once [core] - 10https://gerrit.wikimedia.org/r/202939 (owner: 10Aaron Schulz) [23:13:21] (03PS7) 10Jforrester: Update inspector now it inherits from NodeInspector [extensions/Math] - 10https://gerrit.wikimedia.org/r/198731 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:13:27] (03PS3) 10Jforrester: Remove paragraph unwrapping, moved to parent class. [extensions/Math] - 10https://gerrit.wikimedia.org/r/198741 (owner: 10Esanders) [23:13:35] (03CR) 10Catrope: [C: 032] Update inspector now it inherits from NodeInspector [extensions/Math] - 10https://gerrit.wikimedia.org/r/198731 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:13:41] (03CR) 10Catrope: [C: 032] Update inspector now it inherits from NodeInspector [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/198733 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:13:44] (03PS4) 10Jforrester: Remove paragraph unwrapping, moved to parent class [extensions/Math] - 10https://gerrit.wikimedia.org/r/198741 (owner: 10Esanders) [23:13:55] (03PS4) 10Jforrester: Update inspector now it inherits from NodeInspector [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/198733 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:14:52] (03PS3) 10Aaron Schulz: Moved ActiveUsers updates to recent changes jobs [core] - 10https://gerrit.wikimedia.org/r/202929 (https://phabricator.wikimedia.org/T95501) [23:15:00] 10SUL-Finalization, 7Epic, 3Roadmap, 7Tracking, and 2 others: Complete unification of all accounts to SUL - https://phabricator.wikimedia.org/T37707#1192458 (10Keegan) >>! In T37707#1179841, @Nemo_bis wrote: > IIRC, other than T73241, we have not sent any real email notification to the accounts in question... [23:15:52] (03Merged) 10jenkins-bot: Removed unusable maintenance script [core] (REL1_25) - 10https://gerrit.wikimedia.org/r/202946 (owner: 10Legoktm) [23:16:27] (03Merged) 10jenkins-bot: Update inspector now it inherits from NodeInspector [extensions/Math] - 10https://gerrit.wikimedia.org/r/198731 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:17:18] (03CR) 10jenkins-bot: [V: 04-1] Update inspector now it inherits from NodeInspector [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/198733 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:20:27] Uhm, wat, did the wikihiero parser tests break? [23:20:38] (03CR) 10Catrope: [C: 032] "Trying again?" [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/198733 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:20:42] 10MediaWiki-extensions-TitleBlacklist: Make the title blacklist allow auto creations for global accounts - https://phabricator.wikimedia.org/T35429#1192488 (10Legoktm) [23:20:43] 10Wikimania-Hackathon-2015, 10Wikimedia-General-or-Unknown, 7Tracking: Steward, global sysop and SWMT tasks bugs (tracking) - https://phabricator.wikimedia.org/T43492#1192487 (10Legoktm) [23:20:45] 10MediaWiki-extensions-TitleBlacklist, 10SUL-Finalization, 10Wikimedia-Site-requests, 5Patch-For-Review, 7user-notice: User name blacklist should be managed globally to avoid SUL fragmentation - https://phabricator.wikimedia.org/T38939#1192485 (10Legoktm) 5Open>3Resolved This is now deployed. [23:21:55] 10CirrusSearch: Bump master Cirrus version to 0.3 - https://phabricator.wikimedia.org/T965#1192493 (10bd808) [23:21:56] 10CirrusSearch: Figure out why there are often so many mgwiktionary CirrusLinks jobs - https://phabricator.wikimedia.org/T93219#1192492 (10bd808) [23:21:58] 10CirrusSearch: Think about whether we should do prefix searching on any word everywhere - https://phabricator.wikimedia.org/T1004#1192495 (10bd808) [23:22:00] 10CirrusSearch: Improve searches on poor spellings - https://phabricator.wikimedia.org/T1240#1192494 (10bd808) [23:23:45] 10MediaWiki-Authentication-and-authorization, 10MediaWiki-extensions-OAuth, 7Epic: General authentication improvements for MediaWiki - https://phabricator.wikimedia.org/T90925#1192504 (10bd808) [23:23:53] (03PS1) 10Catrope: Bump VisualEditor for I2149c7a710 [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202953 [23:24:07] (03CR) 10Catrope: [C: 032] Bump VisualEditor for I2149c7a710 [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202953 (owner: 10Catrope) [23:24:22] (03CR) 10jenkins-bot: [V: 04-1] Update inspector now it inherits from NodeInspector [extensions/wikihiero] - 10https://gerrit.wikimedia.org/r/198733 (https://phabricator.wikimedia.org/T93592) (owner: 10Esanders) [23:24:42] (03PS1) 10Catrope: Bump VisualEditor for I2149c7a710 [core] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202954 [23:24:52] (03CR) 10Catrope: [C: 032] Bump VisualEditor for I2149c7a710 [core] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202954 (owner: 10Catrope) [23:25:12] 10MediaWiki-extensions-LocalisationUpdate, 10Deployment-Systems, 7Epic, 7I18n: Localization Cache Redo - https://phabricator.wikimedia.org/T78802#1192507 (10bd808) [23:26:43] 10Wikidata-Query-Service: Investigate Apache Jena for WDQ - https://phabricator.wikimedia.org/T90112#1192510 (10bd808) [23:27:08] 10Wikimedia-Site-requests: Liangent's user page and user subpages on zhwiki - https://phabricator.wikimedia.org/T95408#1192513 (10Vogone) >>! In T95408#1189527, @Legoktm wrote: > I suppose we need a javascript confirm button on the rename queue like Special:GlobalRenameUser has? Quite sad we would need this. Bu... [23:27:38] 10Hovercards: Hovercards: Contact the 3 trial Wikipedias around Monday April 13 with the translated messaging - https://phabricator.wikimedia.org/T95503#1192520 (10Quiddity) 3NEW a:3Quiddity [23:27:54] 6MediaWiki-API-Team, 10MediaWiki-JobQueue, 6Availability-Team: Create separate job loop for LocalRenameUserJob - https://phabricator.wikimedia.org/T87397#1192528 (10bd808) [23:29:15] 10MediaWiki-Email, 10MediaWiki-JobQueue: Use message IDs for email jobs - https://phabricator.wikimedia.org/T89285#1192530 (10bd808) [23:31:07] 10MediaWiki-Search, 6WMF-Design: Interwiki search needs design input - https://phabricator.wikimedia.org/T1224#1192532 (10bd808) [23:31:52] 3Mobile-Web-Sprint-44-Road-Warrior:-Mad-Max-2, 5Patch-For-Review: Mobile menu improvements (alpha) - Remove ® from wordmark - https://phabricator.wikimedia.org/T95007#1192533 (10kaldari) Looks like legal isn't budging. The best we can get is removing the ® from the header, but not the footer, which would requi... [23:32:49] 6MediaWiki-Core-Team: virt1000 DB connection error spam - https://phabricator.wikimedia.org/T78487#1192536 (10bd808) 5Open>3declined a:3bd808 wikitech moved off of virt1000. If it's still a problem it will show up in the log file triage. [23:33:57] (03CR) 10Anomie: Add X-Analytics header for API requests too (031 comment) [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202800 (https://phabricator.wikimedia.org/T92875) (owner: 10Legoktm) [23:35:00] 5Patch-For-Review: U11. Bring Special:FlowSearch back to life - https://phabricator.wikimedia.org/T86257#1192542 (10DannyH) [23:35:43] 6Phabricator: make gerritbot remove the "patch-for-review" tag once a patch is merged (or abandoned) - https://phabricator.wikimedia.org/T95309#1192544 (10MZMcBride) Yeah. I'm not exactly opposed to improving gerritbot (Gerrit is still in production and it manages the repos of record, after all), I'm just wary o... [23:36:13] 5Patch-For-Review: U11. Bring Special:FlowSearch back to life - https://phabricator.wikimedia.org/T86257#964617 (10DannyH) [23:36:57] (03Merged) 10jenkins-bot: Bump VisualEditor for I2149c7a710 [core] (wmf/1.25wmf24) - 10https://gerrit.wikimedia.org/r/202953 (owner: 10Catrope) [23:38:56] (03Merged) 10jenkins-bot: Bump VisualEditor for I2149c7a710 [core] (wmf/1.26wmf1) - 10https://gerrit.wikimedia.org/r/202954 (owner: 10Catrope) [23:39:06] 10MediaWiki-Parser, 6Parsoid-Team: Implement file usage limits in parsing - https://phabricator.wikimedia.org/T75960#1192553 (10bd808) [23:40:38] 10MediaWiki-Email, 10MediaWiki-Recent-changes: UserMailer.php uncaught Exception PEAR mail package is not installed - https://phabricator.wikimedia.org/T95047#1192558 (10bd808) [23:42:07] (03CR) 10Legoktm: Add X-Analytics header for API requests too (031 comment) [extensions/XAnalytics] - 10https://gerrit.wikimedia.org/r/202800 (https://phabricator.wikimedia.org/T92875) (owner: 10Legoktm) [23:42:09] 10MediaWiki-General-or-Unknown, 7Varnish: Separate Cache-Control header for proxy and client - https://phabricator.wikimedia.org/T50835#1192566 (10bd808) [23:42:21] 6Phabricator: Create a phabricator api method to remove 'patch-for-review' tag from a task - https://phabricator.wikimedia.org/T95307#1192569 (10MZMcBride) Where's the Git repo for this again? This task sounds relatively straightforward (#easy maybe), but it's mostly a matter of finding the current code and taki... [23:42:36] (03CR) 10EBernhardson: [C: 031] Stop doing pointless Parsoid roundtrips [extensions/Flow] - 10https://gerrit.wikimedia.org/r/202335 (https://phabricator.wikimedia.org/T90681) (owner: 10Matthias Mullie) [23:47:07] 6MediaWiki-Core-Team, 10Wikimedia-Logstash: Ignore apache2 log2udp messages and transition rules to the syslog stream of the same events - https://phabricator.wikimedia.org/T76760#1192594 (10bd808) 5Open>3Resolved a:3bd808 {{done}} a long time ago via [23:49:37] 6MediaWiki-Core-Team: CSS validator survey and design work - https://phabricator.wikimedia.org/T989#1192610 (10bd808) [23:49:39] 10MediaWiki-RfCs, 6ArchCom: RfC: Allow styling in templates - https://phabricator.wikimedia.org/T483#1192609 (10bd808) [23:50:00] 7Performance: Enable ProfilerSectionOnly - https://phabricator.wikimedia.org/T86215#1192612 (10bd808) [23:52:33] 10MediaWiki-RfCs: MediaWiki RFC: Exposure of user IP addresses - https://phabricator.wikimedia.org/T95144#1192622 (10MZMcBride) That's probably fine. I only filed a task because I felt pressured to by , but really the wiki page is where the meat is here. This issue keeps comin... [23:52:40] James_F, could I bother you with another oojs Q? [23:52:59] halfak: Sorry, in a meeting right now – in a bit? [23:53:06] Sure I can wait. Thanks. [23:54:00] 6Phabricator: Create a phabricator api method to remove 'patch-for-review' tag from a task - https://phabricator.wikimedia.org/T95307#1192627 (10MZMcBride) [23:57:00] 10MediaWiki-RfCs: CSS validator survey and design work - https://phabricator.wikimedia.org/T989#1192635 (10bd808)