Paladox (Paladox)
User

Projects (29)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Oct 7 2014, 9:51 AM (201 w, 1 d)
Availability
Available
LDAP User
Paladox
MediaWiki User
Unknown

I live in the United Kingdom GMT/BST(UTC+0, UTC+1)

Recent Activity

Today

Paladox committed rGERRITDEPLOY8de4ff40960c: Add gerrit-theme.html and also add footer links (authored by Paladox).
Add gerrit-theme.html and also add footer links
Wed, Aug 15, 2:04 PM

Yesterday

Paladox added a comment to T197083: Gerrit has created duplicate accounts for some users.

Quick update. I pushed up a new patchset for review for the plugin we're working on to resolve the problem in the notedb database: https://gerrit-review.googlesource.com/c/plugins/wmf-fixshadowuser/+/190030

There is no documentation for how to delete an account using notedb since it's new and upstream doesn't want users to be deleted (https://groups.google.com/d/msg/repo-discuss/JlV0i9w5lCY/F25h2t_HQzQJ). I'll ask how to do this and what the ramifications are if you're fine with losing all history.

Tue, Aug 14, 9:34 PM · Release-Engineering-Team (Kanban), Gerrit
Paladox added a member for Trusted-Contributors: jijiki.
Tue, Aug 14, 8:35 PM
Paladox changed the status of T201953: Install rename-project plugin from Open to Stalled.

Stalled until notedb is supported in the plugin.

Tue, Aug 14, 7:09 PM · Release-Engineering-Team, Gerrit
Paladox created T201953: Install rename-project plugin.
Tue, Aug 14, 6:54 PM · Release-Engineering-Team, Gerrit
Paladox added a comment to T199463: No keyboard shortcut for adding reviewers in PolyGerrit.

I am not sure what the key is on windows / linux maybe ctrl + A but on the mac it's FN + A that opens the dialog where you write replies but also the one that adds reviewers.

Tue, Aug 14, 3:37 PM · Upstream, Gerrit

Mon, Aug 13

Paladox closed D1097: keystone: refresh keystone server URL.

Change has been merged but this does not auto close this revision.

Mon, Aug 13, 6:28 PM
Paladox created P7453 (An Untitled Masterwork).
Mon, Aug 13, 6:26 PM
Paladox committed R2073:a27c6c0a9592: keystone: refresh keystone server URL (authored by Paladox).
keystone: refresh keystone server URL
Mon, Aug 13, 6:25 PM

Sat, Aug 11

Paladox added a comment to T201750: Gitiles should add username in git clone links.

Duplicate of T189182 ?

Sat, Aug 11, 11:42 AM · Upstream, Gerrit
Paladox added a comment to T201744: Zuul should not start gate-and-submit for WIP patch sets.

“Ready for review"” can be set by project members now

Sat, Aug 11, 11:40 AM · Continuous-Integration-Infrastructure, Gerrit, Jenkins, Zuul

Fri, Aug 10

Paladox committed rECW8c1fc8c1a486: Add support for using geoip2 (authored by Paladox).
Add support for using geoip2
Fri, Aug 10, 7:45 PM

Thu, Aug 9

Paladox added a comment to T201460: Unable to log in to Phabricator via MediaWiki on mobile (CSP error).

Oh sorry that I didn’t respond. Sorry that I was wrong or caused spam. Must be something else then?

Thu, Aug 9, 7:21 PM · Release-Engineering-Team (Kanban), Phabricator

Wed, Aug 8

Paladox added a comment to T201460: Unable to log in to Phabricator via MediaWiki on mobile (CSP error).

Im guessing this problem is the new anti vandalism stuff.

Wed, Aug 8, 12:23 AM · Release-Engineering-Team (Kanban), Phabricator

Tue, Aug 7

Paladox created T201457: Setup wikiapiary to use varnish.
Tue, Aug 7, 7:50 PM · WikiApiary
Paladox created T201455: Install I18nTags mw extension on wikiapiary.
Tue, Aug 7, 7:47 PM · WikiApiary
Paladox updated the task description for T200739: Place holder task for Gerrit 2.16 upgrade.
Tue, Aug 7, 5:56 PM · Gerrit
Paladox added a comment to T196812: Make PolyGerrit the default ui.

Yup sure :)

Tue, Aug 7, 3:39 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T200492: Undefined property: stdClass::$detail in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.

Yup i think so.

Tue, Aug 7, 1:23 PM · Math
Paladox added a comment to T200493: Trying to get property 'status' of non-object in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.

I have this set https://github.com/miraheze/mw-config/blob/master/LocalWiki.php#L59 and this is my mathoid config https://github.com/miraheze/puppet/blob/master/modules/mathoid/files/config.yaml

Tue, Aug 7, 1:23 PM · Math
Paladox added a comment to T200492: Undefined property: stdClass::$detail in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.
Tue, Aug 7, 1:22 PM · Math
Paladox committed rECWce9c5a24e1c5: Add support for using geoip2 (authored by Paladox).
Add support for using geoip2
Tue, Aug 7, 1:01 PM
Paladox committed rECW3f2c67fde6e5: Add support for using geoip2 (authored by Paladox).
Add support for using geoip2
Tue, Aug 7, 1:01 PM

Mon, Aug 6

Paladox added a comment to T196812: Make PolyGerrit the default ui.

Thanks for the reasoning.

I'm not terribly inclined to use our user base as test subjects, especially when there are known issues that are not yet resolved on our install. But, if it's easy for them to go back to the old UI then maybe that's OK.

To be clear, in which upstream release will the old UI be removed?

Mon, Aug 6, 10:01 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T196812: Make PolyGerrit the default ui.

So why not wait until we upgrade to 2.16?

We can wait if you want?

I would prefer solid reasoning for doing it now or solid reasoning for doing it later. Given we have a choice that needs to be made we need reasons to do it or to wait. Can you help with those?

Mon, Aug 6, 9:18 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox created T201359: Tests are broken in CookieWarning.
Mon, Aug 6, 8:08 PM · Patch-For-Review, MediaWiki-extensions-CookieWarning
Paladox added a comment to T196812: Make PolyGerrit the default ui.

2.16 does include that feature :)

Mon, Aug 6, 6:35 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T196812: Make PolyGerrit the default ui.

We can wait if you want?

Mon, Aug 6, 6:30 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T196812: Make PolyGerrit the default ui.

@greg PolyGerrit has been made the default UI as of 2.16. David O (upstream) has a change to remove gwtui (https://gerrit-review.googlesource.com/c/gerrit/+/116790), but wants to keeps gwtui for at least one more release so 3.0 could possibly be the release marked for GWTUI removal.

Mon, Aug 6, 5:39 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T196812: Make PolyGerrit the default ui.
  1. This is set through a cookie so it's per browser.
Mon, Aug 6, 5:22 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit

Sun, Aug 5

Paladox created T201267: count(): Parameter must be an array or an object that implements Countable .
Sun, Aug 5, 3:22 PM · MediaWiki-extensions-CentralAuth

Sat, Aug 4

Paladox added a comment to T201246: Make a PolyGerrit plugin to welcome new users.

we will want to hook into this https://github.com/GerritCodeReview/gerrit/blob/master/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.html#L555

Sat, Aug 4, 8:04 PM · Gerrit
Paladox added a parent task for T200739: Place holder task for Gerrit 2.16 upgrade: T201246: Make a PolyGerrit plugin to welcome new users.
Sat, Aug 4, 8:04 PM · Gerrit
Paladox added a subtask for T201246: Make a PolyGerrit plugin to welcome new users: T200739: Place holder task for Gerrit 2.16 upgrade.
Sat, Aug 4, 8:04 PM · Gerrit
Paladox moved T201246: Make a PolyGerrit plugin to welcome new users from Bugs & stuff to PolyGerrit on the Gerrit board.
Sat, Aug 4, 8:01 PM · Gerrit
Paladox created T201246: Make a PolyGerrit plugin to welcome new users.
Sat, Aug 4, 8:01 PM · Gerrit
Paladox added a comment to T200739: Place holder task for Gerrit 2.16 upgrade.

Also this https://gerrit-review.googlesource.com/c/gerrit/+/191610 has just been uploaded, that will help new users.

Sat, Aug 4, 7:55 PM · Gerrit
Paladox updated the task description for T200739: Place holder task for Gerrit 2.16 upgrade.
Sat, Aug 4, 7:52 PM · Gerrit
Paladox closed T201243: Special:UploadWizard shows "Failed to load resource: the server responded with a status of 404 ()" in the console log as Declined.

After forcing a cache refresh it seems to have worked.

Sat, Aug 4, 6:26 PM · UploadWizard, Multimedia
Paladox created T201243: Special:UploadWizard shows "Failed to load resource: the server responded with a status of 404 ()" in the console log.
Sat, Aug 4, 6:22 PM · UploadWizard, Multimedia

Fri, Aug 3

Paladox added a comment to T163242: git-review fails with "The requested URL /changes/ was not found on this server.".

@CCicalese_WMF 1.26.0 should fix your issue (it was fixed in that release)

Fri, Aug 3, 5:18 PM · Patch-For-Review, Gerrit, MediaWiki-Vagrant
Paladox added a comment to T163242: git-review fails with "The requested URL /changes/ was not found on this server.".

@CCicalese_WMF hi, did you update to git-review 1.26.0?

Fri, Aug 3, 5:13 PM · Patch-For-Review, Gerrit, MediaWiki-Vagrant
Paladox added a comment to T179759: Update Wikimedia's Gerrit documentation for new user interface in Gerrit 2.15.

Well for starters having just screenshots of the old ui and none for the new ui. Will confuse a new user more.

Fri, Aug 3, 3:03 PM · Developer-Advocacy, Documentation, Gerrit
Paladox added a comment to T179759: Update Wikimedia's Gerrit documentation for new user interface in Gerrit 2.15.

the old ui screen shots can be removed when ever upstream removes it (and we update to that version) :).

Fri, Aug 3, 1:27 PM · Developer-Advocacy, Documentation, Gerrit
Paladox added a comment to T179759: Update Wikimedia's Gerrit documentation for new user interface in Gerrit 2.15.

I've updated some of the screenshots to have both the old ui and new ui.

Fri, Aug 3, 12:11 AM · Developer-Advocacy, Documentation, Gerrit

Thu, Aug 2

Paladox added a comment to T196812: Make PolyGerrit the default ui.

I've updated some of those screenshots to include polygerrit. Including the extra expand diff view (you don't have to go inside the diff to view it).

Thu, Aug 2, 11:56 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T196812: Make PolyGerrit the default ui.

I got approval (after speaking with releng).

Thu, Aug 2, 9:23 PM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
Paladox added a comment to T199737: Thumbnails of certain SVG files fail to render.

I think i found the problem. It is because of wgSVGMetadataCutoff.

Thu, Aug 2, 2:36 AM · Patch-For-Review, Multimedia, Commons, MediaWiki-File-management
Paladox added a comment to T199737: Thumbnails of certain SVG files fail to render.

@saper did you test that against stretch?

Thu, Aug 2, 1:00 AM · Patch-For-Review, Multimedia, Commons, MediaWiki-File-management
Paladox added a comment to T199737: Thumbnails of certain SVG files fail to render.

I get this error:

Thu, Aug 2, 12:59 AM · Patch-For-Review, Multimedia, Commons, MediaWiki-File-management

Tue, Jul 31

Paladox committed rECWd9e132330aa4: Add support for using geoip2 (authored by Paladox).
Add support for using geoip2
Tue, Jul 31, 10:34 PM
Paladox added a comment to T200736: Rename Gerrit repository "LdapGroups" to "LDAPGroups".

We should also make the group own its self.

Tue, Jul 31, 1:52 PM · Gerrit
zeljkofilipin awarded T196812: Make PolyGerrit the default ui a Hungry Hippo token.
Tue, Jul 31, 10:29 AM · User-notice, Release-Engineering-Team, Patch-For-Review, Gerrit
TheDJ awarded T111909: Enable vp9 by default a Love token.
Tue, Jul 31, 8:36 AM · TimedMediaHandler-Transcode, TimedMediaHandler
Paladox added a comment to T200749: Move navigation bar above the image.

Like this:

Tue, Jul 31, 1:44 AM · wikimediafoundation.org
Paladox created T200750: Wrong translation on the english version.
Tue, Jul 31, 1:39 AM · wikimediafoundation.org
Paladox created T200749: Move navigation bar above the image.
Tue, Jul 31, 1:37 AM · wikimediafoundation.org
Paladox added a comment to T195235: Research Project Idea: Use AI to suggest improvements to patches uploaded to gerrit.

You can use this api https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#apply-fix i think (I'm not sure where the rest api is to suggest changes though)

Tue, Jul 31, 1:15 AM · Scoring-platform-team, ORES, Gerrit
Paladox edited projects for T174034: Migrate to NoteDb, added: Release-Engineering-Team; removed Patch-For-Review.
Tue, Jul 31, 1:12 AM · Release-Engineering-Team, Gerrit
Paladox closed T174034: Migrate to NoteDb as Resolved.

This has been resolved now :).

Tue, Jul 31, 1:12 AM · Release-Engineering-Team, Gerrit
Paladox closed T174034: Migrate to NoteDb, a subtask of T37534: Free-form tagging in gerrit, as Resolved.
Tue, Jul 31, 1:12 AM · Patch-For-Review, Developer-Wishlist (2017), Upstream, Gerrit
Paladox closed T174034: Migrate to NoteDb, a subtask of T145885: Gerrit shows HTTP 500 error when pasting extended unicode characters, as Resolved.
Tue, Jul 31, 1:12 AM · Release-Engineering-Team (Someday), Patch-For-Review, Operations, Gerrit
Paladox moved T200663: Mirror TemplateWizard on Github from Bugs & stuff to Repo Admin on the Gerrit board.
Tue, Jul 31, 1:10 AM · Gerrit, GitHub-Mirrors, Community-Tech, MediaWiki-extensions-TemplateWizard
Paladox moved T200311: Access to create gerrit groups for WMDE-Leszek from Bugs & stuff to Repo Admin on the Gerrit board.

We could add the user to this https://gerrit.wikimedia.org/r/admin/groups/119,members group which will grant them "create repos" and "create groups" rights in addition to other rights.

Tue, Jul 31, 1:09 AM · Release-Engineering-Team, User-Addshore, Gerrit
Paladox closed T197238: No way to mark patches as not WIP (which blocks merging) as Resolved.

Closing as resolved as 2.15.3 should have resolved most of these issues with only the admin being able to unmark wips :)

Tue, Jul 31, 12:06 AM · Gerrit
Paladox added a comment to T200739: Place holder task for Gerrit 2.16 upgrade.

PolyGerrit's change view is getting a minor redesign on top of the one already done in 2.16.

Tue, Jul 31, 12:05 AM · Gerrit
Paladox added a parent task for T200739: Place holder task for Gerrit 2.16 upgrade: T200310: "Included groups" do not appear in New gerrit UI.
Tue, Jul 31, 12:04 AM · Gerrit
Paladox added a subtask for T200310: "Included groups" do not appear in New gerrit UI: T200739: Place holder task for Gerrit 2.16 upgrade.
Tue, Jul 31, 12:04 AM · User-Addshore, Gerrit
Paladox added parent tasks for T200739: Place holder task for Gerrit 2.16 upgrade: T195280: New Gerrit UI does not contain copyable Change-Id or parent commit id, T186406: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits, T196895: PolyGerrit doesn't support old UI URLs.
Tue, Jul 31, 12:02 AM · Gerrit
Paladox added a subtask for T186406: Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits: T200739: Place holder task for Gerrit 2.16 upgrade.
Tue, Jul 31, 12:02 AM · Upstream, Gerrit
Paladox added a subtask for T195280: New Gerrit UI does not contain copyable Change-Id or parent commit id: T200739: Place holder task for Gerrit 2.16 upgrade.
Tue, Jul 31, 12:02 AM · Upstream, Gerrit

Mon, Jul 30

Paladox triaged T200739: Place holder task for Gerrit 2.16 upgrade as Lowest priority.
Mon, Jul 30, 11:48 PM · Gerrit
Paladox created T200739: Place holder task for Gerrit 2.16 upgrade.
Mon, Jul 30, 11:48 PM · Gerrit
Paladox moved T200736: Rename Gerrit repository "LdapGroups" to "LDAPGroups" from Bugs & stuff to Repo Admin on the Gerrit board.
Mon, Jul 30, 11:38 PM · Gerrit
Paladox added a project to T200736: Rename Gerrit repository "LdapGroups" to "LDAPGroups": Gerrit.
Mon, Jul 30, 11:38 PM · Gerrit
Paladox added a comment to T200702: jenkins-bot is sometimes not added as CC.

jenkins-bot does not need to be in cc to be able to run the tests.

Mon, Jul 30, 3:20 PM · Continuous-Integration-Infrastructure

Sun, Jul 29

Paladox closed T196413: PHP Notice: Array to string conversion in /srv/mediawiki/w/includes/libs/objectcache/BagOStuff.php on line 741 as Declined.

I found where are problem was, caused by a configuration error.

Sun, Jul 29, 1:27 AM · MediaWiki-General-or-Unknown

Fri, Jul 27

Paladox closed T186414: Support token or cert in conduit.connect as Declined.

Nope, can decline it for now.

Fri, Jul 27, 6:55 PM · Phabricator
Paladox updated subscribers of T200117: WikiApiary: extension usage (version matrix) is greatly outdated.
Fri, Jul 27, 3:57 PM · WikiApiary
Paladox created T200531: Move wikiapriary to use puppet (and open source repo).
Fri, Jul 27, 3:57 PM · WikiApiary
Paladox created T200530: extension: pages are not updated the mw version a wiki is running that extension on.
Fri, Jul 27, 3:55 PM · WikiApiary
Paladox created T200528: Fix template Template:General_siteinfo and also update.
Fri, Jul 27, 3:53 PM · WikiApiary
Paladox updated the task description for T200492: Undefined property: stdClass::$detail in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.
Fri, Jul 27, 3:02 AM · Math
Paladox added a comment to T200492: Undefined property: stdClass::$detail in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.

i also see

Fri, Jul 27, 3:02 AM · Math
Paladox created T200493: Trying to get property 'status' of non-object in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.
Fri, Jul 27, 3:01 AM · Math
Paladox created T200492: Undefined property: stdClass::$detail in /srv/mediawiki/w/extensions/Math/MathMathMLCli.php on line 79.
Fri, Jul 27, 3:00 AM · Math

Thu, Jul 26

Paladox committed rGERRITDEPLOY82ae006c2540: Add gerrit-theme.html and also add footer links (authored by Paladox).
Add gerrit-theme.html and also add footer links
Thu, Jul 26, 7:36 PM

Wed, Jul 25

Paladox committed rGERRITDEPLOY09097b78a87e: Fix updated description not showing in file list (authored by Becky Siegel <beckysiegel@google.com>).
Fix updated description not showing in file list
Wed, Jul 25, 8:24 PM
Paladox added a comment to T116816: beta cluster missing vips command needed to render tiffs and pngs, pnmtojpeg for DjVus.

pnmtojpeg: command not found

Wed, Jul 25, 1:10 PM · Beta-Cluster-Infrastructure
Paladox added a comment to T200310: "Included groups" do not appear in New gerrit UI.

When I built that, I didn’t think about the rest api could be different heh.

Wed, Jul 25, 10:09 AM · User-Addshore, Gerrit

Tue, Jul 24

Paladox added a project to T200275: Be able to fetch timezones easily: MediaWiki-General-or-Unknown.
Tue, Jul 24, 3:36 PM · MediaWiki-General-or-Unknown

Mon, Jul 23

Paladox added a comment to T200230: Missing commit history at Gerrit's CreatedPagesList repository.

if you were to clone the repo from GitHub and then changed the .git config file to point to the gerrit repo and push then yes it would overwrite the data (i think)

Mon, Jul 23, 9:30 PM · Gerrit
Paladox added a comment to T200230: Missing commit history at Gerrit's CreatedPagesList repository.

Was that repo imported from another repo (hosted else where?) if yes, then importing will overwrite i think.

Mon, Jul 23, 9:14 PM · Gerrit
Paladox added a comment to T200230: Missing commit history at Gerrit's CreatedPagesList repository.

Did someone force push? I see this https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/CreatedPagesList/+/6339e603fbc15c49b0c6b298aa241dc6fe7f0684 which was force push as it dosen't have a change id.

Mon, Jul 23, 9:13 PM · Gerrit

Sun, Jul 22

Paladox created T200169: Warning: Cannot modify header information - headers already sent by in /vagrant/mediawiki/includes/WebResponse.php on line 72.
Sun, Jul 22, 10:31 PM · MediaWiki-General-or-Unknown
Paladox accepted D1083: Use `?target=$rawusername` to avoid encoding issues.
Sun, Jul 22, 11:54 AM · Wikimedia-Phabricator-Extensions

Sat, Jul 21

Paladox added a comment to T200135: using the importDump.php script is adding imported> to the username field for revision.

We are importing from a import a user gave us to import. Just found that import> prefix to be weird.

Sat, Jul 21, 10:58 PM · MediaWiki-Export-or-Import
Paladox created T200135: using the importDump.php script is adding imported> to the username field for revision.
Sat, Jul 21, 8:00 PM · MediaWiki-Export-or-Import

Fri, Jul 20

Paladox committed rECW23b67c1a6dbe: Add support for using geoip2 (authored by Paladox).
Add support for using geoip2
Fri, Jul 20, 2:37 PM
Paladox committed rECWf3c556df8aef: Add support for using geoip2 (authored by Paladox).
Add support for using geoip2
Fri, Jul 20, 2:34 PM