[00:00:10] hmm [00:00:10] hmm [00:00:16] that didn't do what i expected [00:00:16] that didn't do what i expected [00:01:07] bd808: so we have to force merge the first few patches [00:01:07] bd808: so we have to force merge the first few patches [00:08:27] I had pushed fixed for the flake8 problems I hadn't noticed before locally [00:08:27] I had pushed fixed for the flake8 problems I hadn't noticed before locally [00:09:29] I got sloppy about running the tests before I uploaded :/ [00:09:29] I got sloppy about running the tests before I uploaded :/ [00:11:54] legoktm: yeah, I knew that would end up being the case. That seemed nicer than making a huge patch with all the Django 2.x changes in it. [00:11:54] legoktm: yeah, I knew that would end up being the case. That seemed nicer than making a huge patch with all the Django 2.x changes in it. [00:12:14] I don't have permissions to force merge in that repo [00:12:14] I don't have permissions to force merge in that repo [00:12:23] should I edit the ACL to grant them temporarily? [00:12:23] should I edit the ACL to grant them temporarily? [00:13:11] sure. I think I can force merge them already, but if you want to do the honors that's fine with me :) [00:13:11] sure. I think I can force merge them already, but if you want to do the honors that's fine with me :) [00:23:29] legoktm: I forced all the ones that needed forcing [00:23:29] legoktm: I forced all the ones that needed forcing [00:31:46] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10bd808) 05Open→03Resolved a:03bd808 Fixed with help from @Legoktm who took care of the magic parts on the integration/config si... [00:31:46] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10bd808) 05Open→03Resolved a:03bd808 Fixed with help from @Legoktm who took care of the magic parts on the integration/config si... [01:32:12] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10bd808) [01:32:12] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10bd808) [01:46:14] (03PS1) 10Legoktm: Bump labs-stiker-tox-docker image version [integration/config] - 10https://gerrit.wikimedia.org/r/481813 [01:46:14] (03PS1) 10Legoktm: Bump labs-stiker-tox-docker image version [integration/config] - 10https://gerrit.wikimedia.org/r/481813 [01:46:34] (03CR) 10Legoktm: [C: 03+2] "Already deployed" [integration/config] - 10https://gerrit.wikimedia.org/r/481813 (owner: 10Legoktm) [01:46:34] (03CR) 10Legoktm: [C: 03+2] "Already deployed" [integration/config] - 10https://gerrit.wikimedia.org/r/481813 (owner: 10Legoktm) [01:49:14] (03Merged) 10jenkins-bot: Bump labs-stiker-tox-docker image version [integration/config] - 10https://gerrit.wikimedia.org/r/481813 (owner: 10Legoktm) [01:49:14] (03Merged) 10jenkins-bot: Bump labs-stiker-tox-docker image version [integration/config] - 10https://gerrit.wikimedia.org/r/481813 (owner: 10Legoktm) [07:27:41] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10E.3) [07:27:41] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10E.3) [08:04:49] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10Peachey88) a:05E.3→03None [08:04:49] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10Peachey88) a:05E.3→03None [08:28:56] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10Aklapper) 05Open→03Declined If nobody explicitly plans to work on this (rather vague) project then there is no need for a project tag, hence declining. If you would like to propose a change in a piece of software (which one? what are... [08:28:56] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10Aklapper) 05Open→03Declined If nobody explicitly plans to work on this (rather vague) project then there is no need for a project tag, hence declining. If you would like to propose a change in a piece of software (which one? what are... [09:10:00] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10E.3) Ok sure. Its not vague, its just making an app the same as the WIkipedia application that is widescreen and has less links. Because my medical opinion is that is safer. It comes from a lot of research. Its not a bug, its a project [09:10:00] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10E.3) Ok sure. Its not vague, its just making an app the same as the WIkipedia application that is widescreen and has less links. Because my medical opinion is that is safer. It comes from a lot of research. Its not a bug, its a project [09:20:22] good morning [09:20:22] good morning [09:35:15] 10Phabricator, 10wikimedia.biterg.io: Closed tickets in Bugzilla migrated without closing event? - https://phabricator.wikimedia.org/T107254 (10hashar) Seems like Phabricator has a migration script to populate the `closedEpoch` `closerPHID` fields: `resources/sql/autopatches/20180208.maniphest.02.populate.php`... [09:35:15] 10Phabricator, 10wikimedia.biterg.io: Closed tickets in Bugzilla migrated without closing event? - https://phabricator.wikimedia.org/T107254 (10hashar) Seems like Phabricator has a migration script to populate the `closedEpoch` `closerPHID` fields: `resources/sql/autopatches/20180208.maniphest.02.populate.php`... [09:36:51] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10hashar) [09:36:51] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10hashar) [09:38:08] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10hashar) 65a7a3685444c062cd1adea3e7fdd026c779ad0f Which bumps the image to `docker-registry.wikimedia.org/releng/tox-labs-striker:0.... [09:38:08] 10Continuous-Integration-Config, 10Striker, 10Patch-For-Review: striker CI fails on Debian Stretch due to cryptography dependency - https://phabricator.wikimedia.org/T212398 (10hashar) 65a7a3685444c062cd1adea3e7fdd026c779ad0f Which bumps the image to `docker-registry.wikimedia.org/releng/tox-labs-striker:0.... [09:42:21] (03CR) 10Hashar: [C: 03+2] "Verified by looking at number of occurences of "ZUUL_VOTING" in the generated XML files. Which must be either no occurences or exactly one" [integration/config] - 10https://gerrit.wikimedia.org/r/481672 (owner: 10Hashar) [09:42:21] (03CR) 10Hashar: [C: 03+2] "Verified by looking at number of occurences of "ZUUL_VOTING" in the generated XML files. Which must be either no occurences or exactly one" [integration/config] - 10https://gerrit.wikimedia.org/r/481672 (owner: 10Hashar) [09:44:57] (03Merged) 10jenkins-bot: jjb: fix duplicate ZUUL parameters in debian-glue jobs [integration/config] - 10https://gerrit.wikimedia.org/r/481672 (owner: 10Hashar) [09:44:57] (03Merged) 10jenkins-bot: jjb: fix duplicate ZUUL parameters in debian-glue jobs [integration/config] - 10https://gerrit.wikimedia.org/r/481672 (owner: 10Hashar) [09:48:27] 10Continuous-Integration-Config: labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) [09:48:27] 10Continuous-Integration-Config: labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) [09:48:30] 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): Fix tests around the toollabs repo and packaging - https://phabricator.wikimedia.org/T210663 (10hashar) [09:48:30] 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): Fix tests around the toollabs repo and packaging - https://phabricator.wikimedia.org/T210663 (10hashar) [09:56:25] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) context ===== For repositories having a Debian package, we run the Je... [09:56:25] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) context ===== For repositories having a Debian package, we run the Je... [09:57:37] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) a:03hashar [09:57:37] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) a:03hashar [09:58:15] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10Aklapper) >>! In T212758#4848266, @E.3 wrote: > its just making an app the same as the WIkipedia application that is widescreen and has less links. If you mean the mobile application when writing "the Wikipedia application", then what is... [09:58:15] 10Project-Admins: #Wikiwide - https://phabricator.wikimedia.org/T212758 (10Aklapper) >>! In T212758#4848266, @E.3 wrote: > its just making an app the same as the WIkipedia application that is widescreen and has less links. If you mean the mobile application when writing "the Wikipedia application", then what is... [10:01:02] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) The idea comes from https://gerrit.wikimedia.org/r/#/c/labs/toollabs/+... [10:01:02] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) The idea comes from https://gerrit.wikimedia.org/r/#/c/labs/toollabs/+... [10:08:19] (03PS5) 10Hashar: debian-glue jobs with hardcoded distributions [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) [10:08:19] (03PS5) 10Hashar: debian-glue jobs with hardcoded distributions [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) [10:09:09] (03CR) 10Hashar: "Added trusty and update the jobs for labs/toollabs. Will rebase and rephrase the commit message to mentions toollabs." [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [10:09:09] (03CR) 10Hashar: "Added trusty and update the jobs for labs/toollabs. Will rebase and rephrase the commit message to mentions toollabs." [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [10:09:45] (03PS6) 10Hashar: debian-glue jobs with hardcoded dists for labs/toollabs [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) [10:09:45] (03PS6) 10Hashar: debian-glue jobs with hardcoded dists for labs/toollabs [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) [10:11:14] 10Continuous-Integration-Config, 10MinusX: minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Tgr) [10:11:14] 10Continuous-Integration-Config, 10MinusX: minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Tgr) [10:11:50] 10Continuous-Integration-Config, 10MinusX: minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Tgr) Possibly caused by https://gerrit.wikimedia.org/r/c/mediawiki/core/+/402166 / {T175794}. [10:11:50] 10Continuous-Integration-Config, 10MinusX: minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Tgr) Possibly caused by https://gerrit.wikimedia.org/r/c/mediawiki/core/+/402166 / {T175794}. [10:19:32] !log updating all debian-glue jobs and creating new ones with hardcoded distributions (trusty, jessie, stretch, unstable) T210780 [10:19:32] !log updating all debian-glue jobs and creating new ones with hardcoded distributions (trusty, jessie, stretch, unstable) T210780 [10:19:34] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [10:19:34] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [10:19:34] T210780: labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 [10:19:34] T210780: labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 [10:21:49] (03CR) 10Hashar: [C: 03+2] debian-glue jobs with hardcoded dists for labs/toollabs [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [10:21:49] (03CR) 10Hashar: [C: 03+2] debian-glue jobs with hardcoded dists for labs/toollabs [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [10:24:31] (03Merged) 10jenkins-bot: debian-glue jobs with hardcoded dists for labs/toollabs [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [10:24:31] (03Merged) 10jenkins-bot: debian-glue jobs with hardcoded dists for labs/toollabs [integration/config] - 10https://gerrit.wikimedia.org/r/481670 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [10:46:02] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) I have send a dummy change in Gerrit that just b... [10:46:02] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) I have send a dummy change in Gerrit that just b... [10:57:07] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) For Trusty the dsc has: ` Changes: toollabs (1... [10:57:07] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) For Trusty the dsc has: ` Changes: toollabs (1... [11:06:21] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10Packaging, 10Upstream: jenkins-debian-glue should run the lintian version from cowbuilder instead of from host - https://phabricator.wikimedia.org/T186494 (10hashar) [11:06:21] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10Packaging, 10Upstream: jenkins-debian-glue should run the lintian version from cowbuilder instead of from host - https://phabricator.wikimedia.org/T186494 (10hashar) [11:06:23] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10Patch-For-Review: debian-glue errors out with: hooks/B90lintian: line 21: realpath: command not found - https://phabricator.wikimedia.org/T193906 (10hashar) [11:06:23] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10Patch-For-Review: debian-glue errors out with: hooks/B90lintian: line 21: realpath: command not found - https://phabricator.wikimedia.org/T193906 (10hashar) [11:13:37] (03CR) 10Hashar: [C: 04-1] "That is due to autopkgtest version 5, the package only has autopkgtest binary and lacks adt-run." [integration/config] - 10https://gerrit.wikimedia.org/r/463507 (owner: 10Legoktm) [11:13:37] (03CR) 10Hashar: [C: 04-1] "That is due to autopkgtest version 5, the package only has autopkgtest binary and lacks adt-run." [integration/config] - 10https://gerrit.wikimedia.org/r/463507 (owner: 10Legoktm) [11:17:49] 10Continuous-Integration-Infrastructure: debian-glue does not support autopkgtest version 5 - https://phabricator.wikimedia.org/T212773 (10hashar) [11:17:49] 10Continuous-Integration-Infrastructure: debian-glue does not support autopkgtest version 5 - https://phabricator.wikimedia.org/T212773 (10hashar) [11:20:37] 10Continuous-Integration-Infrastructure: debian-glue does not support autopkgtest version 5 - https://phabricator.wikimedia.org/T212773 (10hashar) [11:20:37] 10Continuous-Integration-Infrastructure: debian-glue does not support autopkgtest version 5 - https://phabricator.wikimedia.org/T212773 (10hashar) [11:23:41] 10Continuous-Integration-Infrastructure: Upgrade jenkins-debian-glue to v0.19.0 - https://phabricator.wikimedia.org/T212774 (10hashar) [11:23:41] 10Continuous-Integration-Infrastructure: Upgrade jenkins-debian-glue to v0.19.0 - https://phabricator.wikimedia.org/T212774 (10hashar) [11:25:26] 10Release-Engineering-Team, 10Scap, 10Operations, 10Patch-For-Review, 10User-ArielGlenn: Make scap and opcache work consistently together - https://phabricator.wikimedia.org/T211964 (10Joe) a:03Joe [11:25:26] 10Release-Engineering-Team, 10Scap, 10Operations, 10Patch-For-Review, 10User-ArielGlenn: Make scap and opcache work consistently together - https://phabricator.wikimedia.org/T211964 (10Joe) a:03Joe [11:52:12] 10Continuous-Integration-Config: Run minus-x on mediawiki/core for all files - https://phabricator.wikimedia.org/T212746 (10Umherirrender) It was reverted for the moment - T212766 - but the described problem here still exists when reapply. [11:52:12] 10Continuous-Integration-Config: Run minus-x on mediawiki/core for all files - https://phabricator.wikimedia.org/T212746 (10Umherirrender) It was reverted for the moment - T212766 - but the described problem here still exists when reapply. [12:02:39] 10Continuous-Integration-Config, 10MinusX, 10MW-1.33-notes (1.33.0-wmf.12; 2019-01-08), 10Patch-For-Review: minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Umherirrender) It seems MinusX can only work on one directory a... [12:02:39] 10Continuous-Integration-Config, 10MinusX, 10MW-1.33-notes (1.33.0-wmf.12; 2019-01-08), 10Patch-For-Review: minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Umherirrender) It seems MinusX can only work on one directory a... [12:19:59] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10aborrero) >>! In T210780#4848466, @hashar wrote: > * htt... [12:19:59] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10aborrero) >>! In T210780#4848466, @hashar wrote: > * htt... [14:30:41] hashar: hi! I reworked the sonarqube job https://gerrit.wikimedia.org/r/c/integration/config/+/475470, if you have time to look at it this week. cc zeljkof [14:30:41] hashar: hi! I reworked the sonarqube job https://gerrit.wikimedia.org/r/c/integration/config/+/475470, if you have time to look at it this week. cc zeljkof [14:31:03] hi kostajh! [14:31:03] hi kostajh! [14:31:17] happy new year zeljkof :) [14:31:17] happy new year zeljkof :) [14:31:25] happy new year! :) [14:31:25] happy new year! :) [14:31:42] I've added the patch to my todo list for this week [14:31:43] I've added the patch to my todo list for this week [14:32:10] cool [14:32:10] cool [14:45:56] 10Continuous-Integration-Config, 10MinusX, 10MW-1.33-notes (1.33.0-wmf.12; 2019-01-08): minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Jdforrester-WMF) [14:45:56] 10Continuous-Integration-Config, 10MinusX, 10MW-1.33-notes (1.33.0-wmf.12; 2019-01-08): minus-x broke MediaWiki core CI: Too many arguments, expected arguments "command" "path" - https://phabricator.wikimedia.org/T212766 (10Jdforrester-WMF) [15:28:59] 10Continuous-Integration-Config: Have jenkins test (but not automerge) localisation updates patches for apps/android/wikipedia.git - https://phabricator.wikimedia.org/T212786 (10MarcoAurelio) [15:28:59] 10Continuous-Integration-Config: Have jenkins test (but not automerge) localisation updates patches for apps/android/wikipedia.git - https://phabricator.wikimedia.org/T212786 (10MarcoAurelio) [15:45:01] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) > Not sure about git buildpackage/cowbuilder but... [15:45:01] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) > Not sure about git buildpackage/cowbuilder but... [15:54:01] (03PS1) 10Hashar: debian-glue fix Distribution in .changes file [integration/config] - 10https://gerrit.wikimedia.org/r/481870 (https://phabricator.wikimedia.org/T210780) [15:54:01] (03PS1) 10Hashar: debian-glue fix Distribution in .changes file [integration/config] - 10https://gerrit.wikimedia.org/r/481870 (https://phabricator.wikimedia.org/T210780) [15:56:48] 10Continuous-Integration-Config: Have jenkins test (but not automerge) localisation updates patches for apps/android/wikipedia.git - https://phabricator.wikimedia.org/T212786 (10MarcoAurelio) [15:56:48] 10Continuous-Integration-Config: Have jenkins test (but not automerge) localisation updates patches for apps/android/wikipedia.git - https://phabricator.wikimedia.org/T212786 (10MarcoAurelio) [15:58:39] (03CR) 10Hashar: [C: 03+2] debian-glue fix Distribution in .changes file [integration/config] - 10https://gerrit.wikimedia.org/r/481870 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [15:58:39] (03CR) 10Hashar: [C: 03+2] debian-glue fix Distribution in .changes file [integration/config] - 10https://gerrit.wikimedia.org/r/481870 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [16:01:18] (03Merged) 10jenkins-bot: debian-glue fix Distribution in .changes file [integration/config] - 10https://gerrit.wikimedia.org/r/481870 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [16:01:18] (03Merged) 10jenkins-bot: debian-glue fix Distribution in .changes file [integration/config] - 10https://gerrit.wikimedia.org/r/481870 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [16:23:20] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) > There is another way to workaround this, and i... [16:23:20] 10Continuous-Integration-Config, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10cloud-services-team (Kanban): labs/toollabs should always run debian-glue, possibly with multiple distributions - https://phabricator.wikimedia.org/T210780 (10hashar) > There is another way to workaround this, and i... [16:25:44] (03PS1) 10Hashar: labs/toollabs .deb works for Stretch [integration/config] - 10https://gerrit.wikimedia.org/r/481876 (https://phabricator.wikimedia.org/T210780) [16:25:44] (03PS1) 10Hashar: labs/toollabs .deb works for Stretch [integration/config] - 10https://gerrit.wikimedia.org/r/481876 (https://phabricator.wikimedia.org/T210780) [16:27:03] (03CR) 10Hashar: "The Stretch job pass on the dummy change https://gerrit.wikimedia.org/r/#/c/labs/toollabs/+/481835/ :)" [integration/config] - 10https://gerrit.wikimedia.org/r/481876 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [16:27:03] (03CR) 10Hashar: "The Stretch job pass on the dummy change https://gerrit.wikimedia.org/r/#/c/labs/toollabs/+/481835/ :)" [integration/config] - 10https://gerrit.wikimedia.org/r/481876 (https://phabricator.wikimedia.org/T210780) (owner: 10Hashar) [16:28:46] 10Phabricator, 10wikimedia.biterg.io: Closed tickets in Bugzilla migrated without closing event? - https://phabricator.wikimedia.org/T107254 (10bd808) >>! In T107254#4848280, @hashar wrote: > Seems like Phabricator has a migration script to populate the `closedEpoch` `closerPHID` fields: `resources/sql/autopat... [16:28:46] 10Phabricator, 10wikimedia.biterg.io: Closed tickets in Bugzilla migrated without closing event? - https://phabricator.wikimedia.org/T107254 (10bd808) >>! In T107254#4848280, @hashar wrote: > Seems like Phabricator has a migration script to populate the `closedEpoch` `closerPHID` fields: `resources/sql/autopat... [16:32:56] 10Phabricator, 10wikimedia.biterg.io: Closed tickets in Bugzilla migrated without closing event? - https://phabricator.wikimedia.org/T107254 (10hashar) I assumed it populates it due to the presence of `'UPDATE %T SET closedEpoch = %d, closerPHID = %ns WHERE id = %d',`. Then there are few conditions that skips... [16:32:56] 10Phabricator, 10wikimedia.biterg.io: Closed tickets in Bugzilla migrated without closing event? - https://phabricator.wikimedia.org/T107254 (10hashar) I assumed it populates it due to the presence of `'UPDATE %T SET closedEpoch = %d, closerPHID = %ns WHERE id = %d',`. Then there are few conditions that skips... [17:19:04] 10Continuous-Integration-Infrastructure, 10Security-Team: jenkins vulnerability checker is hitting ratelimits - https://phabricator.wikimedia.org/T212794 (10Bawolff) [17:19:04] 10Continuous-Integration-Infrastructure, 10Security-Team: jenkins vulnerability checker is hitting ratelimits - https://phabricator.wikimedia.org/T212794 (10Bawolff) [17:28:09] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Next), 10Wikimedia-Logstash, 10Jenkins, 10Patch-For-Review: Send Jenkins build log and results to ElasticSearch - https://phabricator.wikimedia.org/T78705 (10Dzahn) applied the ferm change. now we have: relforge1001: ACCEPT tcp -... [17:28:09] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Next), 10Wikimedia-Logstash, 10Jenkins, 10Patch-For-Review: Send Jenkins build log and results to ElasticSearch - https://phabricator.wikimedia.org/T78705 (10Dzahn) applied the ferm change. now we have: relforge1001: ACCEPT tcp -... [17:39:13] 10Continuous-Integration-Infrastructure, 10Security-Team: jenkins vulnerability checker is hitting ratelimits - https://phabricator.wikimedia.org/T212794 (10hashar) which rate limit? [17:39:13] 10Continuous-Integration-Infrastructure, 10Security-Team: jenkins vulnerability checker is hitting ratelimits - https://phabricator.wikimedia.org/T212794 (10hashar) which rate limit? [18:11:27] Hi, I tracking https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-hhvm-docker/29307/console ~20 minutes. Can anyone check what happening? Looks like it stopped with working [18:11:27] Hi, I tracking https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-hhvm-docker/29307/console ~20 minutes. Can anyone check what happening? Looks like it stopped with working [18:14:08] I think connectivity issues or something related.. [18:14:08] I think connectivity issues or something related.. [18:16:18] hashar: know anything about the issue reported by Zoranzoki21 ? [18:16:18] hashar: know anything about the issue reported by Zoranzoki21 ? [18:16:43] Jobs sometimes die [18:16:43] Jobs sometimes die [18:17:14] I just killed it [18:17:14] I just killed it [18:17:17] recheck the commit [18:17:17] recheck the commit [18:17:57] Reedy: This should be resubmitted https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FlaggedRevs/+/481658/ [18:17:57] Reedy: This should be resubmitted https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FlaggedRevs/+/481658/ [18:18:07] 29307 was for https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FlaggedRevs/+/481658/ [18:18:08] 29307 was for https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FlaggedRevs/+/481658/ [18:32:35] Reedy: do you still need +sysop on Meta (WMF account)? [18:32:35] Reedy: do you still need +sysop on Meta (WMF account)? [18:57:24] (03PS1) 10Volans: cumin: increase build timeout [integration/config] - 10https://gerrit.wikimedia.org/r/481922 [18:57:24] (03PS1) 10Volans: cumin: increase build timeout [integration/config] - 10https://gerrit.wikimedia.org/r/481922 [18:58:02] (03CR) 10Volans: "Example of failed CI:" [integration/config] - 10https://gerrit.wikimedia.org/r/481922 (owner: 10Volans) [18:58:02] (03CR) 10Volans: "Example of failed CI:" [integration/config] - 10https://gerrit.wikimedia.org/r/481922 (owner: 10Volans) [19:18:18] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: TEC3:O3:O3.1:Q2 Goal - Move Blubberoid, ZoteroV2, and Graphoid through the production CD Pipeline - https://phabricator.wikimedia.org/T205919 (10thcipriani) [19:18:18] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: TEC3:O3:O3.1:Q2 Goal - Move Blubberoid, ZoteroV2, and Graphoid through the production CD Pipeline - https://phabricator.wikimedia.org/T205919 (10thcipriani) [19:18:24] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: Migrate production services to kubernetes using the pipeline - https://phabricator.wikimedia.org/T198901 (10thcipriani) [19:18:24] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: Migrate production services to kubernetes using the pipeline - https://phabricator.wikimedia.org/T198901 (10thcipriani) [19:18:28] 10Release-Engineering-Team (Watching / External), 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), 10Services (watching): Revisit the logging work done on Q1 2017-2018 for the standard pod setup - https://phabricator.wikimedia.org/T207200 (10thcipriani) [19:18:28] 10Release-Engineering-Team (Watching / External), 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), 10Services (watching): Revisit the logging work done on Q1 2017-2018 for the standard pod setup - https://phabricator.wikimedia.org/T207200 (10thcipriani) [19:18:48] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: Migrate production services to kubernetes using the pipeline - https://phabricator.wikimedia.org/T198901 (10thcipriani) [19:18:48] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: Migrate production services to kubernetes using the pipeline - https://phabricator.wikimedia.org/T198901 (10thcipriani) [19:18:54] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: TEC3:O3:O3.1:Q2 Goal - Move Blubberoid, ZoteroV2, and Graphoid through the production CD Pipeline - https://phabricator.wikimedia.org/T205919 (10thcipriani) 05Open→03Resolved... [19:18:54] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: TEC3:O3:O3.1:Q2 Goal - Move Blubberoid, ZoteroV2, and Graphoid through the production CD Pipeline - https://phabricator.wikimedia.org/T205919 (10thcipriani) 05Open→03Resolved... [19:18:58] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10greg) p:05Triage→03Normal [19:18:58] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10greg) p:05Triage→03Normal [19:19:07] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: Migrate production services to kubernetes using the pipeline - https://phabricator.wikimedia.org/T198901 (10greg) [19:19:07] 10Release-Engineering-Team, 10Operations, 10Release Pipeline, 10Core Platform Team Backlog (Watching / External), and 2 others: Migrate production services to kubernetes using the pipeline - https://phabricator.wikimedia.org/T198901 (10greg) [19:19:11] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10greg) [19:19:11] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10greg) [19:21:55] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10thcipriani) [19:21:55] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10thcipriani) [19:22:02] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, eventgate (new service) and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10greg) [19:22:02] 10Release-Engineering-Team, 10Release Pipeline, 10Services, 10serviceops: TEC3:O3:O3.1:Q3 Goal - Move cxserver, citoid, changeprop, eventgate (new service) and ORES (partially) through the production CD Pipeline - https://phabricator.wikimedia.org/T212801 (10greg) [20:26:24] (03PS1) 10Thcipriani: changelog: script upload changelog from jenkins [tools/release] - 10https://gerrit.wikimedia.org/r/481939 [20:26:24] (03PS1) 10Thcipriani: changelog: script upload changelog from jenkins [tools/release] - 10https://gerrit.wikimedia.org/r/481939 [20:30:41] 10Deployments, 10TechCom, 10I18n, 10Language-Team (Language-2019-January-March): Determine desired architecture to update localization strings for Wikimedia - https://phabricator.wikimedia.org/T206694 (10Krinkle) @Nikerabbit Do you think this should be merged with T158360? I can see a use for a "prepare th... [20:30:41] 10Deployments, 10TechCom, 10I18n, 10Language-Team (Language-2019-January-March): Determine desired architecture to update localization strings for Wikimedia - https://phabricator.wikimedia.org/T206694 (10Krinkle) @Nikerabbit Do you think this should be merged with T158360? I can see a use for a "prepare th... [20:51:17] maintenance-disconnect-full-disks build 34368 integration-slave-docker-1021 (/: 96%): OFFLINE due to disk space [20:51:17] maintenance-disconnect-full-disks build 34368 integration-slave-docker-1021 (/: 96%): OFFLINE due to disk space [20:51:17] maintenance-disconnect-full-disks build 34368 integration-slave-docker-1021 (/: 96%): OFFLINE due to disk space [20:51:17] maintenance-disconnect-full-disks build 34368 integration-slave-docker-1021 (/: 96%): OFFLINE due to disk space [21:00:19] maintenance-disconnect-full-disks build 34370 integration-slave-docker-1021: OFFLINE due to disk space [21:00:19] maintenance-disconnect-full-disks build 34370 integration-slave-docker-1021: OFFLINE due to disk space [21:15:11] (03PS1) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:15:11] (03PS1) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:15:54] (03CR) 10jerkins-bot: [V: 04-1] Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 (owner: 10Thcipriani) [21:15:54] (03CR) 10jerkins-bot: [V: 04-1] Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 (owner: 10Thcipriani) [21:18:19] (03PS2) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:18:19] (03PS2) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:21:00] (03CR) 10jerkins-bot: [V: 04-1] Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 (owner: 10Thcipriani) [21:21:00] (03CR) 10jerkins-bot: [V: 04-1] Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 (owner: 10Thcipriani) [21:23:59] (03PS3) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:23:59] (03PS3) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:24:37] (03CR) 10jerkins-bot: [V: 04-1] Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 (owner: 10Thcipriani) [21:24:37] (03CR) 10jerkins-bot: [V: 04-1] Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 (owner: 10Thcipriani) [21:25:16] maintenance-disconnect-full-disks build 34375 integration-slave-docker-1021: OFFLINE due to disk space [21:25:16] maintenance-disconnect-full-disks build 34375 integration-slave-docker-1021: OFFLINE due to disk space [21:28:04] (03PS4) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:28:04] (03PS4) 10Thcipriani: Add dockerfile/job for automating deploy notes [integration/config] - 10https://gerrit.wikimedia.org/r/481947 [21:50:16] maintenance-disconnect-full-disks build 34380 integration-slave-docker-1021: OFFLINE due to disk space [21:50:16] maintenance-disconnect-full-disks build 34380 integration-slave-docker-1021: OFFLINE due to disk space [22:06:26] (03CR) 10Legoktm: changelog: script upload changelog from jenkins (031 comment) [tools/release] - 10https://gerrit.wikimedia.org/r/481939 (owner: 10Thcipriani) [22:06:26] (03CR) 10Legoktm: changelog: script upload changelog from jenkins (031 comment) [tools/release] - 10https://gerrit.wikimedia.org/r/481939 (owner: 10Thcipriani) [22:07:25] 10Diffusion, 10Scoring-platform-team: Rename Diffusion repo from wikiclass->articlequality - https://phabricator.wikimedia.org/T212814 (10awight) [22:07:25] 10Diffusion, 10Scoring-platform-team: Rename Diffusion repo from wikiclass->articlequality - https://phabricator.wikimedia.org/T212814 (10awight) [22:12:44] 10Diffusion, 10Scoring-platform-team: Rename Diffusion repo from wikiclass->articlequality - https://phabricator.wikimedia.org/T212814 (10awight) 05Open→03Resolved a:03awight I was able to self-serve even with my plebian credentials :D [22:12:44] 10Diffusion, 10Scoring-platform-team: Rename Diffusion repo from wikiclass->articlequality - https://phabricator.wikimedia.org/T212814 (10awight) 05Open→03Resolved a:03awight I was able to self-serve even with my plebian credentials :D [22:15:12] maintenance-disconnect-full-disks build 34385 integration-slave-docker-1021: OFFLINE due to disk space [22:15:12] maintenance-disconnect-full-disks build 34385 integration-slave-docker-1021: OFFLINE due to disk space [22:17:24] 10Continuous-Integration-Infrastructure: Rebuild PHP 7.3 containers for new PHP version - https://phabricator.wikimedia.org/T212815 (10Legoktm) [22:17:24] 10Continuous-Integration-Infrastructure: Rebuild PHP 7.3 containers for new PHP version - https://phabricator.wikimedia.org/T212815 (10Legoktm) [22:17:40] Hi, I found a gerrit repo completely empty this morning and I think it might be related to an earlier github outage, which we observe. Meanwhile, anyone feel like watching the update process? https://phabricator.wikimedia.org/source/articlequality/manage/basics/ [22:17:40] Hi, I found a gerrit repo completely empty this morning and I think it might be related to an earlier github outage, which we observe. Meanwhile, anyone feel like watching the update process? https://phabricator.wikimedia.org/source/articlequality/manage/basics/ [22:19:38] Needs a repo admin to hit the button I guess [22:19:39] Needs a repo admin to hit the button I guess [22:19:42] that sounds like we are dependent on github being up.. :? [22:19:42] that sounds like we are dependent on github being up.. :? [22:19:49] 10Continuous-Integration-Config, 10Discovery: Move tox-pyspark docker image to Debian Stretch - https://phabricator.wikimedia.org/T212399 (10EBernhardson) Yes this can move to stretch [22:19:49] 10Continuous-Integration-Config, 10Discovery: Move tox-pyspark docker image to Debian Stretch - https://phabricator.wikimedia.org/T212399 (10EBernhardson) Yes this can move to stretch [22:19:54] i've pressed the button Reedy [22:19:54] i've pressed the button Reedy [22:20:15] :) thank you! [22:20:15] :) thank you! [22:20:26] let's not be dependent on a centralized service by Microsoft? [22:20:26] let's not be dependent on a centralized service by Microsoft? [22:20:48] apparently there was no outage reported, so now my only clue is that we renamed the repos a while ago. [22:20:48] apparently there was no outage reported, so now my only clue is that we renamed the repos a while ago. [22:21:11] awight there was a outage today. [22:21:11] awight there was a outage today. [22:21:16] I fixed the URIs just today, as part of debugging this issue [22:21:16] I fixed the URIs just today, as part of debugging this issue [22:21:17] though there issue page does not list it [22:21:17] though there issue page does not list it [22:21:24] paladox: haha that's perfect [22:21:24] paladox: haha that's perfect [22:27:08] paladox: Thanks for pressing the magic button! Is "update now" clickable but silently has no effect when plebians attempt to use? [22:27:08] paladox: Thanks for pressing the magic button! Is "update now" clickable but silently has no effect when plebians attempt to use? [22:27:20] your welcome :) [22:27:20] your welcome :) [22:27:39] and im not sure, it hasen't updated the status, so maybe it will take a while? [22:27:39] and im not sure, it hasen't updated the status, so maybe it will take a while? [22:29:22] that's fine for me; but I'm curious about how I clicked twice without having the same effect. and Reedy mentioned "needs a repo admin" [22:29:22] that's fine for me; but I'm curious about how I clicked twice without having the same effect. and Reedy mentioned "needs a repo admin" [22:29:55] If I'm reading that correctly, maybe we should disable the button for non-admins? [22:29:55] If I'm reading that correctly, maybe we should disable the button for non-admins? [22:30:40] it should already be disabled for non repo admins. [22:30:40] it should already be disabled for non repo admins. [22:30:57] awight i think your the owner of the repo, thus you can press it (or your a repo admin) [22:30:57] awight i think your the owner of the repo, thus you can press it (or your a repo admin) [22:31:01] fwiw, when i click that button i am told that i dont have permissions [22:31:01] fwiw, when i click that button i am told that i dont have permissions [22:31:04] and " Members of the project "Repository-Admins" can take this action." [22:31:04] and " Members of the project "Repository-Admins" can take this action." [22:31:14] * awight scratches head [22:31:14] * awight scratches head [22:31:15] but.. it does tell me about that [22:31:15] but.. it does tell me about that [22:31:26] (03CR) 10Legoktm: [C: 03+2] "Awesome :)" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481651 (https://phabricator.wikimedia.org/T209095) (owner: 10Umherirrender) [22:31:26] (03CR) 10Legoktm: [C: 03+2] "Awesome :)" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481651 (https://phabricator.wikimedia.org/T209095) (owner: 10Umherirrender) [22:31:28] your in https://phabricator.wikimedia.org/project/members/85/ awight [22:31:28] your in https://phabricator.wikimedia.org/project/members/85/ awight [22:31:31] are you not allowing pop-ups ? [22:31:31] are you not allowing pop-ups ? [22:31:39] :) you might have me there [22:31:39] :) you might have me there [22:32:00] (03CR) 10Legoktm: [C: 03+2] Add $wgLang and $wgOut to ExtendClassUsageSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481652 (owner: 10Umherirrender) [22:32:00] (03CR) 10Legoktm: [C: 03+2] Add $wgLang and $wgOut to ExtendClassUsageSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481652 (owner: 10Umherirrender) [22:32:12] (03Merged) 10jenkins-bot: Disallow use of @access [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481651 (https://phabricator.wikimedia.org/T209095) (owner: 10Umherirrender) [22:32:12] (03Merged) 10jenkins-bot: Disallow use of @access [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481651 (https://phabricator.wikimedia.org/T209095) (owner: 10Umherirrender) [22:32:35] (03CR) 10jenkins-bot: Disallow use of @access [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481651 (https://phabricator.wikimedia.org/T209095) (owner: 10Umherirrender) [22:32:35] (03CR) 10jenkins-bot: Disallow use of @access [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481651 (https://phabricator.wikimedia.org/T209095) (owner: 10Umherirrender) [22:32:38] okay so when I click, I get a dialog "update repository now" with a lot of harmless-looking text. at the bottom, there's another button "schedule update". I tried that 2x, waiting mebbe 10 minutes between attempts [22:32:38] okay so when I click, I get a dialog "update repository now" with a lot of harmless-looking text. at the bottom, there's another button "schedule update". I tried that 2x, waiting mebbe 10 minutes between attempts [22:32:41] (03Merged) 10jenkins-bot: Add $wgLang and $wgOut to ExtendClassUsageSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481652 (owner: 10Umherirrender) [22:32:41] (03Merged) 10jenkins-bot: Add $wgLang and $wgOut to ExtendClassUsageSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481652 (owner: 10Umherirrender) [22:32:54] But it seems like the update was only actually scheduled when paladox did the same... [22:32:54] But it seems like the update was only actually scheduled when paladox did the same... [22:33:02] (03CR) 10Legoktm: [C: 03+2] "FYI you can look at the mw-tools-codesniffer-mwcore-testrun job (which works now!) to see how many errors would be flagged in MW core by t" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481526 (https://phabricator.wikimedia.org/T212510) (owner: 10Umherirrender) [22:33:02] (03CR) 10Legoktm: [C: 03+2] "FYI you can look at the mw-tools-codesniffer-mwcore-testrun job (which works now!) to see how many errors would be flagged in MW core by t" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481526 (https://phabricator.wikimedia.org/T212510) (owner: 10Umherirrender) [22:33:08] hard to say for sure though, from my perspective. [22:33:08] hard to say for sure though, from my perspective. [22:33:09] (03CR) 10jenkins-bot: Add $wgLang and $wgOut to ExtendClassUsageSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481652 (owner: 10Umherirrender) [22:33:09] (03CR) 10jenkins-bot: Add $wgLang and $wgOut to ExtendClassUsageSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481652 (owner: 10Umherirrender) [22:33:33] 10MediaWiki-Codesniffer, 10Patch-For-Review: Disallow use of @access using MediaWiki.Commenting.FunctionAnnotations.UnrecognizedAnnotation - https://phabricator.wikimedia.org/T209095 (10Umherirrender) 05Open→03Resolved [22:33:33] 10MediaWiki-Codesniffer, 10Patch-For-Review: Disallow use of @access using MediaWiki.Commenting.FunctionAnnotations.UnrecognizedAnnotation - https://phabricator.wikimedia.org/T209095 (10Umherirrender) 05Open→03Resolved [22:33:42] (03Merged) 10jenkins-bot: Enable Generic.WhiteSpace.IncrementDecrementSpacing [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481526 (https://phabricator.wikimedia.org/T212510) (owner: 10Umherirrender) [22:33:42] (03Merged) 10jenkins-bot: Enable Generic.WhiteSpace.IncrementDecrementSpacing [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481526 (https://phabricator.wikimedia.org/T212510) (owner: 10Umherirrender) [22:34:06] (03CR) 10jenkins-bot: Enable Generic.WhiteSpace.IncrementDecrementSpacing [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481526 (https://phabricator.wikimedia.org/T212510) (owner: 10Umherirrender) [22:34:06] (03CR) 10jenkins-bot: Enable Generic.WhiteSpace.IncrementDecrementSpacing [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481526 (https://phabricator.wikimedia.org/T212510) (owner: 10Umherirrender) [22:34:08] (03CR) 10Legoktm: [C: 03+2] Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:34:08] (03CR) 10Legoktm: [C: 03+2] Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:34:28] 10MediaWiki-Codesniffer, 10Patch-For-Review: Enable Generic.WhiteSpace.IncrementDecrementSpacing in mediawiki ruleset - https://phabricator.wikimedia.org/T212510 (10Umherirrender) 05Open→03Resolved [22:34:28] 10MediaWiki-Codesniffer, 10Patch-For-Review: Enable Generic.WhiteSpace.IncrementDecrementSpacing in mediawiki ruleset - https://phabricator.wikimedia.org/T212510 (10Umherirrender) 05Open→03Resolved [22:35:22] I'll now just chill and watch the fireworks! [22:35:22] I'll now just chill and watch the fireworks! [22:35:41] (03PS6) 10Umherirrender: Expand ExtendClassUsageSniff to check for config globals [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481653 (https://phabricator.wikimedia.org/T200679) [22:35:41] (03PS6) 10Umherirrender: Expand ExtendClassUsageSniff to check for config globals [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481653 (https://phabricator.wikimedia.org/T200679) [22:37:29] (03PS2) 10Umherirrender: Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) [22:37:29] (03PS2) 10Umherirrender: Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) [22:37:37] (03CR) 10Umherirrender: [C: 03+2] "Resubmit" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:37:37] (03CR) 10Umherirrender: [C: 03+2] "Resubmit" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:38:16] (03Merged) 10jenkins-bot: Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:38:16] (03Merged) 10jenkins-bot: Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:38:40] (03CR) 10jenkins-bot: Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:38:40] (03CR) 10jenkins-bot: Enable Generic.Formatting.SpaceAfterNot with spacing 0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481530 (https://phabricator.wikimedia.org/T203799) (owner: 10Umherirrender) [22:39:21] Okay one last thing. Should I file a bug about how our gerrit repo can be wiped clean apparently by a failure to read from github? [22:39:21] Okay one last thing. Should I file a bug about how our gerrit repo can be wiped clean apparently by a failure to read from github? [22:39:26] scary stuff. [22:39:26] scary stuff. [22:40:12] maintenance-disconnect-full-disks build 34390 integration-slave-docker-1021: OFFLINE due to disk space [22:40:12] maintenance-disconnect-full-disks build 34390 integration-slave-docker-1021: OFFLINE due to disk space [22:40:29] yea, i think so. we should not be that dependent on it [22:40:29] yea, i think so. we should not be that dependent on it [22:40:38] especially not if it's actually wiping stuff [22:40:38] especially not if it's actually wiping stuff [22:41:05] awight: do you mean gerrit or phabricator? [22:41:05] awight: do you mean gerrit or phabricator? [22:41:11] and isnt syncing normally just _to_ github [22:41:11] and isnt syncing normally just _to_ github [22:41:13] gerrit [22:41:13] gerrit [22:41:18] i know :-/ [22:41:18] i know :-/ [22:41:26] but we have a non-standard thing [22:41:26] but we have a non-standard thing [22:41:49] that was a phab repo though? [22:41:49] that was a phab repo though? [22:41:50] our authoritative source is github, then we observe from phabricator, finally mirroring to gerrit. [22:41:50] our authoritative source is github, then we observe from phabricator, finally mirroring to gerrit. [22:42:02] okay. [22:42:02] okay. [22:42:07] that seems backwards :( [22:42:07] that seems backwards :( [22:42:10] It's... so that outsiders can collaborate more easily, and we can still deploy to WMF production. [22:42:10] It's... so that outsiders can collaborate more easily, and we can still deploy to WMF production. [22:42:26] yes don't look at me though, this is historical and stuff ;-) [22:42:26] yes don't look at me though, this is historical and stuff ;-) [22:42:31] is the mirroring force pushing? [22:42:31] is the mirroring force pushing? [22:42:45] if we limited it to normal force pushes shouldn't that be good enough? [22:42:45] if we limited it to normal force pushes shouldn't that be good enough? [22:42:48] legoktm: Good question, I'm not sure how that bit works. [22:42:48] legoktm: Good question, I'm not sure how that bit works. [22:42:59] prevents destruction of old stuff and only allows new commits to be added [22:42:59] prevents destruction of old stuff and only allows new commits to be added [22:43:24] Here's what it looks like through my plebe lenses: https://phabricator.wikimedia.org/source/articlequality/manage/uris/ [22:43:24] Here's what it looks like through my plebe lenses: https://phabricator.wikimedia.org/source/articlequality/manage/uris/ [22:43:41] https://gerrit.wikimedia.org/r/admin/projects/scoring/ores,access [22:43:41] https://gerrit.wikimedia.org/r/admin/projects/scoring/ores,access [22:43:44] then ah, [22:43:44] then ah, [22:43:46] so it does have force push [22:43:46] so it does have force push [22:43:47] right okay [22:43:47] right okay [22:44:12] https://gerrit.wikimedia.org/r/plugins/gitiles/scoring/ores/+/592b1399fbf0094e466ec5b1876c74aa348b9189%5E%21/#F0 [22:44:12] https://gerrit.wikimedia.org/r/plugins/gitiles/scoring/ores/+/592b1399fbf0094e466ec5b1876c74aa348b9189%5E%21/#F0 [22:44:18] twentyafterfour is the one who added force pushing [22:44:18] twentyafterfour is the one who added force pushing [22:44:21] if there's ever a valid reason to turn it back on we can, but this seems safer [22:44:21] if there's ever a valid reason to turn it back on we can, but this seems safer [22:44:44] greg-g: this = disabling force pushing? [22:44:44] greg-g: this = disabling force pushing? [22:44:45] it's scary though cos it's *gone* like empty, rather than just rolled back to master=something silly [22:44:45] it's scary though cos it's *gone* like empty, rather than just rolled back to master=something silly [22:44:49] legoktm: yeah [22:44:49] legoktm: yeah [22:45:19] 10MediaWiki-Codesniffer, 10Patch-For-Review: Add sniff to remove space after not operator - https://phabricator.wikimedia.org/T203799 (10Umherirrender) 05Open→03Resolved [22:45:19] 10MediaWiki-Codesniffer, 10Patch-For-Review: Add sniff to remove space after not operator - https://phabricator.wikimedia.org/T203799 (10Umherirrender) 05Open→03Resolved [22:45:23] (03PS1) 10Legoktm: Disable force-push permission per #wikimedia-releng discussion [scoring/ores] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/481961 [22:45:23] (03PS1) 10Legoktm: Disable force-push permission per #wikimedia-releng discussion [scoring/ores] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/481961 [22:45:31] (03CR) 10Legoktm: [V: 03+2 C: 03+2] Disable force-push permission per #wikimedia-releng discussion [scoring/ores] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/481961 (owner: 10Legoktm) [22:45:31] (03CR) 10Legoktm: [V: 03+2 C: 03+2] Disable force-push permission per #wikimedia-releng discussion [scoring/ores] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/481961 (owner: 10Legoktm) [22:48:49] greg-g: legoktm: well there's one legit use, rewriting history for LFS needs, but like you said it's fine to just switch the dangerous perms on temporarily. One question though. Will legoktm's change still allow the repo to be repopulated once the mirroring catches up again? [22:48:49] greg-g: legoktm: well there's one legit use, rewriting history for LFS needs, but like you said it's fine to just switch the dangerous perms on temporarily. One question though. Will legoktm's change still allow the repo to be repopulated once the mirroring catches up again? [22:50:09] as long as it doesn't need to force push [22:50:09] as long as it doesn't need to force push [22:50:51] (which it shouldn't since there's nothing to overwrite!) [22:50:51] (which it shouldn't since there's nothing to overwrite!) [22:51:49] kk that seems right, anyway it's not particularly time sensitive [22:51:49] kk that seems right, anyway it's not particularly time sensitive [22:58:03] legoktm: cool, I see this applies to the whole group of repos. Thanks! [22:58:03] legoktm: cool, I see this applies to the whole group of repos. Thanks! [22:58:10] * legoktm nods [22:58:10] * legoktm nods [22:58:11] np :) [22:58:11] np :) [23:05:13] maintenance-disconnect-full-disks build 34395 integration-slave-docker-1021: OFFLINE due to disk space [23:05:13] maintenance-disconnect-full-disks build 34395 integration-slave-docker-1021: OFFLINE due to disk space [23:16:42] Different repo, new problem. You all don't even want to know... https://phabricator.wikimedia.org/T212530#4850623 [23:16:42] Different repo, new problem. You all don't even want to know... https://phabricator.wikimedia.org/T212530#4850623 [23:17:06] Amir1 was mentioning that he directly pushed the object and it worked [23:17:06] Amir1 was mentioning that he directly pushed the object and it worked [23:17:10] for another repo [23:17:10] for another repo [23:17:18] but this is painful [23:17:18] but this is painful [23:24:01] Here's Amir's workaround, https://phabricator.wikimedia.org/T212544#4841979 [23:24:01] Here's Amir's workaround, https://phabricator.wikimedia.org/T212544#4841979 [23:27:02] well, fixed :) [23:27:02] well, fixed :) [23:29:28] I don't know how lfs works, sorry :/ [23:29:28] I don't know how lfs works, sorry :/ [23:29:42] seems like we should cut out phabricator as the middle step though [23:29:42] seems like we should cut out phabricator as the middle step though [23:30:11] maintenance-disconnect-full-disks build 34400 integration-slave-docker-1021: OFFLINE due to disk space [23:30:11] maintenance-disconnect-full-disks build 34400 integration-slave-docker-1021: OFFLINE due to disk space [23:31:51] 10Diffusion, 10GitHub-Mirrors, 10Scoring-platform-team: LFS objects are not mirroring from Github through Phab to Gerrit consistently - https://phabricator.wikimedia.org/T212818 (10awight) [23:31:51] 10Diffusion, 10GitHub-Mirrors, 10Scoring-platform-team: LFS objects are not mirroring from Github through Phab to Gerrit consistently - https://phabricator.wikimedia.org/T212818 (10awight) [23:32:44] LFS is both great and silly, but yeah it's been unpleasant to pioneer as you can imagine. [23:32:44] LFS is both great and silly, but yeah it's been unpleasant to pioneer as you can imagine. [23:33:06] Crisis is over for now, but I left a task for long-term fixing [23:33:06] Crisis is over for now, but I left a task for long-term fixing [23:45:17] I think there's no direct gateway between GitHub -> gerrit and then we need to gate those via Phab [23:45:17] I think there's no direct gateway between GitHub -> gerrit and then we need to gate those via Phab [23:45:36] and diffusion is slow [23:45:36] and diffusion is slow [23:52:17] https://phabricator.wikimedia.org/T191182 [23:52:17] https://phabricator.wikimedia.org/T191182 [23:52:25] ^ also this task to stop using Diffusion [23:52:25] ^ also this task to stop using Diffusion [23:53:02] sorry, Differential [23:53:02] sorry, Differential [23:54:33] and the reason there is no direct gateway between Github -> Gerrit is because we don't want that for policy reasons afaict [23:54:33] and the reason there is no direct gateway between Github -> Gerrit is because we don't want that for policy reasons afaict [23:55:05] idk - just an observer [23:55:05] idk - just an observer [23:55:12] maintenance-disconnect-full-disks build 34405 integration-slave-docker-1021: OFFLINE due to disk space [23:55:12] maintenance-disconnect-full-disks build 34405 integration-slave-docker-1021: OFFLINE due to disk space [23:56:51] so doing it via something else might be circumventing policy [23:56:51] so doing it via something else might be circumventing policy [23:57:31] There’s a policy on mirroring on github to a gerrit repo? [23:57:31] There’s a policy on mirroring on github to a gerrit repo? [23:58:07] gerrit repos mirror to github using the replication plugin [23:58:07] gerrit repos mirror to github using the replication plugin [23:58:14] but not vice-versa [23:58:14] but not vice-versa