[01:47:12] RECOVERY - SSH on contint2001.mgmt is OK: SSH OK - OpenSSH_6.6 (protocol 2.0) https://wikitech.wikimedia.org/wiki/Dc-operations/Hardware_Troubleshooting_Runbook [04:33:01] 10Continuous-Integration-Infrastructure, 10SRE, 10SRE-Access-Requests: Requesting access to contint-admins for Ladsgroup - https://phabricator.wikimedia.org/T283925 (10Marostegui) I think it needs approval from @greg or @thcipriani [04:33:13] 10Continuous-Integration-Infrastructure, 10SRE, 10SRE-Access-Requests: Requesting access to contint-admins for Ladsgroup - https://phabricator.wikimedia.org/T283925 (10Marostegui) [05:57:11] 10Continuous-Integration-Infrastructure, 10SRE, 10SRE-Access-Requests: Requesting access to contint-admins for Ladsgroup - https://phabricator.wikimedia.org/T283925 (10Marostegui) a:03Marostegui [06:03:03] 10Gerrit, 10Diffusion-Repository-Administrators: Unarchive EditAccount repository on Gerrit - https://phabricator.wikimedia.org/T283935 (10Kizule) Ping. :) Please re-create mirror on GitHub and close this task as resolved. [06:15:33] 10Release-Engineering-Team (Seen), 10Pywikibot, 10Pywikibot-tests: Lint test to match function signature and documentation - https://phabricator.wikimedia.org/T277396 (10Xqt) [06:17:52] (03CR) 10Thiemo Kreuz (WMDE): [C: 03+2] AssertCountSniff: allow manual `count` on both sides [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693535 (https://phabricator.wikimedia.org/T273352) (owner: 10DannyS712) [06:19:46] (03Merged) 10jenkins-bot: AssertCountSniff: allow manual `count` on both sides [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693535 (https://phabricator.wikimedia.org/T273352) (owner: 10DannyS712) [06:22:46] 10Release-Engineering-Team (Seen), 10Pywikibot, 10Pywikibot-tests: Lint test to match function signature and documentation - https://phabricator.wikimedia.org/T277396 (10Xqt) @atagar: are you willing to takeover this task? [06:40:32] 10Release-Engineering-Team (Seen), 10Pywikibot, 10Pywikibot-tests: Lint test to match function signature and documentation - https://phabricator.wikimedia.org/T277396 (10atagar) Hi Xqt. It's a bit unclear to me what the task is... > lint checker that would look at the function's signature (i.e. parameter na... [07:42:01] 10Gerrit, 10Upstream: Gerrit logs are no more generated since April 30st - https://phabricator.wikimedia.org/T284041 (10hashar) [08:06:35] 10Beta-Cluster-Infrastructure, 10cloud-services-team (Kanban): broken puppet on deployment-eventlog05.deployment-prep.eqiad.wmflabs - https://phabricator.wikimedia.org/T276122 (10elukey) a:05elukey→03None [08:41:24] 10Gerrit, 10Upstream: Gerrit logs are no more generated since April 30st - https://phabricator.wikimedia.org/T284041 (10hashar) 05Open→03Declined It is back after I have restarted Gerrit for the Java 11 upgrade. Lesson learned: do not use `gerrit logging set-level reset`. Not much to be done on this task... [08:47:56] hashar: Hey! I need some help on https://gerrit.wikimedia.org/r/c/integration/config/+/696586 which is blocking merging our python projects :/ [08:48:18] Erik wrote that patch, it looks reasonable to me, but I'm all confused about all those variables [08:53:13] gehel: hi, I have noticed it but haven't looked at the code yet [08:53:29] gehel: I am busy this morning though, but would be around at 14:30/15:00 [08:53:35] have yet another meeting at 14:00 [08:53:48] I'll be there at 15:00 [08:54:24] invite sent :] [08:54:43] some mess up related to the branch naming we did a few weeks ago [08:54:54] hashar: I am going to skip the meeting in a few minutes, I added an agenda item about one thing I wanted feedback on (robot comment implementation for phpcs fixes) [08:54:58] but that is all I can tell right now yeah. Lets polish that this afternoon [08:55:15] kostajh: oh sure! I can give feedbakc on the task I guess [08:55:34] kostajh: thank you for the notice! [09:04:58] thanks hashar [09:49:06] 10MediaWiki-Codesniffer, 10MediaWiki-Documentation, 10Patch-For-Review: Standardize return type hint spacing - https://phabricator.wikimedia.org/T220719 (10kostajh) >>! In T220719#6967288, @Jdforrester-WMF wrote: >>>! In T220719#6967168, @DannyS712 wrote: >>>>! In T220719#5704728, @Reedy wrote: >>>>>! In T22... [09:55:47] 10Release-Engineering-Team (Seen), 10Pywikibot, 10Pywikibot-tests: Lint test to match function signature and documentation - https://phabricator.wikimedia.org/T277396 (10Xqt) >>! In T277396#7125020, @atagar wrote: > Hi Xqt. It's a bit unclear to me what the task is... > >> lint checker that would look at th... [10:35:40] 10Continuous-Integration-Infrastructure, 10SRE, 10Patch-For-Review: Have linters/tests results show up as comments in files on gerrit - https://phabricator.wikimedia.org/T209149 (10awight) Thoughts about this: * Why does phpcs have to be run twice? Can we run in reporting mode the first time? Is there some... [10:39:03] 10Continuous-Integration-Infrastructure, 10SRE, 10Patch-For-Review: Have linters/tests results show up as comments in files on gerrit - https://phabricator.wikimedia.org/T209149 (10kostajh) >>! In T209149#7125536, @awight wrote: > Thoughts about this: > * Why does phpcs have to be run twice? Can we run in r... [10:43:12] 10Release-Engineering-Team (Doing), 10Gerrit (Gerrit 3.3), 10Patch-For-Review: Switch Gerrit from Java 8 to Java 11 - https://phabricator.wikimedia.org/T268225 (10hashar) Huge thank you to @MoritzMuehlenhoff for suggesting to manually test the command on the replica, that caught that some options passed to `... [11:22:23] 10Phabricator: Unhandled Exception ("RuntimeException") - https://phabricator.wikimedia.org/T284059 (10IN) [11:23:47] 10Phabricator: Unhandled Exception ("RuntimeException") - https://phabricator.wikimedia.org/T284059 (10IN) [12:20:19] 10Phabricator: Unhandled Exception ("RuntimeException") - https://phabricator.wikimedia.org/T284059 (10Aklapper) 05Open→03Stalled p:05Triage→03Lowest Hi, how and why did you find and enter that URL? Why would you expect broken URLs not to display an error? This task lacks a use case. [12:20:47] 10Phabricator: RuntimeException: "Undefined variable: url" at https://phabricator.wikimedia.org/u/0 - https://phabricator.wikimedia.org/T284059 (10Aklapper) [12:50:45] gehel: je suis dans le meet :d [13:06:55] (03CR) 1020after4: "Thanks for the code review. Sorry for not following up sooner." (038 comments) [releng/ddd] - 10https://gerrit.wikimedia.org/r/694021 (owner: 1020after4) [13:26:48] 10Phabricator (Upstream), 10Developer Productivity, 10Upstream: Disable parent task metadata by default for new sub tasks - https://phabricator.wikimedia.org/T239378 (10kostajh) >>! In T239378#6409869, @Jhernandez wrote: > I was talking with Andre the other day about this and he pointed me to this task. > >... [13:26:50] 10Release-Engineering-Team (Doing), 10Gerrit (Gerrit 3.3), 10Patch-For-Review: Switch Gerrit from Java 8 to Java 11 - https://phabricator.wikimedia.org/T268225 (10hashar) Both Gerrit are running with Java 11 and Puppet is up to date. We are keeping the Java 8 packages around in case we need to roll back, wi... [13:44:28] (03PS3) 10Gehel: python-jobs: Point sonarscanner at expected branch names [integration/config] - 10https://gerrit.wikimedia.org/r/696586 (https://phabricator.wikimedia.org/T283778) (owner: 10Ebernhardson) [13:50:40] (03PS4) 10Gehel: python-jobs: Point sonarscanner at expected branch names [integration/config] - 10https://gerrit.wikimedia.org/r/696586 (https://phabricator.wikimedia.org/T283778) (owner: 10Ebernhardson) [14:19:10] (03CR) 10Hashar: "Guillaume and I talked and paired about it which results in some tweaks to Erik original patch. The job template is used against proposed" (032 comments) [integration/config] - 10https://gerrit.wikimedia.org/r/696586 (https://phabricator.wikimedia.org/T283778) (owner: 10Ebernhardson) [14:53:58] 10Phabricator, 10Release-Engineering-Team, 10User-Matthewrbowker: Phabricator has been declared EOL - https://phabricator.wikimedia.org/T283980 (10Milimetric) Much love to @epriestley for starting this project, helping us move to it, and incorporating our feedback and contributions. In my opinion, moving to... [14:55:54] 10Release-Engineering-Team (Radar), 10GitLab (Initialization), 10Patch-For-Review, 10Puppet, and 2 others: Puppetise gitlab-ansible playbook - https://phabricator.wikimedia.org/T283076 (10jbond) [15:55:01] 10Phabricator (Upstream), 10Developer Productivity, 10Upstream: Disable parent task metadata by default for new sub tasks - https://phabricator.wikimedia.org/T239378 (10MBinder_WMF) This may be moot now that Phacility is EOL, but in the true spirit of Phabbing, a workaround for mitigating all that tag/subscr... [16:35:42] 10Phabricator, 10Release-Engineering-Team, 10User-Matthewrbowker: Phabricator has been declared EOL - https://phabricator.wikimedia.org/T283980 (10nshahquinn-wmf) I also want to appreciate @epriestley for all that he's done over the years. I have really, really enjoyed using Phabricator here. I assume that... [16:54:52] 10Phabricator, 10Developer-Advocacy (Jul-Sep 2021), 10PM: Decrease issues created many years ago with no recent activity (aka stale tickets) - https://phabricator.wikimedia.org/T248034 (10Jdlrobson) I suspect "stale" will vary per team, this it's important this is configurable and opt-in given the feedback f... [16:59:21] 10MediaWiki-Codesniffer, 10User-DannyS712: Codesniffer should prefer assertSame not assertCount when both part of comparision are count() - https://phabricator.wikimedia.org/T273352 (10Umherirrender) 05Open→03Resolved a:03DannyS712 [17:03:19] (03PS3) 10Umherirrender: RedundantVarNameSniff: also check static properties [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693533 (https://phabricator.wikimedia.org/T278471) (owner: 10DannyS712) [17:03:22] (03CR) 10Umherirrender: [C: 03+2] RedundantVarNameSniff: also check static properties [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693533 (https://phabricator.wikimedia.org/T278471) (owner: 10DannyS712) [17:04:04] 10MediaWiki-Codesniffer, 10Patch-For-Review, 10User-DannyS712: codesniffer's RedundantVarNameSniff does not remove var name from @var on static class properties - https://phabricator.wikimedia.org/T278471 (10Umherirrender) 05Open→03Resolved a:03DannyS712 [17:04:55] (03Merged) 10jenkins-bot: RedundantVarNameSniff: also check static properties [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693533 (https://phabricator.wikimedia.org/T278471) (owner: 10DannyS712) [17:29:05] 10Phabricator, 10Release-Engineering-Team, 10User-Matthewrbowker: Phabricator has been declared EOL - https://phabricator.wikimedia.org/T283980 (10Legoktm) >>! In T283980#7123515, @ashley wrote: > Phabricator is a great piece of software and while it's //unfortunate// that Phalicity, Inc. is closing, it does... [17:39:09] 10Phabricator (Upstream), 10Developer Productivity, 10Upstream: Disable parent task metadata by default for new sub tasks - https://phabricator.wikimedia.org/T239378 (10Tgr) I'd also recommend jut blanking all the fields and maybe opting in specific projects if someone complains about them. It would be far l... [17:58:49] 10Phabricator, 10Release-Engineering-Team, 10User-Matthewrbowker: Phabricator has been declared EOL - https://phabricator.wikimedia.org/T283980 (10RhinosF1) I'm currently running (as of this week) wiki media's phab fork on an install without issues [19:10:41] (03CR) 10Gehel: python-jobs: Point sonarscanner at expected branch names (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/696586 (https://phabricator.wikimedia.org/T283778) (owner: 10Ebernhardson) [20:25:07] (03PS5) 10Umherirrender: PrefixedGlobalFunctionsSniff: drop `ef` from default prefixes [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693253 (owner: 10DannyS712) [20:25:18] (03CR) 10Umherirrender: [C: 03+2] "> Patch Set 4:" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693253 (owner: 10DannyS712) [20:27:01] (03Merged) 10jenkins-bot: PrefixedGlobalFunctionsSniff: drop `ef` from default prefixes [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/693253 (owner: 10DannyS712) [20:44:56] 10Phabricator, 10Release-Engineering-Team, 10User-Matthewrbowker: Phabricator has been declared EOL - https://phabricator.wikimedia.org/T283980 (10Aklapper) > Can you share a bit about how much Wikimedia's install is forked from upstream? https://www.mediawiki.org/wiki/Phabricator/Code (in theory) [20:50:13] 10Gerrit, 10wikitech.wikimedia.org, 10LDAP, 10cloud-services-team (Kanban): Rename LDAP CN to 'Ori' - https://phabricator.wikimedia.org/T282631 (10bd808) [21:24:31] (03PS1) 10Nikki Nikkhoui: Custom project pipeline for image-suggestion-api [integration/config] - 10https://gerrit.wikimedia.org/r/697669 (https://phabricator.wikimedia.org/T281132) [21:25:27] 10Project-Admins: Create a Project #mediawiki-extensions-WSOAuth - https://phabricator.wikimedia.org/T283873 (10Aklapper) 05Open→03Resolved a:03Aklapper Requested public project #mediawiki-extensions-wsoauth has been created: https://phabricator.wikimedia.org/project/view/5370/ Interested people are welco... [22:43:40] 10Gerrit, 10LDAP-Access-Requests, 10SRE: Add dancy to `archiva-deployers` LDAP group - https://phabricator.wikimedia.org/T283347 (10colewhite) 05Open→03Stalled a:03colewhite @thcipriani, do you approve this request? [22:43:42] 10Gerrit, 10Release-Engineering-Team (Doing): Upgrade Gerrit to 3.2.10 - https://phabricator.wikimedia.org/T278990 (10colewhite) [22:44:48] 10Gerrit, 10LDAP-Access-Requests, 10SRE: Add dancy to `archiva-deployers` LDAP group - https://phabricator.wikimedia.org/T283347 (10thcipriani) >>! In T283347#7127854, @colewhite wrote: > @thcipriani, do you approve this request? Sorry I missed this one! Approved! [22:59:29] 10Gerrit, 10Release-Engineering-Team (Doing): Upgrade Gerrit to 3.2.10 - https://phabricator.wikimedia.org/T278990 (10colewhite) [23:00:04] 10Gerrit, 10LDAP-Access-Requests, 10SRE: Add dancy to `archiva-deployers` LDAP group - https://phabricator.wikimedia.org/T283347 (10colewhite) 05Stalled→03Resolved `uid=dancy` added to `archiva-deployers` [23:11:35] 10Release-Engineering-Team (Radar), 10SRE, 10serviceops, 10Patch-For-Review: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Izno) [23:12:10] 10Deployments, 10Release-Engineering-Team (Doing): L10n cache files building up on backup deploy hosts - https://phabricator.wikimedia.org/T275826 (10Izno) [23:12:14] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10serviceops, 10Patch-For-Review: Replace production deployment servers and update them to Buster - https://phabricator.wikimedia.org/T265963 (10Izno) [23:12:19] 10Release-Engineering-Team (Radar), 10SRE, 10serviceops, 10Patch-For-Review: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Izno) [23:13:12] 10Scap: scap deploy --init on deployment server fails on first puppet run - https://phabricator.wikimedia.org/T257317 (10Izno) [23:13:14] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10serviceops, 10Patch-For-Review: Replace production deployment servers and update them to Buster - https://phabricator.wikimedia.org/T265963 (10Izno) [23:13:16] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)), 10Patch-For-Review, 10Technical-Debt: Move integration/docroot.git deployment to scap - https://phabricator.wikimedia.org/T256005 (10Izno) [23:13:19] 10Release-Engineering-Team (Radar), 10SRE, 10serviceops, 10Patch-For-Review: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Izno) [23:15:25] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO, 10Patch-For-Review: Drop MediaWiki testing in stretch and instead test only in buster - https://phabricator.wikimedia.org/T252432 (10Izno) [23:15:30] 10Release-Engineering-Team (Radar), 10SRE, 10serviceops, 10Patch-For-Review: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Izno) [23:16:07] 10Continuous-Integration-Infrastructure, 10Patch-For-Review: Migrate all CI jobs from stretch to buster or later and drop stretch testing support - https://phabricator.wikimedia.org/T278203 (10Izno) [23:16:09] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO, 10Patch-For-Review: Drop MediaWiki testing in stretch and instead test only in buster - https://phabricator.wikimedia.org/T252432 (10Izno) [23:16:15] 10Release-Engineering-Team (Radar), 10SRE, 10serviceops, 10Patch-For-Review: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Izno) [23:17:32] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO, 10Patch-For-Review: Drop MediaWiki testing in stretch and instead test only in buster - https://phabricator.wikimedia.org/T252432 (10Izno) [23:17:41] 10Release-Engineering-Team (Radar), 10SRE, 10serviceops, 10Patch-For-Review: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10Izno) [23:39:31] 10Project-Admins, 10User-DannyS712: Create #mediawiki-actions component - https://phabricator.wikimedia.org/T268386 (10Aklapper) 05Open→03Declined [23:53:31] 10Project-Admins: Create #mediawiki-actions component - https://phabricator.wikimedia.org/T268386 (10DannyS712)