[00:03:39] Project beta-code-update-eqiad build #192260: 04FAILURE in 39 sec: https://integration.wikimedia.org/ci/job/beta-code-update-eqiad/192260/ [00:05:28] PROBLEM - Puppet errors on deployment-eventlogging04 is CRITICAL: CRITICAL: 50.00% of data above the critical threshold [0.0] [00:08:21] PROBLEM - Puppet errors on saucelabs-03 is CRITICAL: CRITICAL: 30.00% of data above the critical threshold [0.0] [00:13:43] Yippee, build fixed! [00:13:43] Project beta-code-update-eqiad build #192261: 09FIXED in 43 sec: https://integration.wikimedia.org/ci/job/beta-code-update-eqiad/192261/ [00:31:17] PROBLEM - Long lived cherry-picks on puppetmaster on deployment-puppetmaster02 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [00:45:28] RECOVERY - Puppet errors on deployment-eventlogging04 is OK: OK: Less than 1.00% above the threshold [0.0] [00:48:21] RECOVERY - Puppet errors on saucelabs-03 is OK: OK: Less than 1.00% above the threshold [0.0] [00:51:40] I apologize to everyone for my bad behaviour on gerrit [00:51:57] I promise I will not make anymore problems there [01:01:05] 10Gerrit, 10Upstream: Change Gerrit's default indentation to tabs - https://phabricator.wikimedia.org/T168590#3942568 (10Paladox) 05Open>03Resolved a:03Paladox [01:01:41] legoktm: btw, I noticed mediawiki-phpunit-coverage-patch has its own pipeline (coverage), but uses permanent slaves. [01:01:53] Is that more efficient/faster than if it'd use docker? [01:01:57] 10Gerrit, 10Upstream: In changeset view, clicking the "Owner" name does not display list of their changesets but of unrelated other user - https://phabricator.wikimedia.org/T146361#3942574 (10Paladox) 05Open>03Resolved a:03Paladox This works in polygerrit (it used the email when i tryed just now). [01:02:49] 10Gerrit, 10Regression, 10Upstream: Gerrit doesn't display committed images - https://phabricator.wikimedia.org/T153085#3942577 (10Paladox) 05Open>03Resolved a:03Paladox Works for me in polygerrit https://gerrit.wikimedia.org/r/c/325819/5/app/screenshots-ref/org.wikipedia.descriptions.DescriptionEditVi... [01:03:48] 10Gerrit, 10Patch-For-Review: Replace using certificates with tokens when using its-phabricator - https://phabricator.wikimedia.org/T178385#3942582 (10Paladox) We just need to remove the cert param and then we can close this as resolved :). [01:04:09] 10Gerrit: Gerrit needs a very wide screen or has big horizontal scroll - https://phabricator.wikimedia.org/T38471#3942600 (10Paladox) [01:04:12] 10Gerrit, 10Release-Engineering-Team (Someday), 10Patch-For-Review: Update gerrit to 2.14.6 - https://phabricator.wikimedia.org/T156120#3942583 (10Paladox) 05Open>03Resolved a:03demon [01:05:24] 10Gerrit, 10Patch-For-Review: Gerrit: Convert Velocity templates to Closure Templates - https://phabricator.wikimedia.org/T158008#3023302 (10Paladox) @demon did we remove the .vm templates on gerrit? my changes only stopped re adding them. We need to remove them so that soy is used :). [01:05:54] 10Gerrit, 10Release-Engineering-Team (Someday), 10Operations: Make sure replying to emails in gerrit 2.14 works - https://phabricator.wikimedia.org/T158915#3051483 (10Paladox) we can try this now that we have upgraded to 2.14. [01:05:58] 10Phabricator, 10GerritBot, 10Security: @gerritbot probably allows users to make arbitrary changes to Phabricator tasks by submitting crafted Gerrit changesets - https://phabricator.wikimedia.org/T140366#3942608 (10demon) [01:07:23] 10Gerrit: Data loss: Click "Reply...", enter text, open a diff, have your text lost - https://phabricator.wikimedia.org/T174551#3942611 (10Paladox) 05Open>03Resolved We have updated to 2.14 now. Please reopen if the problem is not fix in polygerrit. [01:07:46] 10Gerrit, 10Release-Engineering-Team, 10Operations: Add prometheus exporter to Gerrit - https://phabricator.wikimedia.org/T184086#3942613 (10Paladox) We can do this now :) [01:08:44] 10Gerrit, 10Analytics-Tech-community-metrics, 10Upstream: Gerrit patchset 99101 cannot be accessed: "500 Internal server error" - https://phabricator.wikimedia.org/T161206#3942614 (10Paladox) This is not resolved in 2.14. And most likly won't as it probaly hit a bug in a very old gerrit release that was fixe... [01:09:21] no_justification i haven't herd any reports yet for https://phabricator.wikimedia.org/T152640 yay [01:09:28] * paladox shuts up now to prevent bad karma [01:11:08] i think this may be resolved https://phabricator.wikimedia.org/T43608 [01:11:41] Krinkle: MediaWiki phpunit bootstrap scripts hasn't been ported to docker yet. Also I think the nodepool images don't have the resources for coverage. But tbh I haven't tested it on nodepool [01:11:51] Also permanent slaves are way easier to debug [01:14:20] legoktm: Yeah, Nodepool would increase load more which is already a problem. [01:14:29] But overall we're moving toward docker, and permanent slaves seems like a step back. [01:14:36] On the other hand, I totally get it, and actually makes a lot of sense. [01:14:44] It has pretty fast startup time. [01:14:52] And good local git caching presumably for small set of repos. [01:15:06] and if it's fast enough, even with only a few executors, won't block much. [01:15:11] So it's quite nice that way [01:15:45] Until docker for more complex mw jobs is ready, I presume, which should be quick too, albeit theoretically marginally slower than perma slaves. [01:50:59] 10Gerrit, 10Patch-For-Review: Gerrit: Convert Velocity templates to Closure Templates - https://phabricator.wikimedia.org/T158008#3942636 (10demon) 05Open>03Resolved a:03demon Just deleted them now. [02:26:37] (03CR) 10Krinkle: [C: 032] doc: Update link for Cumin to point to 'master' [integration/docroot] - 10https://gerrit.wikimedia.org/r/407869 (owner: 10Krinkle) [02:27:09] (03Merged) 10jenkins-bot: doc: Update link for Cumin to point to 'master' [integration/docroot] - 10https://gerrit.wikimedia.org/r/407869 (owner: 10Krinkle) [02:27:15] (03CR) 10jenkins-bot: doc: Update link for Cumin to point to 'master' [integration/docroot] - 10https://gerrit.wikimedia.org/r/407869 (owner: 10Krinkle) [02:53:11] 10Continuous-Integration-Config, 10Jenkins: PHP test coverage decreased - https://phabricator.wikimedia.org/T186380#3942682 (10Huji) [03:09:05] 10Gerrit, 10Release-Engineering-Team, 10User-Addshore: Access to create gerrit repos for Addshore - https://phabricator.wikimedia.org/T185594#3942696 (10demon) 05Open>03Resolved a:03demon Done. [03:13:35] 10Gerrit: Gerrit rebase button returns "500 Internal server error" for a specific changeset - https://phabricator.wikimedia.org/T185849#3926755 (10demon) ``` [2018-01-28 18:14:43,711] [HTTP-18210] ERROR com.google.gerrit.httpd.restapi.RestApiServlet : Error in POST /r/changes/406500/revisions/3b34bcccf483992e05c... [03:14:42] PROBLEM - Free space - all mounts on deployment-fluorine02 is CRITICAL: CRITICAL: deployment-prep.deployment-fluorine02.diskspace._srv.byte_percentfree (<44.44%) [03:33:53] 10Continuous-Integration-Config, 10Jenkins: PHP test coverage decreased - https://phabricator.wikimedia.org/T186380#3942722 (10Krinkle) 05Open>03Invalid @Huji That is working as expected. Your change proposal adds new source code, but does not add new tests to verify the added code. As such, the amount of... [03:46:54] CI is stuck [03:48:12] !log disabled/enabled gearman in jenkins [03:48:20] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [03:49:47] 2018-02-03 03:49:38,038 ERROR zuul.DependentPipelineManager: Exception while launching job mediawiki-core-php70-phan-docker for change : [03:49:47] Traceback (most recent call last): [03:49:47] File "/usr/share/python/zuul/local/lib/python2.7/site-packages/zuul/scheduler.py", line 1518, in _launchJobs [03:49:47] dependent_items) [03:49:47] File "/usr/share/python/zuul/local/lib/python2.7/site-packages/zuul/launcher/gearman.py", line 318, in launch [03:49:48] destination_path = os.path.join(item.change.getBasePath(), [03:49:49] legoktm: 407165 is not getting picked up as having the C+2 removed still. [03:49:50] File "/usr/share/python/zuul/local/lib/python2.7/site-packages/zuul/model.py", line 904, in getBasePath [03:49:53] self.number[-2:], self.number, self.patchset) [03:49:55] TypeError: 'int' object has no attribute '__getitem__' [03:50:02] Fun. [03:50:20] Incompatibility with new gerrit version or just something odd? [03:50:49] 10Continuous-Integration-Infrastructure, 10Zuul: Exception while launching job: TypeError: 'int' object has no attribute '__getitem__' - https://phabricator.wikimedia.org/T186381#3942743 (10Legoktm) p:05Triage>03Unbreak! [03:51:03] I think so [03:51:08] Eurgh. [03:51:09] paladox, no_justification: ^^ [03:51:35] Son of a.... this is why we upgraded [03:51:50] (upgraded zuul) [03:52:10] Did we need to upgrade gearman too or whatever? [03:52:16] Wonder what's different about that change...plenty of others did ok [03:53:18] legoktm: Can you pull 407165,3 from the run so the rest goes through? [03:53:30] I can just drop the queue [03:53:34] I don't know how to edit it [03:54:01] Do it. [03:54:33] That "int" type is suspicious. [03:54:41] I thought that was *explicitly* why we upgraded [03:54:45] Because of an api change [03:54:56] no_justification: do you want to debug anything or is it ok if I restart? [03:55:02] it's one exception per second basically right now [03:55:04] Go ahead and restart. [03:55:08] !log restarting zuul to drop 407165,3 from the queue [03:55:11] We've got a stacktrace [03:55:13] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [03:55:40] ok, it's back [03:56:00] Should we re-C+2 407165,3 to test it? [03:56:46] 2018-02-03 03:56:33,757 ERROR zuul.Gearman: Job is not registered with Gearman [03:56:46] 2018-02-03 03:56:33,757 INFO zuul.Gearman: Build complete, result NOT_REGISTERED [03:56:48] wtffff [03:57:11] That's not good. [03:57:24] legoktm: Since you're on the machine...can you look at /usr/share/python/zuul/local/lib/python2.7/site-packages/zuul/connection/gerrit.py and see if line 65 casts change_number to a string? [03:57:26] eg: https://github.com/openstack-infra/zuul/commit/c7370284742f2ed5c136d0e5fcd08ca2d23c8fe1 [03:58:04] no_justification: yep, it has the str() call [03:58:47] Hmmm [03:59:39] I wonder if there's other casts Zuul needs. The int/str mismatch could be more widespread.... [03:59:41] This is ugly. [04:00:00] Is this a patch on a release or a real release? [04:00:16] 'Cos if it's a real release, how the hell haven't other people had that issue? [04:00:30] uh, let me try one more gearman disable/renable [04:00:30] I'm not entirely sure what Antoine built [04:01:05] James_F: https://phabricator.wikimedia.org/T158243#3886849 [04:01:07] !log disable/enabled gearman in jenkins [04:01:12] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [04:01:13] That whole task, but specifically those two comments.... [04:01:20] * James_F nods. [04:01:50] ok, jobs are back [04:01:56] Processing OK? [04:02:01] So something based on 2.5.1? [04:02:23] seemingly [04:03:59] The CI onion has too many layers. I can't wait until we can at least remove Nodepool from the stack :\ [04:04:01] Huh. [04:05:09] 10Continuous-Integration-Infrastructure, 10Zuul: Exception while launching job: TypeError: 'int' object has no attribute '__getitem__' - https://phabricator.wikimedia.org/T186381#3942785 (10Legoktm) It appears that this was caused specifically by change 407165,3. I restarted zuul (dropping the entire queue), a... [04:05:28] Now, curious if we re-queued that change..... [04:05:41] Yeah. [04:09:40] that change has a depends-on [04:09:46] I don't think there's anything else unique about it [04:10:44] (A merged depends-on.) [04:11:56] ahh [04:12:03] James_F: manifest_version 2 was introduced in 1.29 [04:12:06] it was backed out of 1.28 [04:12:42] legoktm: Umm. Really? Phab Diffusion reported it as in REL1_28. [04:12:46] Oh, it was reverted? Meh. [04:12:54] it was reverted out of REL1_28 [04:13:34] Lame. [04:13:41] OK, time to go make a bunch of fixes. [04:20:19] legoktm: In the end, only three wrong merged patches (and a bunch that need corrections before merge). [04:20:39] :/ [04:20:43] I can help with that after dinner [04:20:53] legoktm: No worries, it can wait a bunch more. :-) [04:21:15] legoktm: >1.28 is not wrong, it's just not sufficient, after all. :-) [04:21:23] right [04:22:07] 10Continuous-Integration-Infrastructure, 10Epic: Provide infrastructure to store files by project/branch post-merge to compare with pre-merge - https://phabricator.wikimedia.org/T101545#3942820 (10Krinkle) [04:22:11] 10Continuous-Integration-Infrastructure, 10phpunit-patch-coverage, 10Epic, 10MediaWiki-Platform-Team (MWPT-Q3-Jan-Mar-2018), and 2 others: Provide (pre-merge) code coverage reports on patchsets - https://phabricator.wikimedia.org/T101544#3942819 (10Krinkle) [04:27:48] Also, wikibugs has stopped reporting events from gerrit. [04:33:48] Hm.. might need a restart, following the ugrades. [04:34:08] https://tools.wmflabs.org/?tool=wikibugs [05:21:58] again? [05:23:04] restarted [05:23:27] 2018-02-03 03:11:55,006 - wikibugs.wb2-grrrrit - ERROR - Error, probably SSH connection dropped. [05:23:27] Traceback (most recent call last): [05:23:27] File "/data/project/wikibugs/wikibugs2/grrrrit.py", line 258, in [05:23:27] main() [05:23:27] File "/data/project/wikibugs/wikibugs2/grrrrit.py", line 243, in main [05:23:29] processed = process_event(parsed) [05:23:31] File "/data/project/wikibugs/wikibugs2/grrrrit.py", line 46, in process_event [05:23:34] 'type': 'PS' + event['patchSet']['number'], [05:32:29] no_justification: all I can think of is that gerrit is being inconsistent with the type of 'number', sometimes making it an int, and sometimes a string?? [05:35:23] Well casting string to string shouldn't matter. [05:35:33] Which is why zuul is/should coerce [05:35:42] In case it is inconsistent [05:36:07] (but really, Gerrit is returning over ssh, it's on the libraries zuul is using to interpret it) [05:54:39] no_justification: the traceback I just posted is from wikibugs, which now uses str() to hopefully avoid any issues [05:55:10] Oh derp [05:55:29] Yeah same issue as zuul [05:55:41] Still baffled as to why it barfed earlier [05:55:42] legoktm: It's double-posting, BTW. [05:56:17] oh hmmm [05:56:44] ghost process I guess [05:56:51] Probably. [05:57:25] I killed and restarted it [05:57:40] there's something weird about toolforge grid endinge signal handling not actually killing scripts [05:57:48] * James_F nods. [07:14:40] RECOVERY - Free space - all mounts on deployment-fluorine02 is OK: OK: All targets OK [09:16:43] (03Draft1) 10Hashar: dockerfile: build draft [integration/config] - 10https://gerrit.wikimedia.org/r/382685 [09:16:45] (03Abandoned) 10Hashar: dockerfile: build draft [integration/config] - 10https://gerrit.wikimedia.org/r/382685 (owner: 10Hashar) [11:33:52] 10Project-Admins, 10I18n: Add "Turkish-sites" tag - https://phabricator.wikimedia.org/T186403#3943170 (10Amire80) [12:09:13] legoktm was this resolved? :) [12:10:38] legoktm i see https://gerrit.wikimedia.org/r/c/407887/1/grrrrit.py :) [12:11:24] 10Continuous-Integration-Infrastructure, 10Zuul: Exception while launching job: TypeError: 'int' object has no attribute '__getitem__' - https://phabricator.wikimedia.org/T186381#3943221 (10Paladox) p:05Unbreak!>03High Per "I restarted zuul (dropping the entire queue), and things look back to normal now." [12:12:00] no_justification legoktm you hit the new api change [12:12:09] gerrit no longer makes the number a string [12:12:39] it's a int, so we have to do str in all the library's where it trys to get the number but dosen't make sure it's a string [12:15:23] 10Gerrit, 10Upstream: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943225 (10Dvorapa) [12:15:55] Hmm upstream no longer have a 2.5 or 2.6 branch [12:15:57] for zuul [12:16:16] 10Gerrit, 10Upstream: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943225 (10Paladox) Hi, in your settings click fit to screen. [12:18:03] 10Gerrit, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10Zuul: Update zuul to upstream master - https://phabricator.wikimedia.org/T158243#3943238 (10Paladox) I doin't think we can upgrade to the latest zuul version now (we can upgrade to the 2.6 tagged version) upstream support 3.x now only. [12:20:13] 10Continuous-Integration-Infrastructure, 10Zuul: Exception while launching job: TypeError: 'int' object has no attribute '__getitem__' - https://phabricator.wikimedia.org/T186381#3943240 (10Paladox) Oh i see what happened here. We didn't test using Depends-On: Ie7073f2048ba2b79a8b36ad913453008ec3555ce . so we... [12:20:53] 10Gerrit, 10Upstream: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943241 (10Dvorapa) Ah, I see. I think this should be on by default [12:23:39] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943243 (10Dvorapa) Probably not an upstream bug, but only our side config issue [12:24:19] 10Continuous-Integration-Infrastructure, 10Zuul: Exception while launching job: TypeError: 'int' object has no attribute '__getitem__' - https://phabricator.wikimedia.org/T186381#3943245 (10Paladox) hmm that works for me on my test site. [12:25:45] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943247 (10Paladox) 05Open>03declined Nope not a config issue. It enables line wrapping so it will affect the gwtui display too. [12:29:48] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should not look like commits - https://phabricator.wikimedia.org/T186406#3943251 (10Dvorapa) [12:33:21] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943262 (10Dvorapa) So everybody will need to change this setting when switching default UI to the New UI? [12:34:17] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits - https://phabricator.wikimedia.org/T186406#3943263 (10Dvorapa) [12:35:36] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943264 (10Paladox) Yep if they want it to fit to the screen. But also there are plans to remove GWTUI when polygerrit is finished. [12:38:14] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943275 (10Dvorapa) @Paladox: OK then BTW what is GWTUI? Never heard, Google doesn't help.. [12:48:46] 10Gerrit, 10Upstream: Polygerrit should fold not current line comments by default - https://phabricator.wikimedia.org/T186408#3943278 (10Dvorapa) [13:06:04] 10Gerrit: Fix access rules for gerrit - mediawiki/extensions/LanguageTool - https://phabricator.wikimedia.org/T186410#3943300 (10MarcoAurelio) [13:13:04] 10Gerrit, 10Patch-For-Review: Fix access rules for gerrit - mediawiki/extensions/LanguageTool - https://phabricator.wikimedia.org/T186410#3943325 (10MarcoAurelio) Modeled after extension-CentralAuth permissions. Let me know if there's something superfluous/wrong, etc. [13:22:06] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943340 (10Paladox) GWTUI is the default ui currently in gerrit. It's being replaced with polygerrit. [13:27:12] 10Gerrit, 10Upstream: Polygerrit should fold not current line comments by default - https://phabricator.wikimedia.org/T186408#3943278 (10Paladox) Filled upstream https://bugs.chromium.org/p/gerrit/issues/detail?id=8276 we will most likly get a comment on it next week (no promises though) but new ui comes a ded... [13:29:11] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits - https://phabricator.wikimedia.org/T186406#3943251 (10Paladox) In polygerrit 2.15 it looks like {F12982482} [13:30:03] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits - https://phabricator.wikimedia.org/T186406#3943350 (10Paladox) You can give polygerrit from the master a try @ http://gerrit-new.wmflabs.org/r/ [13:33:18] 10Gerrit, 10Upstream: Unable to see which patch is the parent/dependency of specific patch - https://phabricator.wikimedia.org/T141947#3943352 (10Paladox) 05Open>03Resolved Please reopen if polygerrit dosen't fix this. But i can tell in polygerrit :). [13:34:23] 10Gerrit: Gerrit: autocomplete to add reviewers slow - https://phabricator.wikimedia.org/T183234#3943354 (10Paladox) @demon in polygerrit 2.14 it only allows you to type 3 characters in before it searches. In 2.15 this is fixed as you can have a 1 letter username apparently or name. [13:35:18] 10Gerrit, 10Upstream: reverting patchsets results in gerrit not wrapping lines in commit message - https://phabricator.wikimedia.org/T175607#3943358 (10Paladox) Polygerrit new revert dialog actually asks you to insert reasons now :) [13:35:56] 10Gerrit: "add comment" feature doesn't allow you to write a comment while viewing the code or viewing the other comments - https://phabricator.wikimedia.org/T48777#3943360 (10Paladox) 05Open>03Resolved Polygerrit allows you click show all which shows all the diffs. [13:35:58] 10Gerrit: Add 'reply' button to Gerrit comments that quotes the original comment - https://phabricator.wikimedia.org/T48776#3943362 (10Paladox) [13:36:35] that was alot of tasks to resolve (sorry for spma) [13:36:37] spam [13:38:42] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits - https://phabricator.wikimedia.org/T186406#3943365 (10Dvorapa) So this will be solved after an update to (not yet released) 2.14.7, right? [13:40:15] 10Gerrit: Polygerrit side-by-side comparison does not fit view - https://phabricator.wikimedia.org/T186405#3943370 (10Dvorapa) Oh, I see, thank you [13:40:50] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits - https://phabricator.wikimedia.org/T186406#3943372 (10Paladox) @Dvorapa nope, this will be solved when we upgrade to 2.15 which is currently in the rc phrase and should hopefu... [13:43:19] 10Gerrit, 10Documentation, 10Google-Code-in-2017, 10Need-volunteer: Update Wikimedia's Gerrit documentation for new user interface in Gerrit 2.14 - https://phabricator.wikimedia.org/T179759#3943374 (10Paladox) We have updated now. We should probaly have to add docs on users to enable "Fit to display" (bas... [13:44:14] 10Gerrit, 10Upstream: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits - https://phabricator.wikimedia.org/T186406#3943375 (10Dvorapa) Oh, I see, thank you for details. I look forward the following UI/UX changes in Gerrit you posted [13:45:32] 10Gerrit, 10Upstream: Polygerrit should fold not current line comments by default - https://phabricator.wikimedia.org/T186408#3943376 (10Dvorapa) Thank you, let's wait then [13:59:45] (03CR) 10Zoranzoki21: "check experimental" [integration/config] - 10https://gerrit.wikimedia.org/r/406524 (https://phabricator.wikimedia.org/T183674) (owner: 10Zoranzoki21) [14:00:06] 10Phabricator: Support token or cert in conduit.connect - https://phabricator.wikimedia.org/T186414#3943412 (10Paladox) by token i mean cli-* or api-* and should deprecate authToken and authSignature so we should have a new field called token. So that token or authSignature and authToken have to be set. [14:05:05] 10Phabricator: Support token or cert in conduit.connect - https://phabricator.wikimedia.org/T186414#3943416 (10Paladox) (i need access to the sessionKey generated when connecting with conduit.connect) [14:29:11] 10Gerrit: Create trusted group in gerrit - https://phabricator.wikimedia.org/T186415#3943428 (10Paladox) [14:34:08] 10Project-Admins, 10I18n: Add "Turkish-sites" tag - https://phabricator.wikimedia.org/T186403#3943170 (10Liuxinyu970226) If this is just a reason to create a tag project, then why not "Catalan-sites"? "Janapese-sites"? "Korean-sites"? "Norwegian-sites"? "Odia-sites"? "Serbian-sites"? "Ukrainian-sites"?... Task... [14:53:16] PROBLEM - Host deployment-puppetdb01 is DOWN: CRITICAL - Host Unreachable (10.68.23.76) [15:30:12] 10Beta-Cluster-Infrastructure, 10Release-Engineering-Team, 10Recommendation-API, 10Patch-For-Review, 10Scoring-platform-team (Current): What to do with deployment-sca03? - https://phabricator.wikimedia.org/T184501#3943535 (10Ladsgroup) Nope :D [16:33:18] PROBLEM - Puppet errors on deployment-logstash2 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [18:54:29] PROBLEM - Puppet errors on deployment-secureredirexperiment is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [20:05:19] 10Gerrit, 10Patch-For-Review: Switch to mariadb java connector once we upgrade to gerrit 2.14 - https://phabricator.wikimedia.org/T176164#3943725 (10Paladox) @demon should we try doing this? :) [20:07:25] 10Gerrit: Short commit hashes with all numbers cannot be searched using gerrit - https://phabricator.wikimedia.org/T86035#3943728 (10Paladox) 05Open>03Resolved Please reopen if this is not resolved. [20:14:57] 10Gerrit, 10Patch-For-Review: List change author in gerrit email templates - https://phabricator.wikimedia.org/T43608#3943730 (10Paladox) 05Open>03Resolved a:03Paladox [20:15:08] 10Gerrit: List change author in gerrit email templates - https://phabricator.wikimedia.org/T43608#455447 (10Paladox) [20:19:40] 10Release-Engineering-Team, 10Zuul: Update zuul to v3 - https://phabricator.wikimedia.org/T186426#3943735 (10Paladox) [20:40:17] 10Continuous-Integration-Infrastructure, 10MediaWiki-extensions-General: Create composer package that contains most of the MediaWiki extension phan config instead of copy/pasting it each time - https://phabricator.wikimedia.org/T186315#3943762 (10Legoktm) [20:55:42] (03PS1) 10Legoktm: Add mediawiki/tools/phan [integration/config] - 10https://gerrit.wikimedia.org/r/407991 [20:59:48] (03CR) 10Legoktm: [C: 032] Add mediawiki/tools/phan [integration/config] - 10https://gerrit.wikimedia.org/r/407991 (owner: 10Legoktm) [21:01:17] (03Merged) 10jenkins-bot: Add mediawiki/tools/phan [integration/config] - 10https://gerrit.wikimedia.org/r/407991 (owner: 10Legoktm) [21:02:12] !log deployed https://gerrit.wikimedia.org/r/407991 [21:02:17] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [21:05:10] !log manually deleted /srv/zuul/git/mediawiki/tools/phan on contint1001 so zuul could clone the new repo [21:05:16] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [21:08:22] PROBLEM - Puppet errors on deployment-mx02 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0]