[00:00:05] Krinkle: whenever Zuul does a query to Jenkins it stop all processing, that is why we had no job being triggered [00:00:20] like: Zuul does the query, jenkins takes 4 minutes to reply, Zuul does nothing meanwhile. [00:00:31] which was considered not needed for me, eventhough I request someone to do something at least twice a week and you use it frequently, too. [00:00:31] ahh [00:00:44] perhaps we should reconsider? [00:00:56] at least until our ci infrastructure is stable, at which point we both don't need it. [00:00:58] right? [00:01:27] we had that discussion within the mediawiki/core team [00:01:47] basically root is not needed, we should identify what are the needs and get a workaround [00:02:05] well, we know that doesn't work in practice because our infrastructure isn't stable yet. [00:02:06] like to get you access to apache logs, we could change the files ownership [00:02:18] right [00:02:22] like making /var/log/apache readable by wikidev group [00:02:36] what else do we/you use root for? [00:02:54] jstack to dump threads, strace and tcpdump [00:03:18] don't those run under jenkins/ [00:04:15] New patchset: Ryan Lane; "mediawiki.htmlform: Set minimum size for field width" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62100 [00:04:16] strace you do not [00:04:34] even has jenkins, you can't attach another jenkins process due to some kernel protection [00:04:47] but the apache logs could be granted no problem. [00:05:06] in contint module, you could get a puppet change that makes /var/log/apache2 wikidev owned [00:05:25] all files there are world readable beside the error.log [00:06:49] New patchset: Catrope; "Allow iiurlheight to be used without iiurlwidth" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60450 [00:08:39] Change merged: jenkins-bot; [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62085 [00:09:18] Change merged: jenkins-bot; [mediawiki/extensions/ConfirmEdit] (master) - https://gerrit.wikimedia.org/r/61161 [00:12:18] New patchset: Alex Monk; "Move overlay module to top queue" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62118 [00:22:32] New review: Alex Monk; "Needs rebase" [mediawiki/extensions/Echo] (master) C: -1; - https://gerrit.wikimedia.org/r/61819 [00:26:00] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60254 [00:26:51] New patchset: Adamw; "Banner preview is done in an iframe" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [00:26:56] New review: Adamw; "PS 2: now with the missing file" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [00:34:18] New patchset: Catrope; "Allow iiurlheight to be used without iiurlwidth" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60450 [00:34:34] New patchset: MarkTraceur; "Fix the Parsoid image tests" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62121 [00:34:42] gwicke_away: ^^ [00:35:14] New patchset: Adamw; "Banner preview is done in an iframe" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [00:37:26] New patchset: Bsitu; "(bug 47910) Minor edit should not trigger talk page notif" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [00:37:57] New patchset: Kaldari; "Temporary fix for bug 47954 - keep section link in edit summary" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62122 [00:39:04] New patchset: Lwelling; "Cache result of Language::isValidCode() to avoid regex processing" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [00:40:37] New review: preilly; "Yeah that's a really simple approach to improving the performance of this function. Nice work Luke. :-)" [mediawiki/core] (master) C: 1; - https://gerrit.wikimedia.org/r/62123 [00:41:53] New patchset: Lwelling; "Cache result of Language::isValidCode() to avoid regex processing" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [00:46:13] New patchset: Krinkle; "Add favicon.php" [integration/docroot] (master) - https://gerrit.wikimedia.org/r/62124 [00:46:35] Change merged: Asher; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [00:49:04] New review: Hashar; "I cant believe we never came across that one before. Nice catch." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [00:49:25] Change merged: Krinkle; [integration/docroot] (master) - https://gerrit.wikimedia.org/r/62124 [00:55:41] New patchset: Alex Monk; "Add hook so extensions can say whether or not the user has new messages" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62126 [00:57:05] New patchset: Alex Monk; "Support api.php?action=query&meta=userinfo&uiprop=hasmsg" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62127 [01:10:23] New patchset: Mattflaschen; "Disable autocomplete for Simple, Fancy, Math, and Questy." [mediawiki/extensions/ConfirmEdit] (master) - https://gerrit.wikimedia.org/r/62128 [01:14:07] Change merged: Mwalker; [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/61493 [01:14:11] Change merged: Mwalker; [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/61497 [01:14:17] Change merged: Mwalker; [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/61923 [01:14:42] Change abandoned: Mwalker; "Replaced by something else...." [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/61492 [01:16:09] Change merged: Mwalker; [wikimedia/fundraising/crm] (master) - https://gerrit.wikimedia.org/r/62116 [01:28:09] New patchset: Adamw; "Amazon IPN listener" [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/60958 [01:33:02] New review: Lwelling; "I suspect I might like this as a permanent solution not an interim, but I like more links in the fly..." [mediawiki/extensions/Echo] (master) C: 2; - https://gerrit.wikimedia.org/r/62122 [01:33:21] Change merged: jenkins-bot; [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62122 [01:57:29] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) C: -1; - https://gerrit.wikimedia.org/r/61819 [01:58:15] Change merged: Kaldari; [mediawiki/extensions/Thanks] (master) - https://gerrit.wikimedia.org/r/61711 [01:59:02] Change merged: Kaldari; [mediawiki/extensions/GettingStarted] (master) - https://gerrit.wikimedia.org/r/61716 [02:01:48] New patchset: Jakub Vrána; "Prepare DatabaseMysql for mysqli" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [02:46:32] New patchset: Mwalker; "Banner preview is done in an iframe" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [03:05:26] New review: Mattflaschen; "https://gerrit.wikimedia.org/r/#/c/59575/ addresses the same issue in a different way (and makes a b..." [mediawiki/extensions/GettingStarted] (master) - https://gerrit.wikimedia.org/r/60787 [03:28:10] New review: Mattflaschen; "Thank you for fixing the commit message." [mediawiki/extensions/Math] (master) C: -1; - https://gerrit.wikimedia.org/r/60882 [03:48:19] New review: Rasel160; "Thanks!!" [mediawiki/extensions/UploadWizard] (master) - https://gerrit.wikimedia.org/r/61339 [03:52:41] New patchset: Mwalker; "Banner preview is done in an iframe" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [04:33:50] New review: Parent5446; "How different is this from https://gerrit.wikimedia.org/r/56193" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [04:45:10] New patchset: Mwalker; "CentralNotice UI Message Handling ++Graceful" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62133 [04:48:09] New review: Mwalker; "(1 comment)" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62133 [05:07:52] New patchset: Adamw; "CentralNotice UI Message Handling ++Graceful" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62133 [05:08:44] New review: Adamw; "cos i was like, Whaaat?" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62133 [05:12:42] New review: Jakub Vrána; "Oh, I didn't notice your diff. Mine seems to reuse more code but your diff is fine too. I wonder if ..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [05:19:15] New review: MZMcBride; "What's the status of this?" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/58686 [05:20:26] New patchset: Physikerwelt; "Introduce getter and setter for the MathRenderer" [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/60882 [05:24:33] New review: Demon; "Someone should review it and merge." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/58686 [05:25:02] rihnapstor: hello [05:27:37] hi [05:27:40] Rahul21: [05:27:49] submitted proposal [05:27:51] ? [05:29:12] New review: Adamw; "Nice improvement! I balked at the security implications, but this looks good." [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [05:42:15] New review: Ori.livneh; "I'd be happy to, but I think it'd be better to merge this together with I41b8263f1 (your follow-up c..." [mediawiki/core] (master) C: 1; - https://gerrit.wikimedia.org/r/58686 [05:43:47] New patchset: Mwalker; "Make CentralNotice device aware" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/61988 [05:44:17] New patchset: Mwalker; "Fixing some UI bugs" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62135 [05:45:18] New patchset: Mwalker; "Make CentralNotice device aware" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/61988 [05:45:23] New patchset: Mwalker; "Fixing some UI bugs" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62135 [06:16:15] New review: Liangent; "(1 comment)" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/19746 [06:16:21] New patchset: Liangent; "Partly disable title conversion if variant == main language code" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/19746 [06:31:13] New patchset: Mwalker; "Forgot to Log Device Changes..." [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62137 [06:32:54] New review: Mwalker; "(1 comment)" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [06:33:14] New patchset: Mwalker; "Forgot to Log Device Changes..." [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62137 [06:54:31] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) C: -1; - https://gerrit.wikimedia.org/r/62127 [07:13:03] New review: Umherirrender; "Only for information: There is already a hook 'UserRetrieveNewTalks' in User::getNewMessageLinks for..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62126 [07:17:02] New review: Nikerabbit; "How was it noticed that this is slow?" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [07:17:37] New patchset: Umherirrender; "Add list=allfileusages" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61817 [07:49:42] New patchset: Netbrain; "fixed issue with older php versions" [mediawiki/extensions/Spreadsheet] (master) - https://gerrit.wikimedia.org/r/62138 [07:55:42] New patchset: Ori.livneh; "Clean-up to enforce consistent Puppet code style" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62139 [07:56:06] Change merged: jenkins-bot; [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62139 [08:08:26] New patchset: Netbrain; "fixed issue with older php versions" [mediawiki/extensions/Spreadsheet] (master) - https://gerrit.wikimedia.org/r/62138 [08:09:44] Change merged: Netbrain; [mediawiki/extensions/Spreadsheet] (master) - https://gerrit.wikimedia.org/r/62138 [08:22:45] New patchset: Matmarex; "wikibits: Wrap jsMsg in mw.log.deprecate" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60268 [08:22:46] New patchset: Matmarex; "Monobook: Remove OperaNFixes.css files" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60255 [08:22:46] New patchset: Matmarex; "wikibits: Pass JSHint" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60269 [08:22:46] New patchset: Matmarex; "wikibits: Deprecate useragent and tooltip/accesskey globals" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60266 [08:22:47] New patchset: Matmarex; "wikibits: Replace checkboxShiftClick globals with dummies" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60267 [08:22:47] New patchset: Matmarex; "mediawiki.log: Add to jsduck index" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60264 [08:22:47] New patchset: Matmarex; "Implement mw.log.warn and mw.log.deprecate" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60265 [08:23:16] New review: Matmarex; "Rebased, now it merges. Please CR+2 again." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60255 [08:43:43] New patchset: Nikerabbit; "I18n for System font" [mediawiki/extensions/UniversalLanguageSelector] (master) - https://gerrit.wikimedia.org/r/62140 [08:45:36] New patchset: Pastakhov; "make compatible with PHP 5.2" [mediawiki/extensions/MyVariables] (master) - https://gerrit.wikimedia.org/r/62141 [08:46:33] Change merged: Pastakhov; [mediawiki/extensions/MyVariables] (master) - https://gerrit.wikimedia.org/r/62141 [09:42:16] New patchset: Mattflaschen; "Remove dead config variable, wgMathJaxUrl" [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/62142 [09:57:52] New review: Mattflaschen; "(1 comment)" [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62126 [10:14:19] New review: Karim.rayani; "Hey, any feedback on this feature of Download PDF," [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/61820 [10:31:02] New patchset: Oantnri; "DairikiDiff: Optimise method WordLevelDiff._split()" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61983 [10:33:34] New patchset: Oantnri; "DairikiDiff: Optimise method WordLevelDiff._split()" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61983 [11:00:23] New patchset: Netbrain; "added functionality to customize class and style attributes of the spreadsheet container." [mediawiki/extensions/Spreadsheet] (master) - https://gerrit.wikimedia.org/r/62144 [11:00:45] Change merged: Netbrain; [mediawiki/extensions/Spreadsheet] (master) - https://gerrit.wikimedia.org/r/62144 [11:40:40] Change merged: jenkins-bot; [mediawiki/extensions/UniversalLanguageSelector] (master) - https://gerrit.wikimedia.org/r/62140 [12:28:50] New patchset: Wizardist; "(Bug 47958) Make GENDER ignore everything after slash" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62146 [12:57:31] New patchset: Zfilipin; "README file for MediaWiki-Vagrant browsertests setup" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62148 [12:57:46] New review: Physikerwelt; "You are right, the variable is never used." [mediawiki/extensions/Math] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/62142 [12:58:05] Change merged: jenkins-bot; [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/62142 [13:01:23] New patchset: Ganeshaditya1; "SpecialPreferences: Remove invalid tag in successbox" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/59822 [13:02:59] New review: Ganeshaditya1; "Really sorry guys .. pushed a wrong file last time." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/59822 [13:15:35] New patchset: Physikerwelt; "Introduce getter and setter for the MathRenderer" [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/60882 [13:30:11] New review: Anomie; "I'm still not entirely happy with that warning message, but it's good enough for now." [mediawiki/core] (master) C: 2; - https://gerrit.wikimedia.org/r/60450 [13:32:49] New patchset: Wizardist; "(Bug 46422) convertPlural makes using 0= and 1= rules impossible" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62149 [13:34:00] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60450 [13:52:22] New review: Krinkle; "Thanks!" [mediawiki/core] (master) C: 2; - https://gerrit.wikimedia.org/r/59822 [13:56:11] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/59822 [13:57:43] New review: Matmarex; "The tests pass for me locally... Krinkle, you broke jenkins :(" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/57526 [13:57:48] Krinkle: ^ [13:59:11] MatmaRex: PhantomJS doesn't implement .click() [13:59:13] https://integration.wikimedia.org/ci/job/mediawiki-core-qunit/3070/console [13:59:14] wtf [14:00:51] so much for headless "WebKit" [14:01:08] I find more and more often it is re-implementing a lot of stuff inside Qt and PhantomJS [14:01:12] are you sure this isn't some weird transitive issue? [14:01:18] doesn't share half as much with WebKit as people think [14:01:24] MatmaRex: I'm sure [14:01:40] heh, okay. [14:02:34] New patchset: Krinkle; "jquery.makeCollapsible: Enable passthru for links in premade toggle" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/57526 [14:09:27] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/57526 [14:10:33] New patchset: Matmarex; "jquery.makeCollapsible: cleanup" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/55466 [14:39:05] New review: Matmarex; "zzz" [mediawiki/extensions/timeline] (master) - https://gerrit.wikimedia.org/r/47336 [14:39:37] it's crazy that herding my patchsets trying to get them reviewed takes longer than writing them... [14:39:51] New review: Matmarex; "*bonk*" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/45944 [14:42:04] New patchset: Zfilipin; "Test scenarios for the buttons in the editing area of Translate UX" [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/61792 [14:42:44] New review: Zfilipin; "Moved feature file to backlog folder, since the scenarios are not yet implemented." [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/61792 [14:42:54] Change merged: Zfilipin; [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/61792 [14:43:55] New review: Alex Monk; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62127 [14:45:48] Quick announcement: Bugday on MediaWiki installer bugs starts in 15 minutes in #wikimedia-office - anybody can join and help retesting some Installer bug reports. See https://www.mediawiki.org/wiki/Bug_management/Triage/20130503 for more info. [14:46:59] New patchset: Zfilipin; "refactor PDF test" [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/61820 [14:50:07] New review: Zfilipin; "(1 comment)" [qa/browsertests] (master) C: -1; - https://gerrit.wikimedia.org/r/61820 [14:55:52] New review: Zfilipin; "(1 comment)" [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/61820 [15:11:29] Iterating through language codes in a maintenance script is causing a he/l10n_cache-stq.cdb): failed to open stream: Too many open files in includes/Cdb.php on line 129 [15:11:51] Do I need to yield somehow? Or is this a bug in i18n backend not closing file descriptors when it should [15:12:06] https://gist.github.com/anonymous/4124db472dd117c5c7da [15:12:17] Reedy: Nikerabbit: [15:12:25] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60255 [15:12:54] Why does it keep the file open? [15:13:23] Or is it writing to it asynchronously somehow? I assume that looping through should close it before I move on to the next one [15:14:34] * Nemo_bis sighs [15:25:20] New patchset: Matmarex; "jquery.makeCollapsible: Enable passthru for links in premade toggle" [mediawiki/core] (REL1_21) - https://gerrit.wikimedia.org/r/62156 [15:26:11] New review: Matmarex; "(This is slightly different than the commit on master, as the" [mediawiki/core] (REL1_21) - https://gerrit.wikimedia.org/r/62156 [15:27:54] hexmode: you there? see ^ [15:28:28] hexmode: should i use the same change-id when backporting if the backported fix is slightly different than the master one, to avoid depending on other commits? [15:28:39] hexmode: or should i just backport everything i need? five commits or so [15:29:08] meh, I'll work around it by not instantiating Language [15:29:31] Krinkle: also, can you look as well? it's the patch you amended and merged earlier today :) [15:29:43] MatmaRex: just saw your ping, 1s [15:29:48] MatmaRex: Same change-id, always. A commit to a non-master branch shouldn't have a change id that doesn't exist in master. [15:30:07] MatmaRex: back port because the refactor is in 1.21? [15:30:32] Krinkle: only a part of the refactor is in 1.21, and that's why i had to make some changes in this patch [15:30:40] lgtm [15:31:05] and i'm wondering what's the protocol in such cases :) [15:31:38] Krinkle: was? [15:31:43] Krinkle: i think "Backport_to_Stable" means "1.20 and earlier" [15:31:45] The person who merges it should also backport it. If forgotten, there is the "backport to stable" flag in bugzilla that'll remind the release manager [15:32:04] 1.21 is still a RC, no? [15:32:07] Krinkle: I have never seen that error before [15:32:11] yeah, we're in a weird mode right now since dev is 1.22 but 1.20 is stable [15:32:31] Krinkle: i think Nemo_bis told me that's how it works currently [15:33:28] Nikerabbit: https://gist.github.com/Krinkle/5509949 [15:33:39] Nikerabbit: (reduced test case, that's not what I'm doing) [15:34:07] Change merged: jenkins-bot; [mediawiki/core] (REL1_21) - https://gerrit.wikimedia.org/r/62156 [15:34:24] Nikerabbit: store->get() triggers a Reader::open [15:34:31] Nikerabbit: Which is then left open [15:34:42] Makes sense I suppose, in normal usage [15:35:34] Krinkle: hmm yes I guess they are left open until the language objects are destroyed [15:35:56] Nikerabbit: Ah, so there's a destroy method I could trigger [15:36:01] I suppose garbage collection doesn't trigger it [15:36:32] also, newFromCode should have some code to prevent caching too many [15:36:52] no wai [15:36:56] Anyway, I only needed it to call ucfirst() [15:37:00] it's in factory [15:37:11] Nikerabbit: Which I did incorrectly, so I'm avoiding the bug for now [15:37:20] so indeed I guess the garbage collector doesn't catch it [15:37:42] I shouldn't call targetlang-ucfirst (message key) to get page title. Only with english since message keys are en [15:38:54] you should use the content language of the wiki [15:39:56] anyway, I think that constructing a language object should not open CDB until needed [15:40:11] it makes it very slow and memory hog to construct many language objects [15:46:52] Nikerabbit: Yep [15:48:44] thinking of it, someone should file a bug about that [15:50:27] New patchset: Waldir; "Minor tweaks to installer UX / UI: - add a tooltip to the "help" links to make it clear they'll expand in place - don't use an IMG tag for the download link; instead, use a css background to the A element, so that there isn't an underlined space betwe" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [15:51:54] New review: Matmarex; "(1 comment)" [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62157 [15:52:12] New patchset: Matmarex; "Minor tweaks to installer UX / UI:" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [15:52:51] New patchset: Waldir; "Minor tweaks to installer UX / UI: - add a tooltip to the "help" links to make it clear they'll expand in place - don't use an IMG tag for the download link; instead, use a css background to the A element, so that there isn't an underlined space betwe" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [15:54:36] New patchset: Waldir; "Minor tweaks to installer UX / UI" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [15:56:36] New patchset: Krinkle; "deleteEqualMessages: Include list of pages in dry run" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62158 [15:56:43] New patchset: Krinkle; "deleteEqualMessages: Refactor to support all langcode subpages" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62159 [16:04:19] New review: Waldir; "For those reviewing, here's a set of screenshots for the download link change:" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [16:06:52] New patchset: Ryan Lane; "Fix another projectname reference" [mediawiki/extensions/OpenStackManager] (master) - https://gerrit.wikimedia.org/r/62160 [16:18:02] New patchset: Matthias Mullie; "Quickly enable/disable AFTv5" [mediawiki/extensions/ArticleFeedbackv5] (master) - https://gerrit.wikimedia.org/r/61797 [16:32:46] New patchset: Alex Monk; "Do not send notifications for page links when the page is a redirect" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62162 [16:35:18] New patchset: Matmarex; "Make Special:MovePage's "Reason" field an input, not a textarea" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62163 [16:36:27] New review: Adamw; "small bug need squish" [mediawiki/extensions/CentralNotice] (master) C: -1; - https://gerrit.wikimedia.org/r/62137 [16:37:59] New review: Brian Wolff; "Someone should probably get feedback from the angsty wikipedians before this goes live." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62163 [16:38:06] New review: Adamw; "DO NOT DEPLOY unless the iframe and msg saving patches are included." [mediawiki/extensions/CentralNotice] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/57090 [16:38:06] Change merged: Adamw; [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/57090 [16:49:15] New patchset: Mwalker; "Some Simple Cleanup in the CN Controller" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/61742 [16:50:23] New review: Mwalker; "Merging per Awights previous +2. His comments are addressed in later patches in this topic." [mediawiki/extensions/CentralNotice] (master) C: 2; - https://gerrit.wikimedia.org/r/61742 [16:50:24] Change merged: Mwalker; [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/61742 [16:52:58] New review: Ganeshaditya1; "No need to thank me man, I need to thank you for editing my commit message :) I really suck at frami..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/59822 [16:56:00] New review: Bsitu; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [16:58:41] New review: Lwelling; "Nikerabbit I'm running XHProf and following up the call stack when I see 100s of calls to the same f..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [16:59:55] New patchset: Mwalker; "Banner preview is done in an iframe" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [17:13:35] New review: Jdlrobson; "Is anyone able to edit the commit message post merge.. to say Opera Mini instead of Opera Mobile? @D..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61899 [17:15:26] New review: Matmarex; "That's not possible in git. We could use the commit notes feature (which we're already using to ment..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61899 [17:16:45] New review: Jdlrobson; "It /is/ possible in git using" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61899 [17:21:38] New review: Jdlrobson; "According to Roan there is no going back. Thus please please take care with commit messages in futur..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61899 [17:23:12] New review: Matmarex; "It would also change the SHA1s of all commits after this one, killing all outside references to them..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61899 [17:23:16] jdlrobson: ^ [17:28:24] New patchset: Lwelling; "Code style fixes in Language::isValidCode()" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62170 [17:29:12] New patchset: Ori.livneh; "Fixes to code style" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62171 [17:29:53] New review: Lwelling; "Style fix in https://gerrit.wikimedia.org/r/#/c/62170/" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62123 [17:32:20] New review: Aaron Schulz; "Looks pretty good" [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62130 [17:32:57] Change merged: jenkins-bot; [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62171 [17:35:00] New patchset: Jakub Vrána; "Pass correct number of parameters to Database constructor from installer" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62172 [17:40:42] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [17:46:40] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62172 [17:55:23] Change abandoned: MarkTraceur; "https://gerrit.wikimedia.org/r/62121" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61910 [18:02:43] New patchset: Jakub Vrána; "Prepare DatabaseMysql for mysqli" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [18:03:38] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/61371 [18:04:41] New review: Aaron Schulz; "(1 comment)" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [18:10:43] New patchset: Jakub Vrána; "Add support for mysqli extension" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62173 [18:10:46] New patchset: Jakub Vrána; "Allow installing MySQLi" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62174 [18:17:34] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [18:21:53] New patchset: Anomie; "Add Javascript login check against the central wiki" [mediawiki/extensions/CentralAuth] (master) - https://gerrit.wikimedia.org/r/58924 [18:24:32] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62127 [18:27:51] New review: Bsitu; "(1 comment)" [mediawiki/extensions/ArticleFeedbackv5] (master) C: -1; - https://gerrit.wikimedia.org/r/61360 [18:34:10] New patchset: Rjain; "location entry tooltip should link to geocoding page" [mediawiki/extensions/UploadWizard] (master) - https://gerrit.wikimedia.org/r/46960 [18:38:26] New patchset: Umherirrender; "Add content model to prop=info" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62177 [18:39:23] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61477 [18:48:16] New patchset: Waldir; "Installer: Minor tweaks to installer UI" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [18:49:25] New review: Waldir; "Thanks for the comments, Krinkle. I've fixed all the issues you mentioned in the latest patchset." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62157 [18:50:48] New patchset: Umherirrender; "Add wgContentModel to mw.config" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62178 [18:53:35] New patchset: Katie Horn; "Nuking recaptcha as cruft that hasn't been used in at least two years on payments." [mediawiki/extensions/DonationInterface] (master) - https://gerrit.wikimedia.org/r/62179 [18:55:33] New review: Mwalker; "*applause*" [mediawiki/extensions/DonationInterface] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/62179 [18:55:34] Change merged: Mwalker; [mediawiki/extensions/DonationInterface] (master) - https://gerrit.wikimedia.org/r/62179 [18:56:29] @seen andre__ [18:57:29] New patchset: Wikinaut; "(bug 47264) SpecialVersion: Add last local update timestamp" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/59373 [19:11:54] New review: Bsitu; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [19:14:01] New patchset: Dr0ptp4kt; "Merging MobileFrontend Vary updates." [mediawiki/core] (wmf/1.22wmf3) - https://gerrit.wikimedia.org/r/62183 [19:14:48] Change merged: Dr0ptp4kt; [mediawiki/core] (wmf/1.22wmf3) - https://gerrit.wikimedia.org/r/62183 [19:46:12] New patchset: Mattflaschen; "Use protocol-relative link for Asirra script." [mediawiki/extensions/ConfirmEdit] (master) - https://gerrit.wikimedia.org/r/62186 [19:49:03] i need someone who's intimate with the parser. [19:49:23] (being dead serious here.) [19:53:57] New patchset: IAlex; "Fix bootstrap in unit tests" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62187 [19:54:57] New review: Mattflaschen; "Chromium doesn't like the cert for https://challenge.asirra.com/js/AsirraClientSide.js , so we proba..." [mediawiki/extensions/ConfirmEdit] (master) - https://gerrit.wikimedia.org/r/62186 [19:56:57] New patchset: MaxSem; "Update MobileFrontend to production tip" [mediawiki/core] (wmf/1.22wmf2) - https://gerrit.wikimedia.org/r/62188 [19:57:12] Change merged: MaxSem; [mediawiki/core] (wmf/1.22wmf2) - https://gerrit.wikimedia.org/r/62188 [20:08:43] New review: Krinkle; "The download image is cut off." [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62157 [20:18:03] New patchset: Krinkle; "resourceloader: Consistently use 'resourceloader' log group" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62189 [20:18:05] New patchset: Krinkle; "resourceloader: Write to debug log for all fatal errors" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62190 [20:24:54] mwalker|away: you still away [20:26:24] Yippie, build fixed! [20:26:24] Project browsertests-en.wikipedia.org-linux-firefox build #37: FIXED in 2 min 51 sec: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.org-linux-firefox/37/ [20:27:18] New patchset: Spage; "Position the "Refresh" link over the image" [mediawiki/extensions/ConfirmEdit] (master) - https://gerrit.wikimedia.org/r/62191 [20:29:00] New review: Mattflaschen; "Good, except please remove testPerformanceGetter" [mediawiki/extensions/Math] (master) C: -1; - https://gerrit.wikimedia.org/r/60882 [20:29:39] New patchset: Krinkle; "mw.hook: Implement callback framework" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/56762 [20:30:16] hey, i only need four more pending changesets before they stop fitting one screen of my dashboard! [20:33:42] Project browsertests-en.wikipedia.org-windows-internet_explorer_8 build #35: FAILURE in 3 min 6 sec: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.org-windows-internet_explorer_8/35/ [20:39:22] Change merged: MarkTraceur; [integration/jenkins-job-builder-config] (master) - https://gerrit.wikimedia.org/r/61955 [20:39:35] New patchset: Lwelling; "Remove reduntant regex from calls to StringUtils::isUtf8()" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62192 [20:40:58] New review: Spage; "Preventing merge until new create account form's JS is compatible." [mediawiki/extensions/ConfirmEdit] (master) C: -1; - https://gerrit.wikimedia.org/r/62191 [20:43:13] New patchset: Alex Monk; "Fire Echo event when revision is rejected" [mediawiki/extensions/FlaggedRevs] (master) - https://gerrit.wikimedia.org/r/62193 [20:49:23] New review: Jakub Vrána; "Parent5446, I've compared this diff (and followups) with https://gerrit.wikimedia.org/r/56193 and th..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [20:49:31] New review: Jakub Vrána; "https://gerrit.wikimedia.org/r/62130/ (and followups) seems better." [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/56193 [20:49:33] New patchset: Anomie; "Replace image-based AutoLogin with iframes" [mediawiki/extensions/CentralAuth] (master) - https://gerrit.wikimedia.org/r/62194 [20:50:58] New patchset: Alex Monk; "Fire Echo event when revision is rejected" [mediawiki/extensions/FlaggedRevs] (master) - https://gerrit.wikimedia.org/r/62193 [20:54:02] New review: TheDJ; "It's defined for texvc in this file https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/Mat..." [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/61284 [20:56:30] Change abandoned: Parent5446; "(no reason)" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/56193 [21:00:12] New patchset: TheDJ; "MathJax: Support the texvc command \vline" [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/61287 [21:02:11] New review: TheDJ; "Reduced the change to vline, while we further discuss \texttt and \textsf in the bug report." [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/61287 [21:02:12] New review: Parent5446; "This solution does look cleaner, so I've abandoned my patch so that we're not splitting efforts." [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62130 [21:02:30] Change merged: MarkTraceur; [mediawiki/extensions/Parsoid/js/contrib] (master) - https://gerrit.wikimedia.org/r/61946 [21:03:50] Change merged: jenkins-bot; [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/62162 [21:04:05] New review: OliverKeyes; "This'd be awesome functionality to have!" [mediawiki/extensions/FlaggedRevs] (master) C: 1; - https://gerrit.wikimedia.org/r/62193 [21:04:32] New review: TheDJ; "I noticed that nageh is using mathJax.userConfig" [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/61182 [21:11:19] New review: Parent5446; "Also, release notes." [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62174 [21:11:58] New review: Kaldari; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [21:13:02] Change merged: MarkTraceur; [mediawiki/extensions/Parsoid/js/contrib] (master) - https://gerrit.wikimedia.org/r/61947 [21:13:14] New patchset: Physikerwelt; "Introduce getter and setter for the MathRenderer" [mediawiki/extensions/Math] (master) - https://gerrit.wikimedia.org/r/60882 [21:14:41] New patchset: TheDJ; "Tablesorter: Support sortable column headers with rowspans" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60871 [21:30:15] New patchset: TheDJ; "Tablesorter: Support sortable column headers with rowspans" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/60871 [21:33:08] mwalker: there? [21:33:42] mwalker: let me know how https://gerrit.wikimedia.org/r/61988 looks to you :) [21:34:49] New review: Bsitu; "(1 comment)" [mediawiki/extensions/Echo] (master) - https://gerrit.wikimedia.org/r/61819 [21:37:43] New patchset: Matmarex; "move a little of common skin code to SkinTemplate" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/57913 [21:38:52] New review: Matmarex; "Rebased due to merge conflict with whitespace changes..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/57913 [21:49:39] New patchset: Adamw; "Banner preview is done in an iframe" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [21:49:53] Change merged: Adamw; [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62097 [21:51:21] Change abandoned: Adamw; "work done in another patch" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/28955 [21:55:15] New patchset: Ori.livneh; "Document *all* manifests and managed files" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62199 [22:06:17] New patchset: Adamw; "CentralNotice UI Message Handling ++Graceful" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62133 [22:09:12] New review: Mattflaschen; "Unfortunately, this seems to hurt the non-ACUX case. Before (e.g. on the old signup form), the refr..." [mediawiki/extensions/ConfirmEdit] (master) C: -1; - https://gerrit.wikimedia.org/r/62191 [22:10:09] Change merged: Adamw; [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62133 [22:10:49] New patchset: Matmarex; "Kill some references to legacy skins" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62200 [22:10:49] New patchset: Matmarex; "Update docs/skin.txt" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62201 [22:23:58] New review: Mattflaschen; "Steven argues that a horizontal jump is better, and that it's worth it to more closely associate it ..." [mediawiki/extensions/ConfirmEdit] (master) - https://gerrit.wikimedia.org/r/62191 [22:26:17] New patchset: Jakub Vrána; "Prepare DatabaseMysql for mysqli" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [22:26:17] New patchset: Jakub Vrána; "Allow installing MySQLi" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62174 [22:26:18] New patchset: Jakub Vrána; "Add support for mysqli extension" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62173 [22:29:47] New patchset: Adamw; "Live preview link for banners" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62204 [22:50:23] New patchset: Adamw; "Live preview link for banners" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62204 [22:56:15] New patchset: Ori.livneh; "Document *all* manifests and managed files" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62199 [22:58:05] New patchset: Jakub Vrána; "Prepare DatabaseMysql for mysqli" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62130 [22:58:08] New patchset: Jakub Vrána; "Allow installing MySQLi" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62174 [22:58:08] New patchset: Jakub Vrána; "Add support for mysqli extension" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62173 [22:58:28] Change merged: Ori.livneh; [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/62199 [23:04:15] New patchset: Adamw; "Ugh... fixing all the spacing!" [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/62107 [23:04:32] Change merged: Adamw; [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/62107 [23:06:05] New patchset: Adamw; "Fixing Adyen Spacing to Tabs" [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/62115 [23:06:29] Change merged: Adamw; [wikimedia/fundraising/PaymentsListeners] (master) - https://gerrit.wikimedia.org/r/62115 [23:12:31] New patchset: Adamw; "Live preview link for banners" [mediawiki/extensions/CentralNotice] (master) - https://gerrit.wikimedia.org/r/62204 [23:14:36] New patchset: Luis Felipe Schenone; "Changed the
    and
  • of galleries for
    and
    " [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62207 [23:15:26] New patchset: Aaron Schulz; "Include user name in weird User insertion exception." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62208 [23:15:45] gerrit slow [23:17:32] New review: Mattflaschen; "Looks good." [mediawiki/extensions/Math] (master) C: 2; - https://gerrit.wikimedia.org/r/60882 [23:18:38] csteipp: https://gerrit.wikimedia.org/r/#/c/62208/1/uincludes/User.php [23:21:49] painfully slow... [23:22:36] New review: Matmarex; "New CSS has to be compatible with old HTML. This is because (at least on Wikipedia) the CSS and JS t..." [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/62207 [23:23:04] New review: CSteipp; "Should help track down 41609." [mediawiki/core] (master) C: 2; - https://gerrit.wikimedia.org/r/62208 [23:24:53] New review: Matmarex; "As for the issue you describe in the commit message – does killing all those \t's and \n's fix it?..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/62207 [23:32:00] Yippie, build fixed! [23:32:01] Project browsertests-en.wikipedia.org-windows-internet_explorer_8 build #36: FIXED in 4 min 28 sec: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.org-windows-internet_explorer_8/36/ [23:41:10] Gerrit errors? [23:58:43] Gerrit seems to be screwed up again [23:59:00] It's not responding to web requests or to git review [23:59:07] s/again/still/ [23:59:11] Oh. [23:59:18] This just came up in -operations as well. [23:59:25] Too many channels.