[00:07:25] 10Release-Engineering-Team (Backlog), 10MW-1.32-release: Release 1.32.1 as a maintenance release - https://phabricator.wikimedia.org/T213595 (10greg) [00:13:36] 10Continuous-Integration-Config, 10Growth-Team, 10StructuredDiscussions, 10Patch-For-Review, 10User-kostajh: Fix Flow random test failures - https://phabricator.wikimedia.org/T208988 (10Catrope) [01:02:41] 10Beta-Cluster-Infrastructure, 10MobileFrontend, 10Technical-Debt: Clean up beta cluster mobile configuration - https://phabricator.wikimedia.org/T213599 (10MaxSem) [01:09:10] 10Release-Engineering-Team (Backlog), 10ORES, 10Operations, 10Scoring-platform-team, 10Release Pipeline (Blubber): The continuous release pipeline should support more than one service per repo - https://phabricator.wikimedia.org/T210267 (10dduvall) >>! In T210267#4873881, @thcipriani wrote: > > What new... [01:23:28] Trying to figure out why https://doc.wikimedia.org/mediawiki-libs-WaitConditionLoop/master/ is blank [01:23:40] should be rendering https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/libs/WaitConditionLoop/+/master/README.md [01:40:32] 10Beta-Cluster-Infrastructure, 10MobileFrontend, 10Technical-Debt: Clean up beta cluster mobile configuration - https://phabricator.wikimedia.org/T213599 (10MaxSem) [02:17:26] [06:13:19] Hmm if I upload something to releases.wikimedia.org should it be live right away? <-- it might take an hour for the varnish cache to expire on the index page [02:18:34] 10MediaWiki-Releasing, 10MW-1.32-release, 10Patch-For-Review: Unknown key used to sign MediaWiki 1.32.0 tarball - https://phabricator.wikimedia.org/T213521 (10Legoktm) 05Open→03Resolved Yesterday: [00:26:32] legoktm: D7B8437.... should be sync'd to pool.sks-keyservers.net Today I pull... [02:20:13] CFisch_NA: legoktm: also not ehtere is one releases server in each data center and the service is active-active [02:20:28] if you upload to eqiad but are located in a place that gets served by codfw .. [02:20:40] you would have to wait for rsync cron between them [06:03:12] (03PS3) 10Krinkle: [WIP] Add mediawiki-patch-fresnel job [integration/config] - 10https://gerrit.wikimedia.org/r/459268 (https://phabricator.wikimedia.org/T133646) [06:04:17] (03PS4) 10Krinkle: [WIP] Add mediawiki-fresnel-patch job [integration/config] - 10https://gerrit.wikimedia.org/r/459268 (https://phabricator.wikimedia.org/T133646) [10:20:59] mutante: thanks, seems to be synced now :-) [13:51:07] (03PS12) 10Mainframe98: Forbid usage of is_null() [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [13:51:40] (03CR) 10Mainframe98: "> Patch Set 11: Code-Review-1" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [13:54:58] (03CR) 10jerkins-bot: [V: 04-1] Forbid usage of is_null() [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [13:55:35] (03PS13) 10Mainframe98: Forbid usage of is_null() [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [14:18:53] 10Continuous-Integration-Config, 10Analytics, 10Patch-For-Review: Add json linting test for schemas in mediawiki/event-schemas - https://phabricator.wikimedia.org/T124319 (10hashar) [14:19:37] 10Continuous-Integration-Infrastructure: CI trusty slaves do not have php5-apcu installed - https://phabricator.wikimedia.org/T124800 (10hashar) 05Open→03Declined [14:21:32] 10Continuous-Integration-Infrastructure, 10Technical-Debt: Remove manual $PHP_BIN handling in slave-scripts - https://phabricator.wikimedia.org/T128008 (10hashar) 05Open→03Resolved Looks like it was a transient state that got fixed at some point :) [14:23:29] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Kanban), 10Patch-For-Review, 10Technical-Debt: Clear /srv/.git on contint1001 - https://phabricator.wikimedia.org/T149924 (10hashar) https://doc.wikimedia.org/ is now served by `doc1001.eqiad.wmnet`. integration/docroot is cloned to /sr... [14:27:19] 10Continuous-Integration-Infrastructure, 10JavaScript, 10Test-Coverage: Generate JS code coverage reports for extensions - https://phabricator.wikimedia.org/T184657 (10hashar) [14:27:24] 10Continuous-Integration-Infrastructure, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10Test-Coverage: [Task] Wikibase: have test coverage of JavaScript QUnit tests - https://phabricator.wikimedia.org/T42656 (10hashar) [17:53:50] (03CR) 10Legoktm: [C: 03+2] Enable Generic.CodeAnalysis.EmptyPHPStatement [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/482375 (owner: 10Umherirrender) [17:54:46] (03Merged) 10jenkins-bot: Enable Generic.CodeAnalysis.EmptyPHPStatement [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/482375 (owner: 10Umherirrender) [17:55:13] (03CR) 10jenkins-bot: Enable Generic.CodeAnalysis.EmptyPHPStatement [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/482375 (owner: 10Umherirrender) [17:57:57] (03CR) 10Legoktm: [C: 03+2] Fix UnusedUseStatementSniff to find more unused statements [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/482007 (https://phabricator.wikimedia.org/T212846) (owner: 10Umherirrender) [17:59:32] (03Merged) 10jenkins-bot: Fix UnusedUseStatementSniff to find more unused statements [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/482007 (https://phabricator.wikimedia.org/T212846) (owner: 10Umherirrender) [18:00:46] (03CR) 10Legoktm: [C: 04-1] "We should use the ExtensionInfo class to only warn based upon the extension's MediaWiki requirement. You can see how it's used in Deprecat" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481653 (https://phabricator.wikimedia.org/T200679) (owner: 10Umherirrender) [18:02:48] (03CR) 10jenkins-bot: Fix UnusedUseStatementSniff to find more unused statements [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/482007 (https://phabricator.wikimedia.org/T212846) (owner: 10Umherirrender) [18:11:17] 10MediaWiki-Codesniffer, 10Need-volunteer, 10Patch-For-Review: Sniff UnusedUseStatement does not find all unused use statements - https://phabricator.wikimedia.org/T212846 (10D3r1ck01) 05Open→03Resolved Thanks @Umherirrender for working on this and @Legoktm for final review and landing the patch :) [19:25:57] (03PS1) 10Mainframe98: Add a sniff to replace !! with a cast to boolean [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) [19:28:10] (03PS2) 10Mainframe98: Add a sniff to replace !! with a cast to boolean [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) [21:04:27] (03CR) 10Umherirrender: "The prefered IContextSource::getConfig is there since 1.23, is that worse to respect? What should be done in extensions without a required" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/481653 (https://phabricator.wikimedia.org/T200679) (owner: 10Umherirrender) [21:27:36] (03CR) 10Umherirrender: Add a sniff to replace !! with a cast to boolean (035 comments) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) (owner: 10Mainframe98) [21:49:06] (03CR) 10Umherirrender: Forbid usage of is_null() (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [22:05:50] (03PS14) 10Mainframe98: Forbid usage of is_null() [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [22:15:57] (03PS3) 10Mainframe98: Add a sniff to replace !! with a cast to boolean [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) [22:19:39] (03CR) 10jerkins-bot: [V: 04-1] Add a sniff to replace !! with a cast to boolean [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) (owner: 10Mainframe98) [22:20:54] (03PS4) 10Mainframe98: Add a sniff to replace !! with a cast to boolean [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) [22:20:58] (03CR) 10Mainframe98: Add a sniff to replace !! with a cast to boolean (035 comments) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) (owner: 10Mainframe98) [22:21:36] (03CR) 10Mainframe98: Forbid usage of is_null() (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/446271 (https://phabricator.wikimedia.org/T199768) (owner: 10Prtksxna) [22:22:47] (03PS5) 10Mainframe98: Add a sniff to replace !! with a cast to boolean [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/483946 (https://phabricator.wikimedia.org/T203800) [23:36:21] 10Release-Engineering-Team (Backlog), 10MW-1.32-release: Release 1.32.1 as a maintenance release - https://phabricator.wikimedia.org/T213595 (10Paladox)