Page MenuHomePhabricator

Legoktm (Legoktm)
UserAdministrator

Projects (98)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Sep 19 2014, 7:30 PM (230 w, 6 d)
Roles
Administrator
Availability
Available
IRC Nick
legoktm
LDAP User
Legoktm
MediaWiki User
Unknown

Recent Activity

Yesterday

Legoktm added a comment to T215465: Development policy amendment: Require use of storage abstraction layers.

I don't think this kind of recommendation ("require use of storage abstraction layers") fits in a development policy, it seems more like a guideline.

Thu, Feb 21, 10:00 AM · Services (watching), Core Platform Team Backlog (Watching / External), Performance-Team, TechCom
Legoktm renamed T214637: Setup CSP http header in Quarry from Setup CSP http header to Setup CSP http header in Quarry.
Thu, Feb 21, 9:57 AM · Patch-For-Review, Security, Quarry
Legoktm reassigned T183211: Defect: API action=block allows blocking non-existent accounts from Legoktm to Anomie.
Thu, Feb 21, 3:44 AM · MediaWiki-User-management, MediaWiki-API
Legoktm closed T183211: Defect: API action=block allows blocking non-existent accounts as Resolved.

My bad, probably not worth it anymore since it's included in the (now) latest LTS.

Thu, Feb 21, 3:44 AM · MediaWiki-User-management, MediaWiki-API

Wed, Feb 20

Legoktm added a comment to T76467: Quarry -> #wikimedia-research & #wikimedia-labs.

Could we remove Quarry from #wikimedia-research? It's cluttering our channel and I'm not sure who in the channel needs this information.

Wed, Feb 20, 5:38 PM · Quarry, Wikibugs

Tue, Feb 19

Legoktm added a comment to T216370: IP address list for grid nodes / Freenode iline request.

Have they added the whole range or just that NAT IP?
If they've just added the NAT IP, I expect problems will resume if the tools exec nodes get given floating IPs?

Tue, Feb 19, 7:33 PM · cloud-services-team (Kanban), wikimedia-irc-freenode, Toolforge
Legoktm committed rELINT18d5dcabe216: DWIM in CategoryManager (authored by Legoktm).
DWIM in CategoryManager
Tue, Feb 19, 7:02 PM
Legoktm committed rELINT0f37222bbb1b: Add new Linter category wikilink-in-extlink (authored by farida29).
Add new Linter category wikilink-in-extlink
Tue, Feb 19, 7:02 PM
Legoktm changed the visibility for T207221: wikimedia/wikimania-scholarships has vulnerable dependencies.
Tue, Feb 19, 3:19 AM · Patch-For-Review, Wikimedia-Wikimania-Scholarships, Security

Mon, Feb 18

Legoktm added a project to T212453: Allow to explicity call certain function when multiple extensions define the same function (conflicting function names): MediaWiki-Parser.
Mon, Feb 18, 8:44 PM · MediaWiki-Parser, MediaWiki-extensions-General
Legoktm added a comment to T216314: loltrs crashes upon migrating to stretch grid.

This is probably the same issue as T216370: IP address list for grid nodes / Freenode iline request.

Mon, Feb 18, 4:24 AM · Toolforge
Legoktm added a comment to T216376: Give access to l10n-bot.

Which repository?

Mon, Feb 18, 4:23 AM · Trash

Sun, Feb 17

Legoktm added a comment to T207389: Rename the Certcentral project.

All taken care of from the CI side AFAICT. One minor note that while rebuilding the docker image, I had to switch pebble to be built with golang 1.11 instead of 1.10 - see https://gerrit.wikimedia.org/r/491028

Sun, Feb 17, 7:25 AM · Patch-For-Review, Acme-chief
Legoktm updated the task description for T207389: Rename the Certcentral project.
Sun, Feb 17, 7:23 AM · Patch-For-Review, Acme-chief
Legoktm added a comment to T216337: New lua function to follow entity redirects.

Ah, gotcha.

Sun, Feb 17, 6:08 AM · Wikibase-Lua, Wikidata
Legoktm added a comment to T216337: New lua function to follow entity redirects.

Can't you use mw.Title for this? https://www.mediawiki.org/wiki/Extension:Scribunto/Lua_reference_manual#Title_objects

Sun, Feb 17, 2:39 AM · Wikibase-Lua, Wikidata
Legoktm added projects to T216341: Allow using Special:Log on user talk pages: MediaWiki-Special-pages, MediaWiki-Logging.
Sun, Feb 17, 2:38 AM · MediaWiki-Logging, MediaWiki-Special-pages, User-DannyS712

Sat, Feb 16

Legoktm created T216314: loltrs crashes upon migrating to stretch grid.
Sat, Feb 16, 9:02 AM · Toolforge
Legoktm placed T216312: Copyvio detection tool is down up for grabs.

Please see https://lists.wikimedia.org/pipermail/cloud-announce/2019-February/000134.html - the general issue is known and it'll take a bit of time to address.

Sat, Feb 16, 7:29 AM · Tools
Legoktm added a comment to T146257: Create objectcache/BagOStuff library.

Nope. What do you want to call it? ObjectCache or BagOStuff?

Sat, Feb 16, 7:04 AM · Patch-For-Review, User-Addshore, MediaWiki-Cache, Librarization

Fri, Feb 15

Legoktm added a comment to T213595: Release 1.32.1 as a maintenance release.

So....who's doing the release? What's happening exactly?

Fri, Feb 15, 10:17 PM · Release-Engineering-Team (Kanban), MW-1.32-notes, MW-1.32-release
Legoktm closed T200185: Create a Special:DisabledSpecialPage to be used instead of Special:BlankPage as Resolved.
commit 608879edbff5a2c6a386eceaa494c98a93c7436e
Author: Gergő Tisza <gtisza@wikimedia.org>
Date:   Thu Nov 15 18:22:26 2018 -0800
Fri, Feb 15, 8:36 PM · MediaWiki-Special-pages

Thu, Feb 14

Legoktm changed the status of T216165: Drop PHP 7.0 support from MediaWiki from Open to Stalled.

Presumably this task will depend upon a TechCom-RFC to bump the minimum PHP requirement for MediaWiki 1.35, the next LTS.

Thu, Feb 14, 7:45 PM · TechCom, MediaWiki-General-or-Unknown, PHP 7.0 support
Legoktm changed the status of T216165: Drop PHP 7.0 support from MediaWiki, a subtask of T216166: Drop PHP 7.1 support from MediaWiki, from Open to Stalled.
Thu, Feb 14, 7:45 PM · PHP 7.1 support, MediaWiki-General-or-Unknown
Legoktm reassigned T216076: PHP does not find mysqli on tools-sgebastion-07 from bd808 to MoritzMuehlenhoff.

OK, I got the timing of events wrong. This should get fixed by Moritz's https://gerrit.wikimedia.org/r/c/operations/puppet/+/490579/ and then the next import.

Thu, Feb 14, 12:13 PM · Patch-For-Review, Toolforge
Legoktm added a comment to T216076: PHP does not find mysqli on tools-sgebastion-07.

For whatever reason, it looks like apt is not pulling in libzip and libpcre from thirdparty/php72. Note that Moritz *just* updated php72, so it'll actually want to pull in a totally different libpcre, and libzip5 I think.

Thu, Feb 14, 12:06 PM · Patch-For-Review, Toolforge
Legoktm updated the task description for T216067: Recover from corrupted beta MySQL slave (deployment-db04).
Thu, Feb 14, 10:28 AM · Beta-Cluster-Infrastructure

Wed, Feb 13

Legoktm added a project to T216003: Linter fails to detect multiple "upright" parameters as a Bogus file option: Parsoid-Linter.
Wed, Feb 13, 6:00 PM · Parsoid-Linter, MediaWiki-extensions-Linter
Legoktm added a project to T215999: Linter does not detect invalid "500px500px" as a bogus file option: Parsoid-Linter.
Wed, Feb 13, 6:00 PM · Parsoid-Linter, MediaWiki-extensions-Linter
Legoktm renamed T215992: CI permission errors causing tests to fail from CI permission errors causing tests to fial to CI permission errors causing tests to fail.
Wed, Feb 13, 4:32 AM · Continuous-Integration-Infrastructure
Legoktm renamed T215992: CI permission errors causing tests to fail from Many tests are currently failing :( to CI permission errors causing tests to fial.
Wed, Feb 13, 4:29 AM · Continuous-Integration-Infrastructure
Legoktm updated subscribers of T193671: Allow searching global rename log by old username via API.

@Anomie do you have suggestions on how this should be implemented? Ideally it would exist in core's list=logevents. We need something like the LogEventsListGetExtraInputs and SpecialLogAddLogSearchRelations hooks.

Wed, Feb 13, 4:17 AM · MediaWiki-API, MediaWiki-extensions-CentralAuth, GlobalRename
Legoktm renamed T170215: Assess how well developers are doing at updating extension.json's "requires" field for MediaWiki core versions, and provide guidance on how to improve it from Assess how well developers are doing at updating extension.json's "requires" field for MediaWiki core versions to Assess how well developers are doing at updating extension.json's "requires" field for MediaWiki core versions, and provide guidance on how to improve it.
Wed, Feb 13, 2:43 AM · MediaWiki-Configuration
Legoktm reopened T170215: Assess how well developers are doing at updating extension.json's "requires" field for MediaWiki core versions, and provide guidance on how to improve it as "Open".

Some people do update it, and now CodeSniffer uses it to see if it should warn about some deprecated stuff. Probably the more useful thing to focus on is "suggestions on how to improve it".

Wed, Feb 13, 2:43 AM · MediaWiki-Configuration
Reedy awarded T170215: Assess how well developers are doing at updating extension.json's "requires" field for MediaWiki core versions, and provide guidance on how to improve it a The World Burns token.
Wed, Feb 13, 2:40 AM · MediaWiki-Configuration
Legoktm added a comment to T213595: Release 1.32.1 as a maintenance release.

Can we bundle the planned security release with this?

Wed, Feb 13, 2:21 AM · Release-Engineering-Team (Kanban), MW-1.32-notes, MW-1.32-release
Legoktm added a comment to T215085: make phabricator search match projects based on alterenate hashtags.

@Legoktm what was the example that didn't work for you which we were investigating at all-hands?

Wed, Feb 13, 12:15 AM · User-MModell, Release-Engineering-Team (Kanban), Phabricator (Search)

Tue, Feb 12

Legoktm added a comment to T215757: Requesting maintainership of Extension:WebDAV.

Since it's just in SVN and abandoned, I think it's fine if you take over the "WebDAV" name and archive the existing extension on mediawiki.org. Since it doesn't exist in Gerrit, there's not really anything I can give you ownership over...

Tue, Feb 12, 7:55 AM · Repository-Ownership-Requests

Mon, Feb 11

Legoktm added a comment to T213318: Wikibase Front-End Architecture.

Do I understand this correctly, that this would add a mandatory Nodejs service to run a Wikibase installation? Is there no client side rendering support planned initially? As an sysadmin for couple of third party wikis (some of which use Wikibase) I am hoping that when Parsoid is ported to PHP, there would be no need for Nodejs for a "basic" installation.

I would hardly define Wikibase as "part of a basic installation".

Mon, Feb 11, 3:30 PM · Wikidata, TechCom-RFC

Sun, Feb 10

Legoktm claimed T215713: Missing dependencies in REL1_32 snapshot of some extensions.
2019-02-07 06:10:26,244 DEBUG:Running composer install for TemplateStyles
2019-02-07 06:10:31,408 ERROR:Traceback (most recent call last):
  File "/srv/extdist/nightly.py", line 190, in update_extension
    self.shell_exec([self.COMPOSER, 'install', '--no-dev'])
  File "/srv/extdist/nightly.py", line 142, in shell_exec
    return subprocess.check_output(args, **kwargs).decode('utf-8')
  File "/usr/lib/python3.4/subprocess.py", line 620, in check_output
    raise CalledProcessError(retcode, process.args, output=output)
subprocess.CalledProcessError: Command '['/srv/composer/vendor/bin/composer', 'install', '--no-dev']' returned non-zero exit status 2
Sun, Feb 10, 7:16 PM · VPS-project-Extdist, MW-1.32-release, ExtensionDistributor
Framawiki awarded T71489: Expose mwgrep functionality on-wiki a Love token.
Sun, Feb 10, 5:49 PM · Discovery-Search, Discovery, CirrusSearch

Fri, Feb 8

Legoktm added a comment to T215046: RfC: Use Github login for mediawiki.org.

Instead of approving github as a one-off, I'd rather we create a set of criteria that we use to evaluate 3rd party login providers, and then evaluate github (and others that will inevitably be raised) against that. We're already effectively going to be creating said criteria in the questions that we ask during the meeting, so we might as well formalize it so we have an consistent answer to "but why not X?"

Fri, Feb 8, 11:03 PM · User-Tgr, Privacy, Security, TechCom-RFC, Wikimedia-General-or-Unknown, GitHub-Mirrors
Legoktm added a comment to T215584: Add TimedMediaHandler to WMF-deployed search preset.

Fixed now, still wondering what would have caused this in the first place though. I wonder if it's the systemd timeouts that force automatic restarts leaving git processes stuck midway through?

Fri, Feb 8, 8:07 AM · VPS-project-codesearch
Legoktm claimed T215584: Add TimedMediaHandler to WMF-deployed search preset.

I also see this in the logs:

Feb 07 07:23:41 codesearch4 docker[9286]: 2019/02/07 07:23:41 Failed to git fetch /data/data/vcs-1a58778e8f1a8bb707a0050aab89f0c507eb9f37, see output below
Feb 07 07:23:41 codesearch4 docker[9286]: fatal: Unable to create '/data/data/vcs-1a58778e8f1a8bb707a0050aab89f0c507eb9f37/.git/shallow.lock': File exists.
Feb 07 07:23:41 codesearch4 docker[9286]: Another git process seems to be running in this repository, e.g.
Feb 07 07:23:41 codesearch4 docker[9286]: an editor opened by 'git commit'. Please make sure all processes
Feb 07 07:23:41 codesearch4 docker[9286]: are terminated then try again. If it still fails, a git process
Feb 07 07:23:41 codesearch4 docker[9286]: may have crashed in this repository earlier:
Feb 07 07:23:41 codesearch4 docker[9286]: remove the file manually to continue.
Feb 07 07:23:41 codesearch4 docker[9286]: Continuing...
Fri, Feb 8, 8:05 AM · VPS-project-codesearch
Legoktm added a comment to T215584: Add TimedMediaHandler to WMF-deployed search preset.
legoktm@codesearch4:/srv/hound/hound-deployed$ ack Timed config.json 
		"mediawiki/extensions/TimedMediaHandler": {
				"base-url": "https://gerrit.wikimedia.org/g/mediawiki/extensions/TimedMediaHandler/+/{rev}/{path}{anchor}"
			"url": "https://gerrit.wikimedia.org/r/mediawiki/extensions/TimedMediaHandler"
Fri, Feb 8, 8:00 AM · VPS-project-codesearch

Thu, Feb 7

Legoktm added a comment to T200034: Jenkins shouldn't post failures when "test coverage" is reduced because the number of lines of code are reduced..

For most files we can probably round to nearest integer, and maybe if the file has more than 500 or 1000 lines, then we go to 1-2 decimal points.

Thu, Feb 7, 7:04 PM · phpunit-patch-coverage
Legoktm closed T202470: phpunit-patch-coverage fails if a file was renamed as Resolved.
Thu, Feb 7, 6:40 PM · phpunit-patch-coverage
Legoktm added a comment to T191182: Stop using Differential for code review.

The other reason however is that the Gerrit/Zuul/Jenkins workflow is complex for new users to understand. It is a powerful system, but not something that we have invested in making easy. We instead invest in making it a good way to support the very specific needs of the movement's most sensitive software. This is not a bad thing, but I do not believe this is a one size fits all problem space. Fragmentation makes discovery and by extension reuse more difficult, but there are portions of our technical community, like Toolforge, where we are much earlier in the software development maturity cycle and added complexity in adoption of basic recommended practices will inhibit adoption.

Thu, Feb 7, 6:35 PM · Release-Engineering-Team (Backlog), Phabricator, Gerrit
Legoktm renamed T184657: Generate JavaScript code coverage reports for extensions from Generate JS code coverage reports for extensions to Generate JavaScript code coverage reports for extensions.
Thu, Feb 7, 6:17 PM · Test-Coverage, JavaScript, Continuous-Integration-Infrastructure
Legoktm added a comment to T215511: Jenkins is posting duplicate comments when a depended on change merge fails.

This is on purpose, and a good thing IMO. It can be a bit confusing if they're all at the same time, but often they're not.

Thu, Feb 7, 6:01 PM · User-Zppix, Release-Engineering-Team
Legoktm committed rMSCA6dd402160658: check_php_syntax: Check for any content before opening <?php tag (authored by Legoktm).
check_php_syntax: Check for any content before opening <?php tag
Thu, Feb 7, 12:10 PM
Legoktm committed rMSCA2b1ed48eff0c: check_php_syntax: Check for any content before opening <?php tag (authored by Legoktm).
check_php_syntax: Check for any content before opening <?php tag
Thu, Feb 7, 12:10 PM
Legoktm committed rMSCA86c179e13fa7: check_php_syntax: Check for any content before opening <?php tag (authored by Legoktm).
check_php_syntax: Check for any content before opening <?php tag
Thu, Feb 7, 12:10 PM
Legoktm committed rMSCA096e0c114ba0: check_php_syntax: Check for any content before opening <?php tag (authored by Legoktm).
check_php_syntax: Check for any content before opening <?php tag
Thu, Feb 7, 12:10 PM
Legoktm committed rMSCA35432e441217: Have utils.check_php_opening_tag check the file extension suffix (authored by Legoktm).
Have utils.check_php_opening_tag check the file extension suffix
Thu, Feb 7, 12:10 PM
Legoktm assigned T215444: mw.ext.data.get from JsonConfig doesn't work with PHP 7 to Anomie.
Thu, Feb 7, 8:53 AM · MW-1.33-notes (1.33.0-wmf.17; 2019-02-12), Patch-For-Review, PHP 7.2 support, Commons-Datasets, MediaWiki-extensions-JsonConfig

Wed, Feb 6

Legoktm closed T214179: mw.ext.data.get Lua call returns false as Resolved.

That's a separate issue then, so I've filed it as a separate task. Let's discuss it there: T215444: mw.ext.data.get from JsonConfig doesn't work with PHP 7.

Wed, Feb 6, 5:49 PM · MW-1.33-notes (1.33.0-wmf.17; 2019-02-12), Patch-For-Review, Regression, MediaWiki-extensions-JsonConfig, Commons-Datasets, MediaWiki-extensions-Scribunto
Legoktm triaged T215444: mw.ext.data.get from JsonConfig doesn't work with PHP 7 as High priority.
Wed, Feb 6, 5:48 PM · MW-1.33-notes (1.33.0-wmf.17; 2019-02-12), Patch-For-Review, PHP 7.2 support, Commons-Datasets, MediaWiki-extensions-JsonConfig
SerDIDG awarded T214179: mw.ext.data.get Lua call returns false a Burninate token.
Wed, Feb 6, 5:20 PM · MW-1.33-notes (1.33.0-wmf.17; 2019-02-12), Patch-For-Review, Regression, MediaWiki-extensions-JsonConfig, Commons-Datasets, MediaWiki-extensions-Scribunto
Legoktm committed rLLUP098426353cde: Escape the sniff name in regexes (authored by Legoktm).
Escape the sniff name in regexes
Wed, Feb 6, 9:07 AM
Legoktm committed rLLUPf808e2377b2f: Disable the prefetch_generator for now (authored by Legoktm).
Disable the prefetch_generator for now
Wed, Feb 6, 9:07 AM
Legoktm closed T215372: Upgrade phpcs to version 3.4.0 or later of PHP_CodeSniffer code to work around SelfMemberReference.NotUsed bug in anon classes as Resolved.

You can also just disable the rule temporarily in .phpcs.xml.

Wed, Feb 6, 4:42 AM · Patch-For-Review, MediaWiki-Codesniffer
Legoktm added a comment to T215372: Upgrade phpcs to version 3.4.0 or later of PHP_CodeSniffer code to work around SelfMemberReference.NotUsed bug in anon classes.

There was, but we may have merged it by now. I'll double check tonight.

Wed, Feb 6, 2:42 AM · Patch-For-Review, MediaWiki-Codesniffer
Legoktm added a comment to T208494: Allow music scores to be uploaded to Wikimedia Commons.

I'm not sure adding 5 different file types for the same type of content is the best idea, but okay...

I have to be honest, I'm confused by this: what harm is there in allowing (e.g.) PNG and GIF and TIFF and JPEG all at once? Yes, those formats have relative strengths and weaknesses but they are all (minus animated GIFs), static images. It's okay to have PDF and DJVU. I just don't see what is *worse* about including a broader array of free file formats. If one is clearly superior (e.g. how SVG is a better format for many static images than any raster option), then won't users generally gravitate toward that? Plus, it's sometimes cumbersome to convert files, so it's a good thing to allow an easy way to upload (e.g.) MP3s rather than require OGG conversion.

Wed, Feb 6, 12:52 AM · MediaWiki-extensions-Score, Commons

Mon, Feb 4

Legoktm committed rMCCL05eb3099b57d: Add 7.3 to travis.yml (authored by Legoktm).
Add 7.3 to travis.yml
Mon, Feb 4, 3:58 AM
Legoktm added a comment to T215115: Inline styles for patch-coverage HTML artefact blocked by CSP on integration.wikimedia.org.

Sorry about the domain confusion. It was supposed to be integration.wikimedia.org.

Mon, Feb 4, 1:28 AM · Jenkins, Release-Engineering-Team, phpunit-patch-coverage, Continuous-Integration-Infrastructure

Sun, Feb 3

Legoktm closed T196323: PHPUnit entry point 'PHPUnit_TextUI_Command' not found during mediawiki-phpunit-coverage-patch job as Declined.

The only reason I can think of this happening is some screwup when installing composer dependencies...in any case I haven't seen it recently. Please re-open if it is still occurring.

Sun, Feb 3, 9:40 AM · phpunit-patch-coverage
Legoktm merged T206230: phpunit-patch-coverage HTML reports are not being saved by jenkins into T205235: mediawiki-phpunit-coverage-patch-docker missing HTML report artifact.
Sun, Feb 3, 9:38 AM · Patch-For-Review, Continuous-Integration-Infrastructure
Legoktm merged task T206230: phpunit-patch-coverage HTML reports are not being saved by jenkins into T205235: mediawiki-phpunit-coverage-patch-docker missing HTML report artifact.
Sun, Feb 3, 9:38 AM · phpunit-patch-coverage, Continuous-Integration-Config
Legoktm claimed T202470: phpunit-patch-coverage fails if a file was renamed.
Sun, Feb 3, 9:32 AM · phpunit-patch-coverage
Legoktm changed the status of T203199: Separate test coverage "increased" from "stayed the same" from Open to Stalled.

This is a limitation in the current setup that jenkins/zuul can only communicate a binary state. So "stayed the same" has to go in either decreased or increased, we can't have a third state. My plan is to merge this into the main jenkins test and not have a separate message, so this won't be relevant for much longer I think.

Sun, Feb 3, 9:32 AM · phpunit-patch-coverage
Legoktm created T215115: Inline styles for patch-coverage HTML artefact blocked by CSP on integration.wikimedia.org.
Sun, Feb 3, 6:29 AM · Jenkins, Release-Engineering-Team, phpunit-patch-coverage, Continuous-Integration-Infrastructure
Legoktm closed T215047: GraphQL extension should use -composer jobs in CI as Resolved.
Sun, Feb 3, 4:46 AM · GraphQL, Patch-For-Review, Continuous-Integration-Config
Legoktm added a comment to T214631: Create a plugin that add's recheck as a button to PolyGerrit's ui.

Adding 'recheck' sounds like a good idea. 'check experimental' is just supposed to be for the CI team testing new jobs and stuff - I don't think that should be exposed to users.

Sun, Feb 3, 3:55 AM · Gerrit

Sat, Feb 2

Legoktm added a comment to T215107: Global rename of The_Photographer → Wilfredor: supervision needed.

Can we not rename users with large edit counts back and forth (c.f. https://meta.wikimedia.org/wiki/Special:GlobalRenameProgress?username=The_Photographer)? Will this be the user's last rename request?

Sat, Feb 2, 10:59 PM · Patch-For-Review, User-MarcoAurelio, DBA, Wikimedia-Site-requests
Legoktm created T215110: ParsoidBatchAPI timeout on frwikisource due to Score extension.
Sat, Feb 2, 10:29 PM · Parsoid, MediaWiki-extensions-Score
Legoktm renamed T215084: Stop depending on jQuery in OOUI from Stop depending on jQuery to Stop depending on jQuery in OOUI.
Sat, Feb 2, 10:13 PM · Epic, OOUI

Fri, Feb 1

Legoktm added a comment to T215072: Request creation of globalcu VPS project.

I assume the proxy will filter out all private information, but if it doesn't (thinking about UA), we can stick the banner from https://wikitech.wikimedia.org/wiki/Wikitech:Cloud_Services_Terms_of_use#If_my_tools_collect_Private_Information... in the wiki's sitenotice.

Fri, Feb 1, 4:44 PM · cloud-services-team (Kanban), Cloud-VPS (Project-requests)
Legoktm changed the status of T214820: Enable CheckUser for beta cluster from Open to Stalled.

I need the latter, Similar or close settings with WMF production environment.

Fri, Feb 1, 4:43 PM · Trust-and-Safety, WMF-Legal, Patch-For-Review, Beta-Cluster-Infrastructure, User-Rxy, CheckUser
Legoktm changed the status of T214820: Enable CheckUser for beta cluster, a subtask of T212779: Implement Global CheckUser, from Open to Stalled.
Fri, Feb 1, 4:43 PM · User-Rxy, Epic, User-notice, WMF-Legal, Trust-and-Safety, Stewards-and-global-tools, CheckUser
Legoktm created T215072: Request creation of globalcu VPS project.
Fri, Feb 1, 4:34 PM · cloud-services-team (Kanban), Cloud-VPS (Project-requests)
Legoktm renamed T215066: Request to add alaasarhan to the wmde LDAP group from Request to be added to the ldap/wmde group to Request to add alaasarhan to the wmde LDAP group.
Fri, Feb 1, 4:29 PM · WMF-Legal, LDAP-Access-Requests, Operations, WMF-NDA-Requests
Legoktm updated the task description for T178703: Add a hidden comment to WikiLove's output stating where it came from.
Fri, Feb 1, 7:50 AM · Patch-For-Review, Growth-Team, good first bug, WikiLove, Collaboration-Team-Triage
Legoktm added a comment to T214618: Requesting repository ownership for /mediawiki/extensions/OdbcDatabase.

Sorry about the delay in responding, and thanks for volunteering. Can you post a patch in Gerrit with some of your changes so far? (See https://www.mediawiki.org/wiki/Gerrit for links on how to get started).

Fri, Feb 1, 5:55 AM · Repository-Ownership-Requests
Legoktm renamed T215047: GraphQL extension should use -composer jobs in CI from Jenkins does not load Composer's autoloader even if load_composer_autoloader is set to true to GraphQL extension should use -composer jobs in CI.
Fri, Feb 1, 1:51 AM · GraphQL, Patch-For-Review, Continuous-Integration-Config
Legoktm edited projects for T215047: GraphQL extension should use -composer jobs in CI, added: Continuous-Integration-Config; removed Continuous-Integration-Infrastructure, Jenkins.

That's not the problem. The problem is that your extension needs to use the -composer job variants instead of the vendor ones, so that composer dependencies are pulled in when PHPUnit tests are run.

Fri, Feb 1, 1:50 AM · GraphQL, Patch-For-Review, Continuous-Integration-Config

Thu, Jan 31

Legoktm updated subscribers of T215000: Fill gaps in PHP DOM's functionality.
Thu, Jan 31, 8:39 PM · Patch-For-Review, Parsoid-PHP
D3r1ck01 awarded T182744: $wgMimeDetectorCommand should have shell restrictions applied a Like token.
Thu, Jan 31, 6:29 PM · MediaWiki-File-management, MediaWiki-Shell, Commons
Legoktm awarded T214998: Remove .m. subdomain, serve mobile and desktop variants through the same URL a Love token.
Thu, Jan 31, 6:13 AM · Readers-Web-Backlog (Tracking), Traffic, Operations, MobileFrontend
Legoktm added a comment to T214998: Remove .m. subdomain, serve mobile and desktop variants through the same URL.

The m. subdomain, as in en.m.wikipedia.org, is annoying. Links shared on social media are randomly mobile or non-mobile, and so desktop users often accidentally visit the mobile site.

Thu, Jan 31, 6:13 AM · Readers-Web-Backlog (Tracking), Traffic, Operations, MobileFrontend

Wed, Jan 30

Legoktm added a comment to T88434: generate Wikibase.git code coverage on Jenkins.

That's one possible option, yes. Or you can adopt the standard layout in one repo, but slightly reversed: includes/lib/, includes/repo/, tests/phpunit/client/. That's more up to the maintainers of Wikibase to figure out, but CI (and a growing number of tools) do assume the standard layout.

Wed, Jan 30, 7:04 PM · Test-Coverage, Continuous-Integration-Infrastructure, MediaWiki-extensions-WikibaseRepository, Wikidata
Legoktm changed the status of T88434: generate Wikibase.git code coverage on Jenkins from Open to Stalled.

This is blocked on Wikibase adopting the standard directory layout, specifically what is documented at https://www.mediawiki.org/wiki/Continuous_integration/Tutorials/Generating_PHP_test_coverage_for_a_MediaWiki_extension#Requirements

Wed, Jan 30, 8:26 AM · Test-Coverage, Continuous-Integration-Infrastructure, MediaWiki-extensions-WikibaseRepository, Wikidata
Legoktm changed the status of T88434: generate Wikibase.git code coverage on Jenkins, a subtask of T71685: Generate PHP code coverage reports for extensions, from Open to Stalled.
Wed, Jan 30, 8:26 AM · MW-1.31-release-notes (WMF-deploy-2018-01-09 (1.31.0-wmf.16)), Patch-For-Review, MediaWiki-Platform-Team (MWPT-Q3-Jan-Mar-2018), Continuous-Integration-Infrastructure

Tue, Jan 29

Legoktm added a comment to T184657: Generate JavaScript code coverage reports for extensions.

@Krinkle implemented this in 0e552715e52c4e757712edb201f05ef5e5eafff9 in the examples extension. It will most likely require re-writing some code to fit in the nodejs only qunit model as well as some CI improvements so it gets displayed in a nice form on doc.wm.o.

Tue, Jan 29, 9:24 PM · Test-Coverage, JavaScript, Continuous-Integration-Infrastructure
MarcoAurelio awarded T214870: Liberate disk space on integration-slave-jessie-1001 a Barnstar token.
Tue, Jan 29, 11:47 AM · Continuous-Integration-Infrastructure, Release-Engineering-Team
Legoktm added a comment to T214876: Sqwiki / Need only registered users to create new articles.

@1l2l3k have you tried setting up AbuseFilter rules to stem the vandalism? Detecting empty new articles should be trivial to catch.

Tue, Jan 29, 9:19 AM · Wikimedia-Site-requests
Legoktm renamed T170360: Moderation and protection feature parity in Flow (tracking) from Moderation and protection feature parity (tracking) to Moderation and protection feature parity in Flow (tracking).
Tue, Jan 29, 9:18 AM · Growth-Team, Collaboration-Team-Triage, Epic, StructuredDiscussions
Ricordisamoa awarded T195689: Support PHP 7.x webservices on Toolforge a Love token.
Tue, Jan 29, 9:17 AM · Toolforge
Legoktm added a comment to T163061: Flow: It's impossible hide/delete the title of topic.

Is there any workaround available right now? This has been abused on mediawiki.org recently.

Tue, Jan 29, 9:17 AM · Growth-Team, Collaboration-Team-Triage, StructuredDiscussions
Legoktm added a comment to T174339: Allow use of phan 0.8.5+ in wikimedia CI.

It is possible to bundle the phan version with phan-config? That would it make easier to write own plug-ins and be sure to have the correct phan version to run.

Tue, Jan 29, 9:10 AM · Patch-For-Review, User-Addshore, Continuous-Integration-Config