[00:51:20] (03CR) 10Krinkle: [C: 03+2] zuul: Add node10 to experimental pipeline of npm-docker template [integration/config] - 10https://gerrit.wikimedia.org/r/485726 (https://phabricator.wikimedia.org/T211784) (owner: 10Krinkle) [00:53:07] (03Merged) 10jenkins-bot: zuul: Add node10 to experimental pipeline of npm-docker template [integration/config] - 10https://gerrit.wikimedia.org/r/485726 (https://phabricator.wikimedia.org/T211784) (owner: 10Krinkle) [00:56:19] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review: Castor: mediawiki-core-qunit-jessie node_modules cache ineffective - https://phabricator.wikimedia.org/T159591 (10Krinkle) 05Open→03Resolved a:03Krinkle Castor strategy for npm was changed from caching `node_modul... [00:56:53] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review: Castor: mediawiki-core-qunit-jessie node_modules cache ineffective - https://phabricator.wikimedia.org/T159591 (10Krinkle) [00:56:56] 10Continuous-Integration-Config, 10Discovery, 10Discovery-Portal-Backlog, 10Wikimedia-Portals: CI tests on wikimedia/portals repo: cache node_modules to save time - https://phabricator.wikimedia.org/T152386 (10Krinkle) 05Stalled→03Resolved a:03Krinkle Per T159591#4897562. node/npm jobs use castor and... [00:58:13] 10Continuous-Integration-Infrastructure, 10VisualEditor, 10User-Ryasmeen: Docker headless Chrome running is sometimes silently terminating on start(?), making it impossible to merge code in VE - https://phabricator.wikimedia.org/T200347 (10Krinkle) 05Open→03Resolved a:03Krinkle Fixed by 4142f9c7f41d, T... [01:07:15] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10Jenkins, 10User-zeljkofilipin: Jenkins job for npm-test fail due to node-gyp requiring python2.7 - https://phabricator.wikimedia.org/T213944 (10Krinkle) [01:07:33] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10Jenkins, 10User-zeljkofilipin: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10Krinkle) [01:08:02] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10Jenkins, 10User-zeljkofilipin: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10Krinkle) [01:08:04] 10Continuous-Integration-Config, 10JavaScript, 10Patch-For-Review: Upgrade all CI jobs from node6/npm3 to node10/npm6 across all projects - https://phabricator.wikimedia.org/T211784 (10Krinkle) [01:12:49] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10Jenkins, 10User-zeljkofilipin: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10Krinkle) I'm marking th... [01:13:24] 10Continuous-Integration-Config, 10JavaScript, 10Patch-For-Review: Upgrade all CI jobs from node6/npm3 to node10/npm6 across all projects - https://phabricator.wikimedia.org/T211784 (10Krinkle) p:05Triage→03Normal [01:13:29] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10Jenkins, 10User-zeljkofilipin: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10Krinkle) p:05Normal→... [02:12:35] (03PS1) 10Tim Starling: Remove ForLoopWithTestFunctionCall [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/485732 [03:56:46] (03PS4) 10Krinkle: castor: add --delay-updates to rsync commands [integration/config] - 10https://gerrit.wikimedia.org/r/479558 (https://phabricator.wikimedia.org/T203506) (owner: 10Thcipriani) [03:59:23] (03CR) 10Krinkle: [C: 03+1] castor: add --delay-updates to rsync commands [integration/config] - 10https://gerrit.wikimedia.org/r/479558 (https://phabricator.wikimedia.org/T203506) (owner: 10Thcipriani) [04:22:06] (03CR) 10Krinkle: [C: 03+1] Remove ForLoopWithTestFunctionCall [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/485732 (owner: 10Tim Starling) [04:24:19] 10MediaWiki-Codesniffer: Include ops/wmf-config in "WMF deployed" (Code search) - https://phabricator.wikimedia.org/T214341 (10Krinkle) [05:46:02] 10Gerrit, 10Release-Engineering-Team (Backlog), 10Icinga, 10Operations, 10monitoring: Install "healthcheck" plugin on gerrit - https://phabricator.wikimedia.org/T214326 (10greg) p:05Triage→03Normal The beneficial information, it reports specific status of these sub-parts of Gerrit (from the plugin re... [07:59:34] 10Continuous-Integration-Config, 10LuaSandbox: CI should run `pear package-validate` for PHP extensions with package.xml files - https://phabricator.wikimedia.org/T207686 (10Legoktm) 05Open→03Stalled I intentionally introduced an error in `package.xml`, and ran the validator: ` km@km-pt:/srv/mediawiki/php/... [08:11:27] 10Continuous-Integration-Config, 10LuaSandbox, 10MediaWiki-extensions-Scribunto: Run Scribunto test suite on luasandbox patches - https://phabricator.wikimedia.org/T175068 (10Legoktm) Once we've compiled LuaSandbox, you can get PHP to load the local module by using `php -dextension=./modules/luasandbox.so ..... [08:25:52] (03PS5) 10Addshore: Add more Wikibase extensions to gatedextensions [integration/config] - 10https://gerrit.wikimedia.org/r/459991 (https://phabricator.wikimedia.org/T204153) [08:25:54] 10Continuous-Integration-Config, 10Wikidata, 10Wikidata-Campsite (Wikidata-Campsite-Iteration-∞): Run ArticlePlaceholder tests as part of Wikibase CI - https://phabricator.wikimedia.org/T213815 (10Tarrow) a:03Tarrow [08:31:50] (03PS1) 10Tarrow: Add ArticlePlaceholder to Wikibase Tests [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) [08:35:03] (03PS1) 10Tarrow: Add ArticlePlaceholder to gated extensions [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) [08:35:32] (03CR) 10Tarrow: "This change is ready for review." [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [08:35:52] (03CR) 10Tarrow: "This change is ready for review." [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [08:36:46] (03CR) 10jerkins-bot: [V: 04-1] Add ArticlePlaceholder to gated extensions [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [08:43:46] hashar: am I correct in thinking that deploy_zuul is all I need to do if I change parameter_functions ? [08:44:06] * addshore just updated his jjb and fab setup for CI so he can do things again [08:47:38] (03PS2) 10Tarrow: Add ArticlePlaceholder to gated extensions [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) [08:52:25] 10Continuous-Integration-Config, 10LuaSandbox: CI should run `pear package-validate` for PHP extensions with package.xml files - https://phabricator.wikimedia.org/T207686 (10hashar) I went with throwing an exception in the validation function: ` $ pear package-validate PHP Fatal error: Uncaught Exception in /... [09:02:07] addshore: yup should be good. It would pull the repo on contint1001 then reload zuul [09:04:47] hashar: the best part about `pear package-validate` is that it's even more useless, because the actual PECL server has additional checks that the validator tool won't fail on [09:11:12] legoktm: :((((((( [09:11:48] potentially because the server has a more recent version of it [09:11:57] no [09:12:09] the checks just aren't implemented in the validator [09:12:25] Tim looked into it when he got bitten by this when uploading LuaSandbox to PECL [09:13:19] addshore, hashar: btw, I'd appreciate a look over on my phan upgrade plan before I proceed: https://phabricator.wikimedia.org/T174339#4893380 [09:13:43] and the patch itself https://gerrit.wikimedia.org/r/#/c/485241/ [09:14:29] it turns out that most of the stuff we worked around has been implemented in newer upstream releases :) [09:14:57] iirc addshore asked for a new php-ast version to be made available [09:15:04] seems like phan is tightly coupled with php-ast [09:16:14] if you look at the Dockerfile, you'll see how I'm installing two versions of it at the same time [09:16:38] to support both phan versions right? [09:17:30] yep [09:18:12] and I guess it requires php-ast 0.1.7 which prevents you from using php-ast 0.1.6 from Debian [09:18:28] I think it requires 0.1.5 [09:19:06] ^0.1.5|^1.0.0 [09:19:21] but https://gerrit.wikimedia.org/r/#/c/integration/config/+/485241/2/dockerfiles/mediawiki-phan/Dockerfile.template gets 0.1.7 ;) [09:19:26] (technically phan can use a pure-PHP parser, however the performance in my testing was significantly slower) [09:19:53] yeah, in my gerrit comment I said " I'll probably change the v0.1.7 ast to v1.0.0. since we're going to be using phan 1.2.x+ anyways. " [09:20:14] that way whenever phan increases their requirement, we don't have to do anything in CI to support it [09:36:45] legoktm: and I think you missed including the php wrapper (from COPY php /usr/local/bin/php7.0 ) [09:37:35] ugh [09:37:36] one sec [09:38:05] (03PS3) 10Legoktm: mediawiki-phan: Support newer versions of php-ast and phan [integration/config] - 10https://gerrit.wikimedia.org/r/485241 (https://phabricator.wikimedia.org/T174339) [09:38:12] hashar: thanks, added ^ [09:38:59] legoktm: I am reviewing it :) expect some thing when you wake up tomorrow! [09:40:41] ty! [10:24:58] 10Continuous-Integration-Config, 10Wikidata, 10Patch-For-Review, 10Wikidata-Campsite (Wikidata-Campsite-Iteration-∞): Run ArticlePlaceholder tests as part of Wikibase CI - https://phabricator.wikimedia.org/T213815 (10Tarrow) Patches are up; needs review and then possible deployment [10:38:52] (03CR) 10Hashar: [C: 04-1] "It is a bit hacky but sounds reasonable :) See inline comments for some random nitpicks and shell tips." (0310 comments) [integration/config] - 10https://gerrit.wikimedia.org/r/485241 (https://phabricator.wikimedia.org/T174339) (owner: 10Legoktm) [10:40:47] hashar: thanks :)) [10:40:58] legoktm: and you should sleep !! :] [10:47:44] 10Gerrit, 10Release-Engineering-Team: Upgrade to Gerrit 2.15.9 - https://phabricator.wikimedia.org/T214359 (10Paladox) [11:36:08] (03PS1) 10WMDE-Fisch: Enable reference previews [integration/jenkins] - 10https://gerrit.wikimedia.org/r/485793 (https://phabricator.wikimedia.org/T213415) [12:55:49] (03CR) 10Addshore: [C: 03+2] Add ArticlePlaceholder to Wikibase Tests [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [12:56:00] (03CR) 10Addshore: "I should deploy this first... and verify.." [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [12:57:02] (03CR) 10Addshore: [C: 03+2] "Ahh wait no, I need to merge, then deploy zuul :)" [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [12:58:57] (03Merged) 10jenkins-bot: Add ArticlePlaceholder to Wikibase Tests [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [13:03:15] 10Project-Admins: Convenient Discussions project - https://phabricator.wikimedia.org/T214291 (10Iniquity) [13:03:50] !log reload zuul for Add ArticlePlaceholder to Wikibase Tests [integration/config] - https://gerrit.wikimedia.org/r/485753 [13:03:50] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [13:04:55] (03CR) 10Addshore: [C: 03+2] "validating with PS12 of https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/464096/" [integration/config] - 10https://gerrit.wikimedia.org/r/485753 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [13:05:02] 10Project-Admins: Convenient Discussions project - https://phabricator.wikimedia.org/T214291 (10Iniquity) 05Stalled→03Open [13:12:12] 10Release-Engineering-Team (Kanban), 10Code-Health-Metrics, 10Patch-For-Review, 10User-zeljkofilipin: Code health metrics spike - https://phabricator.wikimedia.org/T207046 (10zeljkofilipin) a:05zeljkofilipin→03None I'm not working on this at the moment, so removing myself as assignee. I will probably c... [13:14:33] 10Release-Engineering-Team (Kanban), 10Code-Health-Metrics, 10Patch-For-Review, 10User-zeljkofilipin: Code health metrics spike - https://phabricator.wikimedia.org/T207046 (10zeljkofilipin) [13:14:46] 10Release-Engineering-Team (Kanban), 10Code-Health-Metrics, 10Patch-For-Review, 10User-zeljkofilipin: Code health metrics spike - https://phabricator.wikimedia.org/T207046 (10zeljkofilipin) [13:14:52] 10Release-Engineering-Team (Kanban), 10Code-Health-Metrics, 10User-zeljkofilipin: Code health metrics spike - https://phabricator.wikimedia.org/T207046 (10zeljkofilipin) [13:21:50] (03PS3) 10Addshore: Add ArticlePlaceholder to gated extensions [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [13:22:12] (03CR) 10Addshore: [C: 03+2] Add ArticlePlaceholder to gated extensions [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [13:23:59] (03Merged) 10jenkins-bot: Add ArticlePlaceholder to gated extensions [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [13:26:55] !log reloaded zuul for Add ArticlePlaceholder to gated extensions [integration/config] - https://gerrit.wikimedia.org/r/485754 [13:26:56] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [13:27:11] 10Release-Engineering-Team (Kanban), 10MediaWiki-Core-Tests, 10User-zeljkofilipin: Gem::InstallError: watir requires Ruby version >= 2.3.0. - https://phabricator.wikimedia.org/T214382 (10zeljkofilipin) a:03zeljkofilipin [13:27:55] (03CR) 10Addshore: [C: 03+2] "Validation with PS4 of https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ArticlePlaceholder/+/260009/" [integration/config] - 10https://gerrit.wikimedia.org/r/485754 (https://phabricator.wikimedia.org/T213815) (owner: 10Tarrow) [13:28:17] 10Release-Engineering-Team (Kanban), 10MediaWiki-Core-Tests, 10User-zeljkofilipin: Gem::InstallError: watir requires Ruby version >= 2.3.0. - https://phabricator.wikimedia.org/T214382 (10zeljkofilipin) p:05Triage→03Normal [13:30:28] 10Release-Engineering-Team (Kanban), 10MediaWiki-Core-Tests, 10User-zeljkofilipin: Gem::InstallError: watir requires Ruby version >= 2.3.0. - https://phabricator.wikimedia.org/T214382 (10zeljkofilipin) [13:32:23] 10Release-Engineering-Team (Kanban), 10MediaWiki-Core-Tests, 10User-zeljkofilipin: Gem::InstallError: watir requires Ruby version >= 2.3.0. - https://phabricator.wikimedia.org/T214382 (10zeljkofilipin) [13:43:15] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10Jenkins, 10User-zeljkofilipin: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10zeljkofilipin) [13:43:55] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10MediaWiki-extensions-ORES, and 3 others: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10zeljkofilipin) [13:50:42] 10Release-Engineering-Team (Kanban), 10MediaWiki-Core-Tests, 10User-zeljkofilipin: Gem::InstallError: watir requires Ruby version >= 2.3.0. - https://phabricator.wikimedia.org/T214382 (10zeljkofilipin) a:05zeljkofilipin→03None [13:51:30] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10MediaWiki-extensions-ORES, and 3 others: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10zeljkofilipin)... [13:56:54] 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-MultimediaViewer, 10Multimedia, 10User-zeljkofilipin: Misconfigured -- Unsupported OS/browser/version/device combo - https://phabricator.wikimedia.org/T214389 (10zeljkofilipin) [13:57:08] 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-MultimediaViewer, 10Multimedia, 10User-zeljkofilipin: Misconfigured -- Unsupported OS/browser/version/device combo - https://phabricator.wikimedia.org/T214389 (10zeljkofilipin) a:05zeljkofilipin→03None [14:05:55] 10Gerrit, 10Release-Engineering-Team (Kanban), 10Developer-Wishlist (2017), 10Patch-For-Review: Enable Gerrit reviewers-by-blame plugin - https://phabricator.wikimedia.org/T101131 (10hashar) 05Resolved→03Declined Thank you @Tgr and @Paladox . We are going to uninstall the plugin entirely based on the f... [14:06:31] 10Release-Engineering-Team (Kanban), 10Release Pipeline, 10serviceops: Expose blubberoid to the public allowing CI in WMCS to be able to reach out as well to it - https://phabricator.wikimedia.org/T212251 (10akosiaris) [14:07:15] 10Gerrit, 10Developer-Advocacy: When uploading a new patch, reviewers should be added automatically - https://phabricator.wikimedia.org/T91190 (10hashar) Eventually the Gerrit plugin from T101131 has been disabled. Adding reviewers solely based on them having the last author does not reflect our reality. We mi... [14:28:43] FYI I just added the whale emoji to github repo of blubber: https://github.com/wikimedia/blubber [14:29:19] (03PS1) 10Addshore: Revert "Add ArticlePlaceholder to gated extensions" [integration/config] - 10https://gerrit.wikimedia.org/r/485817 [14:29:24] (03CR) 10Addshore: [C: 03+2] Revert "Add ArticlePlaceholder to gated extensions" [integration/config] - 10https://gerrit.wikimedia.org/r/485817 (owner: 10Addshore) [14:31:05] (03Merged) 10jenkins-bot: Revert "Add ArticlePlaceholder to gated extensions" [integration/config] - 10https://gerrit.wikimedia.org/r/485817 (owner: 10Addshore) [14:31:55] !log reloaded zuul for REVERT: Add ArticlePlaceholder to gated extensions [integration/config] - https://gerrit.wikimedia.org/r/485754 [14:31:55] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [14:32:01] :( [14:37:37] 10Continuous-Integration-Config, 10Wikidata, 10wikidata-tech-focus, 10Patch-For-Review, 10User-Addshore: Add more Wikibase extensions to gatedextensions - https://phabricator.wikimedia.org/T204153 (10Addshore) [14:37:40] 10Continuous-Integration-Config, 10Wikidata, 10Patch-For-Review, 10Wikidata-Campsite (Wikidata-Campsite-Iteration-∞): Run ArticlePlaceholder tests as part of Wikibase CI - https://phabricator.wikimedia.org/T213815 (10Addshore) 05Open→03Resolved Adding ArticlePlaceholder to gated extensions failed as it... [14:37:48] 10Continuous-Integration-Config, 10Wikidata, 10Patch-For-Review, 10Wikidata-Campsite (Wikidata-Campsite-Iteration-∞): Run ArticlePlaceholder tests as part of Wikibase CI - https://phabricator.wikimedia.org/T213815 (10Addshore) [14:37:52] 10Continuous-Integration-Config, 10Wikidata, 10wikidata-tech-focus, 10Patch-For-Review, 10User-Addshore: Add more Wikibase extensions to gatedextensions - https://phabricator.wikimedia.org/T204153 (10Addshore) [14:37:54] 10Continuous-Integration-Config, 10Release-Engineering-Team, 10Epic: Have dependencies of gated extensions in the gate - https://phabricator.wikimedia.org/T204252 (10Addshore) [14:37:59] 10Continuous-Integration-Config, 10Wikidata, 10wikidata-tech-focus, 10Patch-For-Review, 10User-Addshore: Add more Wikibase extensions to gatedextensions - https://phabricator.wikimedia.org/T204153 (10Addshore) [14:38:05] 10Continuous-Integration-Config, 10Release-Engineering-Team, 10Epic: Have dependencies of gated extensions in the gate - https://phabricator.wikimedia.org/T204252 (10Addshore) [14:38:10] 10Continuous-Integration-Config, 10Wikidata, 10wikidata-tech-focus, 10Patch-For-Review, 10User-Addshore: Add more Wikibase extensions to gatedextensions - https://phabricator.wikimedia.org/T204153 (10Addshore) [14:38:24] addshore: yeah that gated job is a mess ://// [14:38:31] and unsurprisingly easy to break [14:40:08] 10Continuous-Integration-Config, 10MediaWiki-extensions-Scribunto, 10Wikidata: [Task] Add Scribunto to extension-gate in CI - https://phabricator.wikimedia.org/T125050 (10Addshore) We tried to add ArticlePlaceholder to the gated extension list in T213815 but had to revert it due to this ticket not being solv... [14:40:32] hashar: yeh, in my head it should really pull in all of the ext deps of the extensions listed as gated extensions [14:41:18] addshore: possibly. Then there are extensions conflicting with each other, tests that are broken/have side effects etc ;( [14:41:47] merghhhh [14:42:00] yeah it is a huge mess of doom :-/ [14:43:26] the fact that ArticlePlaceholder wasn't in gated caused an UBN last week [14:43:37] (it also wasn't in Wikibase CI though) [14:43:46] :/// [14:44:28] I think we need a job that runs a quibble job for core and all deployed skins and extensions and all deps, and just have it report the state each day [14:44:41] then we can slowly work on any of the "conflicting with each other, tests that are broken/have side effects" etc you said and try to fix them [14:44:48] then maybe one day we can add more things [15:08:06] 10Gerrit, 10Release-Engineering-Team: Gerrit: reviewers-by-blame plugin improvements - https://phabricator.wikimedia.org/T214397 (10thcipriani) [15:08:32] 10Gerrit, 10Release-Engineering-Team: Gerrit: reviewers-by-blame plugin improvements - https://phabricator.wikimedia.org/T214397 (10thcipriani) p:05Triage→03Normal [15:08:46] (03PS3) 10Hashar: docker: add missing control files [integration/config] - 10https://gerrit.wikimedia.org/r/485199 [15:10:55] (03CR) 10jerkins-bot: [V: 04-1] docker: add missing control files [integration/config] - 10https://gerrit.wikimedia.org/r/485199 (owner: 10Hashar) [15:12:27] (03PS4) 10Hashar: docker: add missing control files [integration/config] - 10https://gerrit.wikimedia.org/r/485199 [15:13:49] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:14:08] (03CR) 10Hashar: "Some more tweaks here and there :)" [integration/config] - 10https://gerrit.wikimedia.org/r/485199 (owner: 10Hashar) [15:17:24] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:17:59] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10MediaWiki-extensions-ORES, and 3 others: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10hashar) So the... [15:18:56] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:29:37] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:32:13] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:35:07] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:36:30] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:38:04] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:41:47] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:42:52] twentyafterfour: I have a config change that would be nice to get into today's train -- https://gerrit.wikimedia.org/r/#/c/operations/mediawiki-config/+/485843/ -- It's a bug fix to stop anon edits from Cloud VPS/Toolforge. [15:42:54] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:44:18] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [15:44:22] bd808, so instead we've been blocking part of apple's network, nice [15:44:56] a tiny part of Apple's /8 [16:00:27] (03Abandoned) 10Mholloway: Add CI jobs for new ConfigurableCounters extension [integration/config] - 10https://gerrit.wikimedia.org/r/483527 (owner: 10Mholloway) [16:12:49] (03CR) 10Thiemo Kreuz (WMDE): [C: 03+1] Enable reference previews [integration/jenkins] - 10https://gerrit.wikimedia.org/r/485793 (https://phabricator.wikimedia.org/T213415) (owner: 10WMDE-Fisch) [16:26:15] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [16:26:50] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [16:38:17] 10Continuous-Integration-Config, 10Reading-Infrastructure-Team-Backlog: Basic CI checks for mediawiki/extension/AuthenticatedKeyValueStore - https://phabricator.wikimedia.org/T214417 (10Mholloway) [16:42:20] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [16:42:59] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10User-zeljkofilipin: 5 of the 15 prioritized repositories have at least 1 end-to-end test - https://phabricator.wikimedia.org/T206621 (10zeljkofilipin) [16:43:31] (03PS1) 10Jforrester: Add MediaWiki extension AuthenticatedKeyValueStore [integration/config] - 10https://gerrit.wikimedia.org/r/485848 (https://phabricator.wikimedia.org/T214417) [16:44:07] 10Continuous-Integration-Config, 10Reading-Infrastructure-Team-Backlog, 10Patch-For-Review: Basic CI checks for mediawiki/extension/AuthenticatedKeyValueStore - https://phabricator.wikimedia.org/T214417 (10Jdforrester-WMF) a:03Jdforrester-WMF [16:44:14] (03CR) 10Mholloway: [C: 03+1] Add MediaWiki extension AuthenticatedKeyValueStore [integration/config] - 10https://gerrit.wikimedia.org/r/485848 (https://phabricator.wikimedia.org/T214417) (owner: 10Jforrester) [16:49:36] 10Continuous-Integration-Config, 10Patch-For-Review, 10Reading-Infrastructure-Team-Backlog (Kanban): Basic CI checks for mediawiki/extension/AuthenticatedKeyValueStore - https://phabricator.wikimedia.org/T214417 (10Mholloway) [17:41:00] 10Gerrit, 10Release-Engineering-Team: Gerrit: reviewers-by-blame plugin improvements - https://phabricator.wikimedia.org/T214397 (10hashar) [17:43:32] i've spoken to thcipriani and im considering making a totally new plugin where users can add them selfs as reviewers to projects (then users just press a button that adds them). [18:19:50] 10Continuous-Integration-Infrastructure, 10Gerrit, 10Release-Engineering-Team (Backlog), 10Zuul: Display Zuul status of jobs for a change on Gerrit UI - https://phabricator.wikimedia.org/T214068 (10Paladox) a:03Paladox [18:22:12] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10MediaWiki-extensions-Newsletter, 10MediaWiki-extensions-ORES, and 3 others: Jenkins jobs for npm-test fail on project with deps on node-gyp which requires python2.7 - https://phabricator.wikimedia.org/T213944 (10Krinkle) >>! In... [18:32:25] (03PS2) 10Thcipriani: Scap: Add CI for move back to gerrit [integration/config] - 10https://gerrit.wikimedia.org/r/485331 [18:32:27] (03PS1) 10Thcipriani: Scap: adding postmerge jenkins jobs [integration/config] - 10https://gerrit.wikimedia.org/r/485870 [18:35:24] (03CR) 10Thcipriani: Scap: Add CI for move back to gerrit (033 comments) [integration/config] - 10https://gerrit.wikimedia.org/r/485331 (owner: 10Thcipriani) [18:35:41] Is there a task for "don't have any test wikis in production"? If no, should I make one? [18:36:12] (03CR) 1020after4: [C: 03+2] Scap: adding postmerge jenkins jobs [integration/config] - 10https://gerrit.wikimedia.org/r/485870 (owner: 10Thcipriani) [18:36:18] (03CR) 1020after4: [C: 03+2] Scap: Add CI for move back to gerrit [integration/config] - 10https://gerrit.wikimedia.org/r/485331 (owner: 10Thcipriani) [18:37:57] we need to rebuild all the docker images [18:38:03] (03Merged) 10jenkins-bot: Scap: Add CI for move back to gerrit [integration/config] - 10https://gerrit.wikimedia.org/r/485331 (owner: 10Thcipriani) [18:38:07] for the apt security issue [18:38:52] (03Merged) 10jenkins-bot: Scap: adding postmerge jenkins jobs [integration/config] - 10https://gerrit.wikimedia.org/r/485870 (owner: 10Thcipriani) [18:46:51] !log reloading zuul to deploy https://gerrit.wikimedia.org/r/#/c/integration/config/+/485870/ and https://gerrit.wikimedia.org/r/#/c/integration/config/+/485331/ [18:46:51] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [18:56:08] !log (beta) Update mobileapps to 0aac268 (fix pronunciation detection in mobile-sections T214338) [18:56:10] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [18:56:10] T214338: Pronunciations are not detected in mobile-section anymore - https://phabricator.wikimedia.org/T214338 [18:57:30] * legoktm gets started [18:59:46] (03PS1) 10Legoktm: Rebuild base images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485873 [19:00:31] legoktm: yep, we do, need any help? [19:01:08] it's mostly just going to be testing to make sure all the package upgrades don't break anything... [19:01:17] I suspect there are traps we've accidentally laid for ourselves [19:02:43] !bash < legoktm> I suspect there are traps we've accidentally laid for ourselves [19:02:43] bd808: Stored quip at https://tools.wmflabs.org/bash/quip/AWh28Xs5A1BDhGjCURuH [19:03:16] yeah, there are likely some there, if you want to divide and concur running example-run.sh to test images, I could try a batch [19:03:50] (03PS2) 10Legoktm: Rebuild base images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485873 [19:03:52] (03PS1) 10Legoktm: Rebuild ci support images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485874 [19:03:59] yes, let's [19:04:19] first I'm going to upgrade ci-stretch/ci-jessie independently so we can verify that apt is the correct version before rebuilding the rest [19:04:30] (03CR) 10Legoktm: [C: 03+2] Rebuild base images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485873 (owner: 10Legoktm) [19:04:36] sounds good [19:06:03] https://etherpad.wikimedia.org/p/ci-apt-docker-why [19:06:22] (03Merged) 10jenkins-bot: Rebuild base images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485873 (owner: 10Legoktm) [19:08:39] !log manually docker pulled docker-registry.wikimedia.org/wikimedia-jessie and docker-registry.wikimedia.org/wikimedia-stretch on contint1001 [19:08:40] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [19:09:13] !log rebuilding ci-stretch/ci-jessie images https://gerrit.wikimedia.org/r/485873 [19:09:14] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [19:11:19] 10Release-Engineering-Team, 10Release Pipeline, 10serviceops: Pipeline: provide a way to rebuild all blubber images - https://phabricator.wikimedia.org/T214431 (10thcipriani) [19:14:13] (03PS1) 10Legoktm: jjb: Update ci support images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485875 [19:17:51] km@km-pt ~> docker run --rm -it --entrypoint bash --user=root docker-registry.wikimedia.org/releng/ci-stretch:latest [19:17:52] root@f1e3b5e95073:/# dpkg -l | grep apt [19:17:52] ii apt 1.4.9 amd64 commandline package manager [19:21:41] km@km-pt ~> docker run --rm -it --entrypoint bash --user=root docker-registry.wikimedia.org/releng/ci-jessie:latest [19:21:41] root@d288c3f6c7a3:/# dpkg -l | grep apt [19:21:41] ii apt 1.0.9.8.5 amd64 commandline package manager [19:21:43] :thumbsup: [19:22:39] hrm, so docker-pkg should just automagically rebuild at this point, right? But we don't want that do we? [19:23:01] (03CR) 10Legoktm: [C: 03+2] Rebuild ci support images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485874 (owner: 10Legoktm) [19:23:26] since the base image was updated, it should update the tree depending on the base image. [19:23:30] ^ legoktm ? [19:23:31] thcipriani: we'd need to add changelog entries to everything [19:23:48] but that's easy [19:23:59] sure, but I thought this was the appeal of docker-pkg [19:24:23] maybe I misunderstood something about docker-pkg dependency trees [19:24:33] there are two issues I expect us to run into [19:24:47] 1) package upgrades (usually firefox/chromium) being pulled in, causing tests to fail [19:24:54] (03Merged) 10jenkins-bot: Rebuild ci support images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485874 (owner: 10Legoktm) [19:25:20] 2) middle base images (e.g. npm) being updated but their child images (npm-foo) not being, and now there's some incompatibility we run into to ("the trap") [19:25:42] we just have to live with #1 and if tests fail with new firefox, that's probably a real problem! [19:26:40] my rough understanding is that docker-pkg was supposed to handle updating child images when parent images get new tags? [19:27:09] i.e., running docker-pkg at this point, having bumped the base image changelogs, should retrigger a rebuild and push of all images in the ci repo? [19:27:17] no, I don't think so [19:27:24] you have to add changelog entries to trigger the images to get rebuilt [19:27:34] cc _joe_ [19:28:21] !log rebuilding ci support images for apt security upgrade https://gerrit.wikimedia.org/r/485874 [19:28:22] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [19:31:49] one more bit of minor quibbling about this, what about this in the docs: https://gerrit.wikimedia.org/r/plugins/gitiles/operations/docker-images/docker-pkg/+/master/README.rst#98 [19:32:38] hrm, but while the dockerfile might be different, it might not trigger as being necessary to build [19:32:50] so may not even generate the dockerfile [19:32:51] thcipriani: right, but that only happens if the image is rebuilt, which only happens if there's a new changelog entry [19:33:29] docker-pkg checks the latest changelog version, compares with what's in the `docker images`, and then builds it if it's missing [19:34:13] bd808: got it, merging the config fix before the train [19:34:20] hrm, my mental model of this is likely incorrect in just the right way for me to be confused in this instance. I'll follow your lead and bump changelogs. [19:34:26] thanks twentyafterfour [19:36:07] <_joe_> legoktm: you can rebuild images with the --nightly switch [19:36:11] <_joe_> exactly for this use-case [19:36:20] <_joe_> rebuilding everything fast [19:36:45] <_joe_> and well, in the future (depending on when I have a weekend to do it) docker-pkg will make it easier [19:36:56] relevant: https://octodon.social/@schmonz/101461771236858054 [19:37:18] <_joe_> greg-g: ahah good one indeed [19:37:27] _joe_: what does --nightly do? [19:37:50] <_joe_> legoktm: rebuild all your catalog with the tag in changelog + today's date [19:38:05] <_joe_> so 0.1.0 becomes 0.1.0-20190122 [19:38:31] <_joe_> it is explained in the README and in the inline help IIRC [19:38:41] <_joe_> that's how alex upgraded the production images [19:38:50] <_joe_> :latest will be updated too [19:39:20] <_joe_> hashar was saying earlier that CI images may break when we update all packages [19:39:53] and in the future when we want to change some child image, how will docker-pkg know to pull the nightly image instead of the one in the changelog? [19:42:23] <_joe_> legoktm: sorry, page [19:43:03] (03CR) 10Legoktm: [C: 03+2] "INFO:jenkins_jobs.cli.subcommand.update:Number of jobs updated: 122" [integration/config] - 10https://gerrit.wikimedia.org/r/485875 (owner: 10Legoktm) [19:43:55] <_joe_> legoktm: --nightly is just thought to work on the whole catalog you're proposing, so every image gets rebuilt [19:44:10] <_joe_> legoktm: what you want is a proper upgrade with dependencies method [19:44:14] thcipriani: btw, I've been using: dch --distribution wikimedia --changelog changelog "Rebuild for apt security update" --urgency high [19:44:17] <_joe_> and I want to work on it, yes [19:44:34] <_joe_> legoktm: basically it will be a glorified version of a for loop on dch [19:44:36] <_joe_> :P [19:44:40] _joe_: I mean once we're on the nightly images, it seems unclear on how to go back to normal images [19:44:43] <_joe_> but well, maybe using python-debian [19:45:07] yeah [19:45:09] <_joe_> legoktm: you just publish the new one with a higher version? [19:45:52] (03Merged) 10jenkins-bot: jjb: Update ci support images for apt security upgrade [integration/config] - 10https://gerrit.wikimedia.org/r/485875 (owner: 10Legoktm) [19:45:53] legoktm: added some flags to the command I was using in the etherpad [19:46:22] _joe_: if I create npm-foo:0.2.3, how does it know to pull npm:0.2.2-20190122 instead of npm:0.2.2? [19:46:44] <_joe_> ok, that won't happen, yes [19:47:18] <_joe_> --nightly is just a stopgap [19:47:48] <_joe_> I would prefer to have the time to get to implement an update feature [19:47:54] (03PS1) 10Thcipriani: Docker: Rebuild tox-* images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485882 [19:49:39] <_joe_> in my mind, the workflow is: you launch docker-pkg --select "python3*" update "Security CVE1234", it will create a changelog entry for the images you selected and all the images that depend on them, and submit a patch to gerrit [19:49:57] <_joe_> on gate-and-submit, ci builds said images and publishes them [19:50:38] <_joe_> and in the future, debmonitor will integrate information from debian, the images, and clair to trigger the builds automatically [19:51:53] (03PS1) 10Legoktm: Rebuild php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485883 [19:52:20] <_joe_> I'm 1 quarter away (hopefully, more probably 2) from fully implementing it [19:52:26] 10Release-Engineering-Team (Kanban), 10Release, 10Train Deployments: 1.33.0-wmf.14 deployment blockers - https://phabricator.wikimedia.org/T206668 (10mmodell) a:03mmodell [19:54:41] (03CR) 10Thcipriani: [C: 03+2] Docker: Rebuild tox-* images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485882 (owner: 10Thcipriani) [19:56:26] (03Merged) 10jenkins-bot: Docker: Rebuild tox-* images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485882 (owner: 10Thcipriani) [19:57:19] ^ legoktm want me to wait to run deploy_docker so you can merge the phan stuff? [19:57:32] thcipriani: still locally testing, go ahead [19:57:42] * thcipriani does [19:59:59] !log Updating docker-pkg files on contint1001 for https://gerrit.wikimedia.org/r/#/c/integration/config/+/485882/ [20:00:00] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [20:03:31] we should keep all of the docker image versions in one place in jjb [20:04:23] would save some grepping [20:13:21] (03PS1) 10Legoktm: jjb: Update php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485886 [20:14:32] thcipriani: did your docker_deploy finish? [20:14:42] legoktm: yep, all yours [20:15:01] (03PS2) 10Legoktm: Rebuild php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485883 [20:15:03] (03PS2) 10Legoktm: jjb: Update php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485886 [20:15:13] (03CR) 10Legoktm: [C: 03+2] Rebuild php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485883 (owner: 10Legoktm) [20:17:14] (03Merged) 10jenkins-bot: Rebuild php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485883 (owner: 10Legoktm) [20:18:00] !log rebuilding php, composer, phan images for apt security update https://gerrit.wikimedia.org/r/485883 [20:18:01] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [20:18:33] thcipriani: uh, can you delete /tmp/docker-pkg-build.log ? [20:18:56] sorry, done [20:19:56] funny how that acts as a lock file :p [20:34:44] 10Release-Engineering-Team (Kanban): Article Names do not allow changes - https://phabricator.wikimedia.org/T214433 (10Oradel) [20:35:01] 10Release-Engineering-Team (Kanban): Article Names do not allow changes - https://phabricator.wikimedia.org/T214433 (10Oradel) Fix this [20:37:04] hmm, is that a spammer or a user lost ^^? [20:44:37] paladox: uhm.. hard to tell so far, their edit history is very small though https://meta.wikimedia.org/wiki/Special:CentralAuth?target=Oradel [20:45:23] legoktm: thcipriani: we cant just blindly rebuild the images when one of the ancestor has changed :// [20:45:39] hashar: I've been testing them locally [20:46:04] legoktm: thcipriani: that brings in a bunch of unrelated changes beside the wanted update :\ [20:46:16] they did 2 things.. upload a foto and edit the article of some industrial park in Mexico which is also their user name... and 1 edit to a page of Brazilian musicians.. that is all [20:46:27] not much we can do about that [20:46:32] the update is pretty critical [20:46:49] I have been running example-run.sh to ensure that tests pass for...whatever..master [20:47:07] although I have to redo my tox change :\ [20:47:32] last time it happened I ended up diffing the rebuild images with the previous version. Which is hmm painful [20:48:20] in a more ideal world we'd be able to swap out individual docker image layers, but currently not an option unfortunately :( [20:49:25] having multiple ancestors would be nice [20:49:54] I had shower thoughts about docker for most of 2018. And i really think it is just broken :-\ [20:51:00] ostree!!!! [20:51:26] (03PS1) 10Thcipriani: dockerfiles: tox: update for apt-security update [integration/config] - 10https://gerrit.wikimedia.org/r/485895 [20:51:47] thcipriani: oh no :((( [20:52:25] sigh, yeah :( [20:55:57] (03PS2) 10Thcipriani: dockerfiles: tox: update for apt-security update [integration/config] - 10https://gerrit.wikimedia.org/r/485895 [20:58:17] legoktm: ok, I'm going to rebuild ^ and try again, is deploy_docker done for you? [20:58:25] thcipriani: yep, I'm done [20:58:31] thanks [21:00:29] (03PS3) 10Thcipriani: dockerfiles: tox: update for apt-security update [integration/config] - 10https://gerrit.wikimedia.org/r/485895 [21:02:53] (03CR) 10Thcipriani: [C: 03+2] dockerfiles: tox: update for apt-security update [integration/config] - 10https://gerrit.wikimedia.org/r/485895 (owner: 10Thcipriani) [21:03:21] 10Beta-Cluster-Infrastructure, 10Performance-Team: Move XHGui from tungsten to webperf-002 - https://phabricator.wikimedia.org/T180761 (10Krinkle) a:05Imarlier→03Krinkle [21:04:46] (03Merged) 10jenkins-bot: dockerfiles: tox: update for apt-security update [integration/config] - 10https://gerrit.wikimedia.org/r/485895 (owner: 10Thcipriani) [21:06:00] !log Updating docker-pkg files on contint1001 for https://gerrit.wikimedia.org/r/#/c/integration/config/+/485895/ [21:06:01] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [21:18:07] 10Gerrit, 10Release-Engineering-Team: Gerrit: reviewers-by-blame plugin improvements - https://phabricator.wikimedia.org/T214397 (10Umherirrender) I support "Get reviewers based on paste code-reviews votes" Use Code-Review+1/+2/-1/-2 to find new reviewers, because that would let people review the code and repo... [21:24:00] (03CR) 10Legoktm: [C: 03+2] "INFO:jenkins_jobs.cli.subcommand.update:Number of jobs updated: 40" [integration/config] - 10https://gerrit.wikimedia.org/r/485886 (owner: 10Legoktm) [21:25:13] 10Continuous-Integration-Config, 10Patch-For-Review, 10User-Addshore: Allow use of phan 0.8.5+ in wikimedia CI - https://phabricator.wikimedia.org/T174339 (10Umherirrender) For mediawiki-codesniffer the upstream version is always bundled with a new version. That is needed to install it correctly with compose... [21:26:40] (03Merged) 10jenkins-bot: jjb: Update php, composer, phan images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485886 (owner: 10Legoktm) [21:39:22] slow tests are slow [21:40:10] (03PS1) 10Legoktm: Rebuild java8 images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485959 [21:40:12] (03PS1) 10Legoktm: jjb: Update java8 images for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485960 [21:40:16] 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10Release, 10Train Deployments: 1.33.0-wmf.14 deployment blockers - https://phabricator.wikimedia.org/T206668 (10mmodell) [21:42:15] o/ [21:42:24] i'm trying to provision a mw vagrant instance [21:42:28] i've done thei git-update stuff [21:42:44] am getting a lot of PHP Fatal error: Uncaught Exception: EventBus is not compatible with the current MediaWiki core (version 1.31.0-alpha), it requires: >= 1.32.0. for a lot of extensions [21:47:06] ottomata: you need to update MediaWiki core then [21:48:19] does vagrant git-update do that? [21:48:30] (am re installing it now starting from scratch, will check that first) [21:50:16] theoretically [21:50:23] by reinstalling do you mean a fresh mw-v git clone? [21:50:31] or just vagrant destroy, vagrant up? [21:50:36] because the latter won't fix your problem [21:50:46] destroy and up [21:50:47] you need to git pull in ../vagrant/mediawiki [21:51:02] and now amd doing git-update, then will try proviison [21:52:19] (03CR) 10Umherirrender: [C: 03+1] "libraryupdater would update the following places thanks to autofix" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) (owner: 10Mainframe98) [21:56:51] (03CR) 10Umherirrender: [C: 03+1] "Having the complex part not autofixed is a good idea for me to keep the autofix easy." [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [21:59:52] thanks legoktm i've got it working now i think [22:00:10] i think there was a working change in my mediawiki/ dir that was keeping git-update from pulling [22:01:47] 10Release-Engineering-Team (Kanban), 10docker-pkg, 10serviceops: Some HEAD requests to docker-registry yields 405 Method not allowed - https://phabricator.wikimedia.org/T214441 (10hashar) [22:03:24] blerg. would be nice if these tox jobs worked before I updated them, hard to compare. [22:06:11] 10Release-Engineering-Team (Kanban), 10docker-pkg, 10serviceops: Some HEAD requests to docker-registry yields 405 Method not allowed - https://phabricator.wikimedia.org/T214441 (10hashar) I have subscribed giuseppe/alexandros/fabian seems you know about the Docker registry/nginx. [22:06:14] "master is always buildable" or something [22:06:34] https://gerrit-review.googlesource.com/c/plugins/reviewers-by-blame/+/211189 [22:08:31] it now uses batches instead of individually too :) [22:22:13] 10Release-Engineering-Team (Kanban), 10docker-pkg, 10serviceops: Some HEAD requests to docker-registry yields 405 Method not allowed - https://phabricator.wikimedia.org/T214441 (10hashar) Or just now using: curl --HEAD https://docker-registry.wikimedia.org/v2/releng/composer-php73/manifests/0.1.4 ` nam... [22:25:14] maintenance-disconnect-full-disks build 40145 integration-slave-docker-1017 (/: 95%): OFFLINE due to disk space [22:25:15] maintenance-disconnect-full-disks build 40145 integration-slave-docker-1017 (/: 95%): OFFLINE due to disk space [22:50:13] maintenance-disconnect-full-disks build 40150 integration-slave-docker-1017: OFFLINE due to disk space [22:56:19] (03PS1) 10Thcipriani: jjb: update tox jobs for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485966 [23:00:48] (03PS1) 10ValeryanM: Add ValeryanM to the CI whitelist [integration/config] - 10https://gerrit.wikimedia.org/r/485967 [23:03:41] 10Gerrit, 10Release-Engineering-Team: Gerrit: reviewers-by-blame plugin improvements - https://phabricator.wikimedia.org/T214397 (10Paladox) [23:15:12] maintenance-disconnect-full-disks build 40155 integration-slave-docker-1017: OFFLINE due to disk space [23:19:07] (03CR) 10Thcipriani: [C: 03+2] "INFO:jenkins_jobs.builder:Number of jobs generated: 44" [integration/config] - 10https://gerrit.wikimedia.org/r/485966 (owner: 10Thcipriani) [23:23:02] (03Merged) 10jenkins-bot: jjb: update tox jobs for apt security update [integration/config] - 10https://gerrit.wikimedia.org/r/485966 (owner: 10Thcipriani) [23:24:38] !log integration-slave-docker-1017 clean docker images and repool [23:24:39] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [23:29:09] (03PS1) 10Thcipriani: dockerfiles: apt security update for more tox images [integration/config] - 10https://gerrit.wikimedia.org/r/485971 [23:35:26] (03PS2) 10Thcipriani: dockerfiles: apt security update for more tox images [integration/config] - 10https://gerrit.wikimedia.org/r/485971 [23:42:36] (03CR) 10Thcipriani: [C: 03+2] dockerfiles: apt security update for more tox images [integration/config] - 10https://gerrit.wikimedia.org/r/485971 (owner: 10Thcipriani) [23:44:26] (03Merged) 10jenkins-bot: dockerfiles: apt security update for more tox images [integration/config] - 10https://gerrit.wikimedia.org/r/485971 (owner: 10Thcipriani) [23:46:07] !log Updating (more) tox docker image on contint1001 for https://gerrit.wikimedia.org/r/#/c/integration/config/+/485971/ [23:46:08] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL