[00:02:35] (03PS1) 10Jforrester: dockerfiles: [tox-labs-striker] Migrate to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651296 [00:02:54] (03PS1) 10Jforrester: jjb: Switch striker jobs to one based off buster [integration/config] - 10https://gerrit.wikimedia.org/r/651297 [00:47:35] (03PS9) 10DannyS712: Make use of PHPUnitTestTrait in more sniffs [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/650189 [00:47:39] (03CR) 10DannyS712: Make use of PHPUnitTestTrait in more sniffs (032 comments) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/650189 (owner: 10DannyS712) [01:09:59] 10MediaWiki-Codesniffer, 10User-DannyS712: Disallow multi class property declaration with code sniff - https://phabricator.wikimedia.org/T268287 (10DannyS712) 05Open→03Resolved a:03DannyS712 [01:29:29] 10Diffusion, 10Phabricator, 10Release-Engineering-Team-TODO (2020-01 to 2020-03 (Q3)), 10Toolforge, and 2 others: Mirroring Diffusion repositories to GitHub seems to be broken - https://phabricator.wikimedia.org/T242898 (10bd808) I just updated the credentials for all of the mirrors to https://github.com/t... [01:37:44] (03CR) 10BryanDavis: [C: 03+1] Add gate-and-submit-l10n to Toolhub [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [02:36:38] (03CR) 10DannyS712: "This change is ready for review." [wvui] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651219 (owner: 10DannyS712) [03:10:44] anyone have any patches they want reviewed? [03:32:56] * DannyS712 messing around on beta cluster to debug T268210 [04:17:00] * DannyS712 is done [04:45:37] 10Continuous-Integration-Config, 10Release-Engineering-Team, 10Pywikibot, 10Pywikibot-tests: Enable Pywikibot fasttest with Python 3.8 / Python 3.9 - https://phabricator.wikimedia.org/T270389 (10Xqt) >>! In T270389#6707082, @Jdforrester-WMF wrote: > Now running 3.8; 3.9 isn't available yet, sorry. > > Do... [08:29:36] 10Gerrit, 10Release-Engineering-Team, 10observability: Improve monitoring of Gerrit's memory usage - https://phabricator.wikimedia.org/T270521 (10hashar) [08:29:49] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO, 10Operations, and 2 others: Add prometheus exporter to Gerrit - https://phabricator.wikimedia.org/T184086 (10hashar) [08:49:38] 10Continuous-Integration-Infrastructure, 10Tracking-Neverending: Have unit tests of all wmf deployed extensions pass when installed together, in both PHP-Zend and HHVM (tracking) - https://phabricator.wikimedia.org/T69216 (10Nikerabbit) [10:18:05] 10Continuous-Integration-Infrastructure, 10Gerrit, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (2020-10-01 to 2020-12-31 (Q2)): Zuul not running tests for gerrit change 650524 - https://phabricator.wikimedia.org/T270524 (10hashar) [10:18:07] 10Gerrit, 10Release-Engineering-Team-TODO: Gerrit errors on change #650396, missing blob - https://phabricator.wikimedia.org/T270472 (10hashar) [10:18:09] 10Continuous-Integration-Infrastructure, 10Gerrit, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (2020-10-01 to 2020-12-31 (Q2)): Zuul not running tests for gerrit change 650524 - https://phabricator.wikimedia.org/T270524 (10hashar) Tyler pointed that it worked properl... [10:19:32] 10Gerrit, 10Release-Engineering-Team-TODO: Gerrit errors on change #650396, missing blob - https://phabricator.wikimedia.org/T270472 (10hashar) From T270524#6702561 the Gerrit event lacks the `comment-added` message because the reference can not be seen. [10:20:37] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO: Upgrade Gerrit to 3.3 - https://phabricator.wikimedia.org/T262241 (10hashar) [10:20:40] 10Gerrit, 10Release-Engineering-Team-TODO: Gerrit errors on change #650396, missing blob - https://phabricator.wikimedia.org/T270472 (10hashar) [10:21:27] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO: Upgrade Gerrit to 3.3 - https://phabricator.wikimedia.org/T262241 (10hashar) [10:21:29] 10Gerrit, 10Release-Engineering-Team (Development services): Switch Gerrit from Java 8 to Java 11 - https://phabricator.wikimedia.org/T268225 (10hashar) [10:23:47] 10Phabricator: Make sure anti-vandalism features are up to snuff - https://phabricator.wikimedia.org/T84 (10Aklapper) [10:42:09] 10Phabricator: Make sure anti-vandalism features are up to snuff - https://phabricator.wikimedia.org/T84 (10Aklapper) [10:45:26] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)): Upgrade Gerrit to 3.3 - https://phabricator.wikimedia.org/T262241 (10hashar) The rough order: * **blocker** upgrade to Java 11 T268225 * If at all possible setup CI jobs using Bazeli... [10:57:57] 10Phabricator: Decide how to distribute phabricator badges - https://phabricator.wikimedia.org/T196390 (10DannyS712) It seems like everything but the trust and safety badge (https://phabricator.wikimedia.org/badges/view/13/) is restricted - can I suggest restricting that one too? [11:03:17] 10Phabricator, 10User-DannyS712: Exception - Testing for capability "interact" on an object ("PhabricatorBadgesBadge") which does not support that capability. - https://phabricator.wikimedia.org/T270696 (10DannyS712) [11:04:03] 10Phabricator, 10User-DannyS712: Exception - Testing for capability "interact" on an object ("PhabricatorBadgesBadge") which does not support that capability. - https://phabricator.wikimedia.org/T270696 (10DannyS712) [11:10:09] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10incubator.wikimedia.org, 10I18n: Allow creating an independent "incubator wiki" instead of hosting all new wikis in one Incubator wiki with prefixes - https://phabricator.wikimedia.org/T228745 (10Exec8) I would agree that... [11:12:10] 10Phabricator, 10User-DannyS712: Combine Jeblad's Herald rules 252, 253, and 254 - https://phabricator.wikimedia.org/T265520 (10DannyS712) @jeblad can you take a look at this? [11:24:24] 10Phabricator, 10User-DannyS712: Update herald rules following T259156 - https://phabricator.wikimedia.org/T270697 (10DannyS712) [11:24:48] 10Phabricator, 10User-DannyS712: Update herald rules following T259156 - https://phabricator.wikimedia.org/T270697 (10DannyS712) See also {T108586} [11:43:26] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10incubator.wikimedia.org, 10I18n: Allow creating an independent "incubator wiki" instead of hosting all new wikis in one Incubator wiki with prefixes - https://phabricator.wikimedia.org/T228745 (10Amire80) Having a separate... [11:49:55] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO (2020-10-01 to 2020-12-31 (Q2)), 10git-protocol-v2: Gerrit out of heap - https://phabricator.wikimedia.org/T263008 (10hashar) 05Open→03Stalled >>! In T263008#6707707, @Stashbot wrote: > {nav icon=file, name=Men... [11:50:05] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO (2020-10-01 to 2020-12-31 (Q2)), 10git-protocol-v2: Gerrit out of heap - https://phabricator.wikimedia.org/T263008 (10hashar) [11:50:07] 10Gerrit, 10Release-Engineering-Team (Development services): Switch Gerrit from Java 8 to Java 11 - https://phabricator.wikimedia.org/T268225 (10hashar) [11:50:09] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)): Upgrade Gerrit to 3.3 - https://phabricator.wikimedia.org/T262241 (10hashar) [11:51:53] 10Phabricator, 10User-DannyS712: Combine Jeblad's Herald rules 252, 253, and 254 - https://phabricator.wikimedia.org/T265520 (10Peachey88) I don't believe @jeblad is active anymore, example on their phab userpage or on meta https://meta.wikimedia.org/w/index.php?title=User%3AJeblad&type=revision&diff=20300956&... [11:55:29] (03PS1) 10Hashar: Review access change [wikimedia/toolhub] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651487 [11:55:58] (03PS2) 10Hashar: Permissions for l10n-bot [wikimedia/toolhub] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651487 (https://phabricator.wikimedia.org/T269469) [11:57:34] (03CR) 10Hashar: "That would let l10n-bot to vote and submit the changes. I have copy pasted from https://gerrit.wikimedia.org/r/admin/repos/labs/tools,acc" [wikimedia/toolhub] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651487 (https://phabricator.wikimedia.org/T269469) (owner: 10Hashar) [11:57:50] (03PS2) 10Hashar: Add gate-and-submit-l10n to Toolhub [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [11:57:58] (03CR) 10jerkins-bot: [V: 04-1] Add gate-and-submit-l10n to Toolhub [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [11:59:55] (03CR) 10Hashar: "We need the bot to be granted the right to submit which is done by: https://gerrit.wikimedia.org/r/c/wikimedia/toolhub/+/651487" [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [12:00:39] 10Continuous-Integration-Config, 10Toolhub, 10translatewiki.net, 10Language-Team (Language-2020-October-December), 10Patch-For-Review: Add Toolhub to translatewiki.net - https://phabricator.wikimedia.org/T269469 (10hashar) [12:17:51] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10incubator.wikimedia.org, 10I18n: Allow creating an independent "incubator wiki" instead of hosting all new wikis in one Incubator wiki with prefixes - https://phabricator.wikimedia.org/T228745 (10Gnangarra) >>! In T228745#... [12:19:12] 10Phabricator, 10User-DannyS712: Update herald rules following T259156 - https://phabricator.wikimedia.org/T270697 (10Aklapper) 05Open→03Resolved a:03Aklapper Thanks a lot for catching this! (I wonder if there is any feasible, technical, semi-automated way to identify such things, hmm.) I archived both H... [12:27:51] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10incubator.wikimedia.org, 10I18n: Allow creating an independent "incubator wiki" instead of hosting all new wikis in one Incubator wiki with prefixes - https://phabricator.wikimedia.org/T228745 (10Theklan) >>! In T228745#67... [12:47:32] 10Phabricator, 10User-DannyS712: Combine Jeblad's Herald rules 252, 253, and 254 (or disable them if not used anymore) - https://phabricator.wikimedia.org/T265520 (10Aklapper) a:05jeblad→03None [12:47:59] 10Phabricator, 10User-DannyS712: Combine Jeblad's Herald rules 252, 253, and 254 (or disable them if not used anymore) - https://phabricator.wikimedia.org/T265520 (10Aklapper) 05Open→03Resolved a:03Aklapper I disabled those three Herald rules as the account seems to be inactive. [12:49:49] 10Phabricator: Make sure anti-vandalism features are up to snuff - https://phabricator.wikimedia.org/T84 (10Aklapper) [12:52:38] 10Phabricator, 10User-Zppix: Determine if H207 is still needed - https://phabricator.wikimedia.org/T270700 (10Majavah) [13:00:29] 10Phabricator, 10translatewiki.net: Make it easy to query tasks for new project requests for translatewiki.net (create anew subproject?) - https://phabricator.wikimedia.org/T270701 (10Nikerabbit) [13:02:18] 10Phabricator, 10User-Zppix: Determine if H207 is still needed - https://phabricator.wikimedia.org/T270700 (10Zppix) Delete it [13:03:06] 10Phabricator, 10User-Zppix: Determine if H207 is still needed - https://phabricator.wikimedia.org/T270700 (10Aklapper) Thanks for catching this! (...plus folks should really watch projects or become members, instead of parsing random fields for random strings. :P [13:03:37] 10Phabricator, 10User-Zppix: Determine if H207 is still needed - https://phabricator.wikimedia.org/T270700 (10Aklapper) >>! In T270700#6708112, @Zppix wrote: > Delete it Impossible (in general). Plus it's a personal rule so you should be able to disable the rule. [13:04:58] 10Phabricator, 10User-Zppix: Determine if H207 is still needed - https://phabricator.wikimedia.org/T270700 (10Zppix) 05Open→03Resolved Disabled [13:11:37] 10Phabricator, 10Project-Admins, 10translatewiki.net: Make it easy to query tasks for new project requests for translatewiki.net (create anew subproject?) - https://phabricator.wikimedia.org/T270701 (10Majavah) [13:35:03] (03PS1) 10Gehel: WDQS builds should use the standard maven-java8 docker image. [integration/config] - 10https://gerrit.wikimedia.org/r/651528 [13:36:26] (03PS1) 10Gehel: The java8-wikidata-query-rdf image is no longer used and can be removed. [integration/config] - 10https://gerrit.wikimedia.org/r/651529 [13:37:29] hashar: I'm trying to do some cleanup to our java builds: https://gerrit.wikimedia.org/r/q/topic:%22java-cleanup%22 [13:37:43] I'm probably going to be lost soon and I'll scream for your help! [13:41:23] (03CR) 10Nikerabbit: Permissions for l10n-bot (031 comment) [wikimedia/toolhub] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651487 (https://phabricator.wikimedia.org/T269469) (owner: 10Hashar) [13:41:46] gehel: ;) [13:42:21] hashar: si tu sais pas quoi faire cet après-midi, je me fais bien un bout de pair programming avec toi! [13:43:38] (03CR) 10Hashar: [C: 03+2] "Thanks. Deployed and ran at https://integration.wikimedia.org/ci/job/php-composer-security-docker/10876/console" [integration/config] - 10https://gerrit.wikimedia.org/r/651282 (https://phabricator.wikimedia.org/T270660) (owner: 10Legoktm) [13:44:08] je tente de nettoyer mon backlog dans gerrit mais c'est une tache sans fin :] [13:45:38] (03Merged) 10jenkins-bot: Run PHP security dependency scans for Shellbox [integration/config] - 10https://gerrit.wikimedia.org/r/651282 (https://phabricator.wikimedia.org/T270660) (owner: 10Legoktm) [13:45:40] he, he ... [13:54:20] 10Phabricator, 10Project-Admins, 10translatewiki.net: Make it easy to query tasks for new project requests for translatewiki.net (create anew subproject?) - https://phabricator.wikimedia.org/T270701 (10Aklapper) I assume this is about the `New projects for translation` column on the #translatewiki.net workbo... [13:55:40] (03CR) 10Hashar: [C: 03+2] "Deployed" [integration/config] - 10https://gerrit.wikimedia.org/r/651528 (owner: 10Gehel) [13:57:06] (03Merged) 10jenkins-bot: WDQS builds should use the standard maven-java8 docker image. [integration/config] - 10https://gerrit.wikimedia.org/r/651528 (owner: 10Gehel) [14:06:19] (03CR) 10Hashar: [C: 03+2] "Thank you so much for cleaning it out :]" [integration/config] - 10https://gerrit.wikimedia.org/r/651529 (owner: 10Gehel) [14:07:31] (03Merged) 10jenkins-bot: The java8-wikidata-query-rdf image is no longer used and can be removed. [integration/config] - 10https://gerrit.wikimedia.org/r/651529 (owner: 10Gehel) [14:11:24] (03PS1) 10Lars Wirzenius: refactor: use subshell instead of pushd/popd [tools/train-dev] - 10https://gerrit.wikimedia.org/r/651532 [14:24:58] hashar: https://gerrit.wikimedia.org/r/c/integration/config/+/651534 [14:25:09] (03PS1) 10Gehel: WDQS build does not need to specify docker image. [integration/config] - 10https://gerrit.wikimedia.org/r/651534 [14:25:11] (03CR) 10jerkins-bot: [V: 04-1] WDQS build does not need to specify docker image. [integration/config] - 10https://gerrit.wikimedia.org/r/651534 (owner: 10Gehel) [14:25:25] (03PS2) 10Gehel: WDQS build does not need to specify docker image. [integration/config] - 10https://gerrit.wikimedia.org/r/651534 [14:27:38] (03CR) 10Hashar: [C: 03+2] "Paired with Guillaume, does not change anything in JJB :)" [integration/config] - 10https://gerrit.wikimedia.org/r/651534 (owner: 10Gehel) [14:28:59] (03Merged) 10jenkins-bot: WDQS build does not need to specify docker image. [integration/config] - 10https://gerrit.wikimedia.org/r/651534 (owner: 10Gehel) [14:38:20] (03PS1) 10Gehel: Removed unused variables from WDQS jobs. [integration/config] - 10https://gerrit.wikimedia.org/r/651536 [14:38:29] (03PS2) 10Gehel: Removed unused variables from WDQS jobs. [integration/config] - 10https://gerrit.wikimedia.org/r/651536 [14:43:24] (03CR) 10Hashar: [C: 03+2] "Paired with Guillaume. None of those variables are used anymore." [integration/config] - 10https://gerrit.wikimedia.org/r/651536 (owner: 10Gehel) [14:44:59] (03Merged) 10jenkins-bot: Removed unused variables from WDQS jobs. [integration/config] - 10https://gerrit.wikimedia.org/r/651536 (owner: 10Gehel) [15:05:06] (03PS1) 10Gehel: Align analytics and WDQS release builds. [integration/config] - 10https://gerrit.wikimedia.org/r/651540 [15:06:20] (03CR) 10Gehel: [C: 04-1] "needs some more work" [integration/config] - 10https://gerrit.wikimedia.org/r/651540 (owner: 10Gehel) [15:21:30] (03PS2) 10Gehel: Align analytics and WDQS release builds. [integration/config] - 10https://gerrit.wikimedia.org/r/651540 [15:22:42] (03CR) 10jerkins-bot: [V: 04-1] Align analytics and WDQS release builds. [integration/config] - 10https://gerrit.wikimedia.org/r/651540 (owner: 10Gehel) [15:26:50] (03PS3) 10Gehel: Align analytics and WDQS release builds. [integration/config] - 10https://gerrit.wikimedia.org/r/651540 [15:30:57] (03CR) 10Hashar: "Paired with Guillaume. Still need to be verified :)" [integration/config] - 10https://gerrit.wikimedia.org/r/651540 (owner: 10Gehel) [15:54:52] (03PS1) 10Thcipriani: Add grep-able comment for overloading "mod" operator [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/651547 [15:58:18] (03CR) 10Thcipriani: [C: 03+2] "merging trivial comment change" [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/651547 (owner: 10Thcipriani) [15:59:13] (03Merged) 10jenkins-bot: Add grep-able comment for overloading "mod" operator [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/651547 (owner: 10Thcipriani) [16:04:57] (03CR) 10Ottomata: [C: 03+1] Align analytics and WDQS release builds. [integration/config] - 10https://gerrit.wikimedia.org/r/651540 (owner: 10Gehel) [16:05:43] (03CR) 10Gehel: [C: 04-1] "still needs more work" [integration/config] - 10https://gerrit.wikimedia.org/r/651540 (owner: 10Gehel) [16:17:34] (03PS1) 10Lars Wirzenius: refactor: move scap.{python,yaml} into new sub-directory subplot/ [tools/scap] - 10https://gerrit.wikimedia.org/r/651552 [16:17:50] (03Abandoned) 10Lars Wirzenius: refactor: move scap.{python,yaml} into new sub-directory subplot/ [tools/scap] - 10https://gerrit.wikimedia.org/r/651552 (owner: 10Lars Wirzenius) [17:11:02] 10Continuous-Integration-Config, 10Release-Engineering-Team, 10Pywikibot, 10Pywikibot-tests: Enable Pywikibot fasttest with Python 3.8 / Python 3.9 - https://phabricator.wikimedia.org/T270389 (10Jdforrester-WMF) 05Open→03Resolved OK, in that case we're done. Please re-open if things break. When you wan... [17:15:21] 10Release-Engineering-Team-TODO, 10WVUI, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Upstream, 10Vue.js: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10Paladox) Gerrit needs a restart I think for that to take effect. [17:16:57] 10Release-Engineering-Team-TODO, 10WVUI, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Upstream, 10Vue.js: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10Paladox) @Dzahn Actually you could see if reloading the plugin works (which doesn't require downtime). https://... [17:17:40] (03CR) 10Ahmon Dancy: [C: 03+2] refactor: use subshell instead of pushd/popd [tools/train-dev] - 10https://gerrit.wikimedia.org/r/651532 (owner: 10Lars Wirzenius) [17:18:31] (03CR) 10Ahmon Dancy: [V: 03+2 C: 03+2] refactor: use subshell instead of pushd/popd [tools/train-dev] - 10https://gerrit.wikimedia.org/r/651532 (owner: 10Lars Wirzenius) [17:21:29] (03CR) 10Ahmon Dancy: [V: 03+2 C: 03+2] refactor: use subshell instead of pushd/popd [tools/train-dev] - 10https://gerrit.wikimedia.org/r/651532 (owner: 10Lars Wirzenius) [17:30:53] 10Project-Admins, 10Security-Team, 10User-RhinosF1: Create acl*miraheze-sysadmins for security tasks we file - https://phabricator.wikimedia.org/T270203 (10sbassett) @RhinosF1 - The #security-team had a chat about this today. We don't really have a problem with people creating custom ACLs for various projec... [17:31:17] 10Project-Admins, 10Security-Team, 10User-RhinosF1: Create acl*miraheze-sysadmins for security tasks we file - https://phabricator.wikimedia.org/T270203 (10sbassett) 05Open→03Resolved p:05Triage→03Low a:03sbassett [17:39:48] (03PS1) 10Lars Wirzenius: chore: update Standards-Version in debian/control [tools/scap] - 10https://gerrit.wikimedia.org/r/651567 [17:45:20] (03CR) 10BryanDavis: "whatever gerrit config magic is needed is ok with me" [wikimedia/toolhub] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651487 (https://phabricator.wikimedia.org/T269469) (owner: 10Hashar) [17:47:28] (03CR) 10Lars Wirzenius: "Debian packaging updates for Scap." [tools/scap] - 10https://gerrit.wikimedia.org/r/651567 (owner: 10Lars Wirzenius) [18:06:43] (03CR) 10Hashar: [C: 03+1] "I never quite understood what Standards-Version enforces really ;D" [tools/scap] - 10https://gerrit.wikimedia.org/r/651567 (owner: 10Lars Wirzenius) [18:09:47] (03CR) 10Hashar: [V: 03+2 C: 03+2] "Thanks Niklas, I think it is fine to have humans be able to CR+2 as well, it might be needed for some corner case." (031 comment) [wikimedia/toolhub] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651487 (https://phabricator.wikimedia.org/T269469) (owner: 10Hashar) [18:10:22] (03CR) 10Hashar: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [18:12:20] (03CR) 10Hashar: [C: 03+2] Add gate-and-submit-l10n to Toolhub [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [18:12:50] (03PS10) 10Umherirrender: Remove semicolon in UnusedUseStatementSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) [18:13:52] (03Merged) 10jenkins-bot: Add gate-and-submit-l10n to Toolhub [integration/config] - 10https://gerrit.wikimedia.org/r/651183 (https://phabricator.wikimedia.org/T269469) (owner: 10Abijeet Patro) [18:16:37] (03CR) 10Nray: [C: 03+2] Edit Repo Config - rebase if necessary [wvui] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651219 (owner: 10DannyS712) [18:17:13] 10Continuous-Integration-Config, 10Toolhub, 10translatewiki.net, 10Language-Team (Language-2020-October-December), 10Patch-For-Review: Add Toolhub to translatewiki.net - https://phabricator.wikimedia.org/T269469 (10hashar) [18:30:02] (03CR) 10Legoktm: "Thanks!" [integration/config] - 10https://gerrit.wikimedia.org/r/651282 (https://phabricator.wikimedia.org/T270660) (owner: 10Legoktm) [18:31:43] 10Continuous-Integration-Config, 10Shellbox, 10serviceops: Set up PHP security scanning for Shellbox - https://phabricator.wikimedia.org/T270660 (10Legoktm) 05Open→03Resolved a:03Legoktm [18:31:46] 10Release-Engineering-Team (Pipeline), 10MW-on-K8s, 10Operations, 10serviceops, 10Release Pipeline (Blubber): Deployment infrastructure for PHP microservices - https://phabricator.wikimedia.org/T261369 (10Legoktm) [18:51:58] (03PS1) 10Ahmon Dancy: Update mariadb repository location [releng/local-charts] - 10https://gerrit.wikimedia.org/r/651572 [19:00:47] 10Phabricator (Upstream), 10Upstream, 10User-zeljkofilipin: Phabricator does not send any e-mail notification when user is awarded a badge - https://phabricator.wikimedia.org/T270405 (10zeljkofilipin) @Dzahn I guess all I wanted to say is that I think the default should be to notify the user when they are aw... [19:04:49] 10Gerrit: Do not expand large files (like package-lock.json) by default when `Expand All` is clicked - https://phabricator.wikimedia.org/T262944 (10zeljkofilipin) [19:05:02] 10Beta-Cluster-Infrastructure: The certificate for en.wikipedia.beta.wmflabs.org expired on 2020-09-14 - https://phabricator.wikimedia.org/T262816 (10zeljkofilipin) [19:06:46] 10Continuous-Integration-Config, 10Release-Engineering-Team, 10Quality-and-Test-Engineering-Team (QTE), 10Platform Team Initiatives (API Integration Tests), 10Testing-Roadblocks: Create and run a suite of end-to-end tests for the Wikimedia environment - https://phabricator.wikimedia.org/T248683 (10zeljkof... [19:13:49] 10Release-Engineering-Team-TODO, 10WVUI, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Upstream, 10Vue.js: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10Dzahn) @Paladox Thanks for that, I tried but: ` ssh -p 29418 dzahn@gerrit.wikimedia.org gerrit plugin reload f... [19:15:48] 10Release-Engineering-Team-TODO, 10WVUI, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Upstream, 10Vue.js: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10Paladox) Need to enable https://github.com/GerritCodeReview/gerrit/blob/774100cf7e3b28af0087f77baf83b0da9c715977... [19:16:11] 10Release-Engineering-Team-TODO, 10WVUI, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Upstream, 10Vue.js: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10Paladox) though I guess its disabled for a reason, so I think restarting gerrit is the only way. [19:39:47] 10Phabricator (Upstream), 10Upstream, 10User-zeljkofilipin: Phabricator does not send any e-mail notification when user is awarded a badge - https://phabricator.wikimedia.org/T270405 (10Dzahn) ACK. I think the difference here might be "a ticket that a user is subscribed to gets awarded a badge" vs "a user... [19:45:48] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10Operations, 10serviceops, and 2 others: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Dzahn) >>! In T245757#6695685, @hnowlan wrote: > mw1265 is now reimaged and poo... [19:48:02] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10Operations, 10serviceops, and 2 others: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Dzahn) [19:55:47] 10Release-Engineering-Team-TODO, 10WVUI, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Upstream, 10Vue.js: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10nnikkhoui) I'm not seeing the changes BUT I realized this is likely my fault, I think `markdown.safehtml` also [... [20:20:14] (03PS3) 10Jforrester: jjb: Switch integration-config jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651290 [20:20:25] (03CR) 10Jforrester: [C: 03+2] "Deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/651290 (owner: 10Jforrester) [20:20:30] (03PS3) 10Jforrester: jjb: [integration-config-qa] Switch from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651291 [20:21:50] (03PS3) 10Jforrester: jjb: [mw-tools-scap-tox-doc-publish] Switch from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651292 [20:21:57] (03PS3) 10Jforrester: jjb: Switch cumin jobs from tox to tox-buster image [integration/config] - 10https://gerrit.wikimedia.org/r/651293 [20:22:02] (03Merged) 10jenkins-bot: jjb: Switch integration-config jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651290 (owner: 10Jforrester) [20:23:24] (03CR) 10Jforrester: [C: 03+2] jjb: [integration-config-qa] Switch from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651291 (owner: 10Jforrester) [20:24:34] (03Merged) 10jenkins-bot: jjb: [integration-config-qa] Switch from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651291 (owner: 10Jforrester) [20:25:46] (03CR) 10Jforrester: [C: 03+2] jjb: [mw-tools-scap-tox-doc-publish] Switch from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651292 (owner: 10Jforrester) [20:27:19] (03Merged) 10jenkins-bot: jjb: [mw-tools-scap-tox-doc-publish] Switch from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651292 (owner: 10Jforrester) [20:27:30] 10Release-Engineering-Team-TODO, 10WVUI, 10Patch-For-Review, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), and 2 others: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10thcipriani) >>! In T269300#6708965, @nnikkhoui wrote: > @Dzahn I'm not seeing the changes BUT I realiz... [20:27:46] (03PS2) 10Jforrester: jjb: [labs-tools-wikibugs2-autopull] Update URL [integration/config] - 10https://gerrit.wikimedia.org/r/651555 [20:35:09] (03CR) 10Jforrester: [C: 03+2] jjb: [labs-tools-wikibugs2-autopull] Update URL [integration/config] - 10https://gerrit.wikimedia.org/r/651555 (owner: 10Jforrester) [20:37:03] (03Merged) 10jenkins-bot: jjb: [labs-tools-wikibugs2-autopull] Update URL [integration/config] - 10https://gerrit.wikimedia.org/r/651555 (owner: 10Jforrester) [20:38:16] (03PS4) 10Jforrester: jjb: Switch cumin jobs from tox to tox-buster image [integration/config] - 10https://gerrit.wikimedia.org/r/651293 [20:44:48] 10Release-Engineering-Team-TODO, 10WVUI, 10Patch-For-Review, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), and 2 others: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10thcipriani) >>! In T269300#6709044, @thcipriani wrote: >>>! In T269300#6708965, @nnikkhoui wrote: >> @... [20:48:52] (03PS3) 10Jforrester: jjb: Switch tox*publish jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651294 [20:49:02] (03PS3) 10Jforrester: jjb: Switch remaining jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651295 [20:56:33] (03CR) 10Jforrester: [C: 04-1] "Jobs get stuck on `/src/.tox/py37-tests/bin/python -m pip install '.[tests]' >.tox/py37-tests/log/py37-sphinx-1.log`" [integration/config] - 10https://gerrit.wikimedia.org/r/651293 (owner: 10Jforrester) [20:57:18] (03PS4) 10Jforrester: jjb: Switch tox*publish jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651294 [21:00:49] (03CR) 10Jforrester: [C: 03+2] jjb: Switch tox*publish jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651294 (owner: 10Jforrester) [21:00:59] 10Release-Engineering-Team-TODO, 10WVUI, 10Patch-For-Review, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), and 2 others: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10nnikkhoui) @thcipriani okay so this was //definitely// a case of PEBCAK haha I was trying to reference... [21:01:01] (03PS4) 10Jforrester: jjb: Switch remaining jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651295 [21:03:06] (03Merged) 10jenkins-bot: jjb: Switch tox*publish jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651294 (owner: 10Jforrester) [21:03:11] 10Release-Engineering-Team-TODO, 10WVUI, 10Patch-For-Review, 10Readers-Web-Backlog (Kanbanana-FY-2020-21), and 2 others: Modify readme.md for gerrit markdown - https://phabricator.wikimedia.org/T269300 (10thcipriani) >>! In T269300#6709111, @nnikkhoui wrote: > I just uploaded a new patch aaaand i see it wo... [21:08:25] (03CR) 10Jforrester: [C: 03+2] jjb: Switch remaining jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651295 (owner: 10Jforrester) [21:10:15] (03Merged) 10jenkins-bot: jjb: Switch remaining jobs from tox to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651295 (owner: 10Jforrester) [21:12:32] (03PS3) 10Jforrester: dockerfiles: [tox-labs-striker] Migrate to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651296 [21:12:43] (03CR) 10Jforrester: [C: 03+2] dockerfiles: [tox-labs-striker] Migrate to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651296 (owner: 10Jforrester) [21:13:20] (03PS3) 10Jforrester: jjb: Switch striker jobs to one based off buster [integration/config] - 10https://gerrit.wikimedia.org/r/651297 [21:14:54] (03Merged) 10jenkins-bot: dockerfiles: [tox-labs-striker] Migrate to tox-buster [integration/config] - 10https://gerrit.wikimedia.org/r/651296 (owner: 10Jforrester) [21:16:29] !log Docker: Building and publoshing tox-labs-striker:0.5.0 [21:16:31] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [21:33:57] (03CR) 10Jforrester: [C: 04-1] "Fails in the same was as on stretch, except on buster it actually notices and errors?" [integration/config] - 10https://gerrit.wikimedia.org/r/651297 (owner: 10Jforrester) [21:35:07] 10Release-Engineering-Team, 10serviceops, 10Patch-For-Review: Replace production deployment servers and update them to Buster - https://phabricator.wikimedia.org/T265963 (10Dzahn) php-readline 2:7.2+69+0~20190215163918.14+stretch~1.gbpfa617b+wmf1 is installed on the new servers now after deploying https://... [21:42:57] 10Release-Engineering-Team, 10serviceops, 10PHP 7.2 support, 10Patch-For-Review: Drop PHP 7.2 support from MediaWiki master branch, once Wikimedia production is on 7.3 - https://phabricator.wikimedia.org/T261872 (10Reedy) [21:58:40] 10MediaWiki-Releasing, 10Security: Release MediaWiki 1.31.11/1.35.1 - https://phabricator.wikimedia.org/T263802 (10sbassett) [21:58:44] 10MediaWiki-Releasing, 10Security, 10user-sbassett: Write and send supplementary release announcement for extensions and skins with security patches (1.31.11/1.35.1) - https://phabricator.wikimedia.org/T263810 (10sbassett) 05Open→03Resolved Sent. https://lists.wikimedia.org/pipermail/mediawiki-announce/... [21:58:47] 10MediaWiki-Releasing, 10Security, 10user-sbassett: Write and send supplementary release announcement for extensions and skins with security patches (1.31.11/1.35.1) - https://phabricator.wikimedia.org/T263810 (10sbassett) [22:31:11] (03PS2) 10Jforrester: build: Upgrade phpunit to ^9.5 for PHP 8.0 compatibility [integration/docroot] - 10https://gerrit.wikimedia.org/r/651195 [22:32:35] (03CR) 10jerkins-bot: [V: 04-1] build: Upgrade phpunit to ^9.5 for PHP 8.0 compatibility [integration/docroot] - 10https://gerrit.wikimedia.org/r/651195 (owner: 10Jforrester) [22:36:27] (03PS1) 10Jforrester: Zuul: [integration/docroot] Only test PHP 7.3+ from now on [integration/config] - 10https://gerrit.wikimedia.org/r/651639 [22:38:11] (03CR) 10Jforrester: [C: 03+2] Zuul: [integration/docroot] Only test PHP 7.3+ from now on [integration/config] - 10https://gerrit.wikimedia.org/r/651639 (owner: 10Jforrester) [22:39:42] (03Merged) 10jenkins-bot: Zuul: [integration/docroot] Only test PHP 7.3+ from now on [integration/config] - 10https://gerrit.wikimedia.org/r/651639 (owner: 10Jforrester) [22:40:15] !log Zuul: [integration/docroot] Only test PHP 7.3+ from now on [22:40:17] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [22:40:39] (03CR) 10Jforrester: "recheck" [integration/docroot] - 10https://gerrit.wikimedia.org/r/651195 (owner: 10Jforrester) [23:17:17] legoktm: Because you'll be interested too: With everything in flight in some form (dbal, pimple, less.php, and monolog 2.1.1), core now passes `composer check-platform-reqs` on PHP 8.0 for me. Hurrah. [23:17:28] wooooo [23:17:36] <333 [23:17:39] However. [23:17:40] Fancy! [23:17:46] That doesn't mean it /works/. [23:17:52] Hahah [23:17:57] But it means it has a chance of working... [23:18:03] More likely than before the bumps... [23:18:08] Especially, given that the last monolog patch I merged broke everything, I'm not tempted to merge that patch. [23:18:12] True. :-) [23:18:19] https://paste.debian.net/plain/1178008 is what I have been working on today [23:18:21] I dun fscked up on that one though [23:18:42] James_F: hey if there are no exceptions in logstash, that means it's all working, right? [23:18:45] Reedy: Yes, but I was the merger, so the onus is on me. [23:18:50] legoktm: Well, indeed. :-) oops. [23:18:58] legoktm: Fancy. :-) [23:21:14] not sure if you also saw https://gerrit.wikimedia.org/r/651629 [23:22:59] Oh, weighting of updates? Interesting. [23:23:46] legoktm: Do we want a "minimal changes in old branches" concept? If so, is now a good time to blow up your plans and inserting it as a need? [23:24:34] so each branch will have its own config, so it would only upgrade those packages explicitly set for that branch [23:24:42] or did you mean disabling the fixers for other branches? [23:25:16] The latter. [23:25:45] E.g. if we're pushing codesniffer to old branches for 7.3 support, we probably (?) don't want to also auto-fix the repos. [23:26:00] Or maybe we do? [23:26:15] * James_F is worried about bumpy 7.3 backports. [23:26:18] Err. [23:26:32] * James_F is worried about bumpy REL1_31 backports once we try to fix 7.3 testing support.. [23:26:42] hmm [23:26:51] Maybe I'm being pessimistic? [23:27:16] I'm genuinely not sure [23:27:23] * James_F either. [23:27:54] I think I would favor less fixes in branches, if a backport needs some fixing, just cherry-pick more stuff over [23:28:25] Yeah. [23:28:40] OTOH, if the patches land in master with auto-fixes, it'd be nice to have the same happen again. [23:28:47] * James_F shrugs. [23:30:11] legoktm: Also also T270632 would be nice. ;-) [23:30:12] T270632: Provide AST on the image on which LibUp runs so it can handle phan - https://phabricator.wikimedia.org/T270632 [23:30:20] * James_F just exists to be demanding. [23:30:54] I have a half-baked plan to just drop our custom docker image and only use the CI ones but that'll happen after branch support [23:31:39] Oh, deffo. [23:43:34] hi relengers! [23:43:56] over in fr-tech-land, we're wondering if we can get rid of some complexity in our CRM test setup [23:43:58] (03CR) 10DannyS712: "@Nray for refs/meta/config you need to set V+2 and merge manually" [wvui] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/651219 (owner: 10DannyS712) [23:44:45] we created a somewhat indirect setup step initially [23:45:17] in order to not need super permission in the db [23:50:21] so the CI infrastructure pre-creates 3 dbs for us, then hands off to a script that just has permissions to create stuff in those dbs [23:51:12] DELETE ALL THE THINGS [23:51:13] anyway, now that CI is dockerized, can our scripts create dbs? [23:53:59] oh hey, looks like the civicrm image is already running the create script under the same user as the populate script [23:54:09] so there you go, I guess we CAN do it all in one [23:55:28] 10Phabricator, 10User-DannyS712: Update herald rules following T259156 - https://phabricator.wikimedia.org/T270697 (10DannyS712) Not sure if there is a semi-automated way, I just get really bored some times and go looking through herald rules to see if anything should be turned off [23:55:42] tech debt carried forward? [23:58:51] 10Beta-Cluster-Infrastructure, 10Release-Engineering-Team, 10Operations, 10observability, 10User-DannyS712: Beta cluster logstash down - https://phabricator.wikimedia.org/T268200 (10DannyS712) >>! In T268200#6708351, @colewhite wrote: > Hi @DannyS712 > > As of this writing, logs are flowing in deployme...