Gilles (Gilles Dubuc)
Senior Performance Engineer, WMF

Projects (20)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Oct 7 2014, 8:24 AM (150 w, 1 d)
Availability
Available
IRC Nick
gilles
LDAP User
Gilles
MediaWiki User
Unknown

Recent Activity

Fri, Aug 18

Gilles claimed T144479: Ensure thumbor container access is preserved by mw filebackend setzoneaccess.
Fri, Aug 18, 4:17 PM · MediaWiki-Maintenance-scripts, Operations, Performance-Team, Thumbor
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.

I don't think anyone criticized the value of the feature, it's a great feature! But launching it in this state to all users in production was too rushed. Opt-in would have been more sensible and this issue discovered/worked on before deploying it to all enwiki users. Bandwidth is very expensive for some people, a 60% increase of anon first pageview JS weight is considerable. Things written by staff and volunteers alike get rolled back for similar reasons all the time, it's never personal.

Fri, Aug 18, 4:16 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)

Thu, Aug 17

Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

Running the stress test again, requesting about 2000 uncached thumbnails of the same image with a concurrency of 200 requests, I got:

Thu, Aug 17, 10:28 AM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles closed T172939: Thumbor webp handling as Resolved.

Works fine now: https://en.wikipedia.org/wiki/File:PlayMagnus.webp

Thu, Aug 17, 8:53 AM · Patch-For-Review, Performance-Team, Thumbor

Wed, Aug 16

Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

I've filed a revert for the 1-connection-per-backend: https://gerrit.wikimedia.org/r/#/c/372199/ On Vagrant, while reproducing the conditions above, adding the maxconn setting causes 502s instantly. Without the max connection setting, a given thumbor instance can switch between several concurrent requests just fine via yielding. In essence, a given thumbor process is capable of serving concurrent requests and therefore, concurrent connections.

Wed, Aug 16, 10:45 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles lowered the priority of T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview from Unbreak Now! to Normal.
Wed, Aug 16, 10:40 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.

Thank you, lowering priority back to normal

Wed, Aug 16, 10:40 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.

The graph for anons:

Wed, Aug 16, 10:28 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles renamed T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview from Extra 100KB of JS loaded by mobilemaps gadget on pageview for authenticated mobile users to Extra 100KB of JS loaded by mobilemaps gadget on pageview.
Wed, Aug 16, 10:27 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles raised the priority of T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview from High to Unbreak Now!.
Wed, Aug 16, 10:25 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.

Indeed, it seems like this applies to anonymous users as well. It works fine for Sweden, but same as you it seems to be broken for the Sierra Leone article, where the tiles all seem to 404. @TheDJ could this gadget be undeployed until these issues are resolved?

Wed, Aug 16, 10:25 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles removed a project from T171909: Does TemplateStyles work well with the preview feature?: Performance-Team.
Wed, Aug 16, 7:39 PM · TemplateStyles
Gilles moved T172357: $wgLocalVirtualHosts should include login.wikimedia.org, wikidata.org and others? from Inbox to Next-up on the Performance-Team board.
Wed, Aug 16, 7:14 PM · Patch-For-Review, Performance-Team, Deployment-Systems, MediaWiki-extensions-CentralAuth
Gilles assigned T172357: $wgLocalVirtualHosts should include login.wikimedia.org, wikidata.org and others? to Krinkle.
Wed, Aug 16, 7:14 PM · Patch-For-Review, Performance-Team, Deployment-Systems, MediaWiki-extensions-CentralAuth
Gilles moved T172479: Collect error logs from jobchron/jobrunner services in Logstash from Inbox to Backlog on the Performance-Team board.
Wed, Aug 16, 7:03 PM · Performance-Team, Release-Engineering-Team (Watching / External), Operations, monitoring, JobRunner-Service, Wikimedia-Incident
Gilles triaged T172479: Collect error logs from jobchron/jobrunner services in Logstash as Low priority.
Wed, Aug 16, 7:03 PM · Performance-Team, Release-Engineering-Team (Watching / External), Operations, monitoring, JobRunner-Service, Wikimedia-Incident
Gilles moved T172930: Long running thumbnail requests locking up Thumbor instances from Inbox to Doing on the Performance-Team board.
Wed, Aug 16, 7:02 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles moved T172939: Thumbor webp handling from Inbox to Doing on the Performance-Team board.
Wed, Aug 16, 7:00 PM · Patch-For-Review, Performance-Team, Thumbor
Gilles moved T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview from Inbox to Radar on the Performance-Team board.
Wed, Aug 16, 7:00 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles moved T173294: Investigate periodical highs in SpeedIndex for mobile from Inbox to Doing on the Performance-Team board.
Wed, Aug 16, 6:58 PM · Performance-Team
Gilles moved T173434: Consider using more common image sizes for Page previews from Inbox to Radar on the Performance-Team board.
Wed, Aug 16, 6:55 PM · Readers-Web-Backlog (Design), Performance-Team (Radar), Page-Previews
Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

Some relieving news: I've tested a specific lock (per-original) and the event-based async-like behavior of thumbor works perfectly:

Wed, Aug 16, 6:28 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles created T173434: Consider using more common image sizes for Page previews.
Wed, Aug 16, 5:24 PM · Readers-Web-Backlog (Design), Performance-Team (Radar), Page-Previews
Gilles added a comment to T170817: Upgrade Thumbor servers to Stretch.

OK, so if I'm following that means people are now advised to use other fonts than these ones, right? Meaning it's ok if those specific fonts don't render "right" on https://commons.wikimedia.org/wiki/File:MediaWiki_SVG_fonts.svg? And I'm guessing that this reference SVG will have to be updated to use the new "reference" fonts that are replacing those?

Wed, Aug 16, 3:56 PM · Patch-For-Review, User-fgiunchedi, Performance-Team (Radar), Operations, Thumbor
Gilles removed a project from T172930: Long running thumbnail requests locking up Thumbor instances: Patch-For-Review.
Wed, Aug 16, 3:49 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles claimed T151554: Track incoming HTTP request count on the Thumbor boxes.
Wed, Aug 16, 3:43 PM · Patch-For-Review, User-fgiunchedi, Operations, Performance-Team, Thumbor
Gilles added a comment to T151554: Track incoming HTTP request count on the Thumbor boxes.

As a note, just looking at yesterday's data, nginx 502s once per minute on average. Much larger old error log files suggest that this might peak at times. We really need to record that in a graph.

Wed, Aug 16, 3:43 PM · Patch-For-Review, User-fgiunchedi, Operations, Performance-Team, Thumbor
Gilles claimed T172939: Thumbor webp handling.
Wed, Aug 16, 3:39 PM · Patch-For-Review, Performance-Team, Thumbor
Gilles added a comment to T172939: Thumbor webp handling.

Seems like it was just never enabled in Puppet.

Wed, Aug 16, 3:39 PM · Patch-For-Review, Performance-Team, Thumbor
Gilles added a comment to T170817: Upgrade Thumbor servers to Stretch.

I've compared deployment-imagescaler02 again and I see rendering differences for kochi fonts. Isn't it the same issue as the ttf ubuntu fonts?

Wed, Aug 16, 3:15 PM · Patch-For-Review, User-fgiunchedi, Performance-Team (Radar), Operations, Thumbor
Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

What I need to verify is whether a thumbor process is truly blocking while waiting on a poolcounter lock for timeout seconds and unable to process other requests (queueing them, I guess). If that's the case, then the thumbor processes waiting for their turn are hitting the worst case scenario where they're waiting more than 8 seconds for that lock, since the actual processes rendering those thumbnails for the given IP-based lock take a while.

Wed, Aug 16, 2:54 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

I still triggered 502s, that wasn't sufficient.

Wed, Aug 16, 2:38 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles created P5889 Stress test Thumbor.
Wed, Aug 16, 2:37 PM

Tue, Aug 15

Gilles added a comment to T170817: Upgrade Thumbor servers to Stretch.

I've updated the Thumbor package here: https://github.com/gi11es/thumbor-debian/tree/master/thumbor with the latest master from upstream, that includes my bugfix and the relaxed Pillow version check.

Tue, Aug 15, 3:43 PM · Patch-For-Review, User-fgiunchedi, Performance-Team (Radar), Operations, Thumbor
Gilles updated subscribers of T161719: Add STL support (with 3d2png) to Thumbor.

@mmodell what's the process for deploying an update of 3d2png to deployment-imagescaler01?

Tue, Aug 15, 3:23 PM · Patch-For-Review, Performance-Team, Thumbor
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.

It's mostly a bandwidth waste, since the download happens after the pageload. I'll let others be the judge of how urgent that makes it.

Tue, Aug 15, 3:20 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles added a comment to T170817: Upgrade Thumbor servers to Stretch.

Right off the bat, the first one with major differences, Century Schoolbook L, comes from the "gsfonts" package, which is found on thumbor1001, deployment-imagescaler01, but not on deployment-imagescaler02. @fgiunchedi is a role missing from deployment-imagescaler02 or something?

Tue, Aug 15, 3:17 PM · Patch-For-Review, User-fgiunchedi, Performance-Team (Radar), Operations, Thumbor
Gilles added a comment to T170817: Upgrade Thumbor servers to Stretch.

The font config files from fonts.pp that you wrote for Jessie are definitely present on deployment-imagescaler02.

Tue, Aug 15, 3:14 PM · Patch-For-Review, User-fgiunchedi, Performance-Team (Radar), Operations, Thumbor
Gilles added a comment to T170817: Upgrade Thumbor servers to Stretch.

It's probably a minor difference in rsvg rendering. 98.8% is very good similarity. Let's double check if the rendering difference is significant.

Tue, Aug 15, 2:47 PM · Patch-For-Review, User-fgiunchedi, Performance-Team (Radar), Operations, Thumbor
Gilles added a comment to T161719: Add STL support (with 3d2png) to Thumbor.

Installed the new package on deployment-imagescaler01.deployment-prep.eqiad.wmflabs, restarted thumbor, and added a manual config to enable the engine in /etc/thumbor.d/98-3d.conf

Tue, Aug 15, 2:36 PM · Patch-For-Review, Performance-Team, Thumbor
Gilles assigned T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview to TheDJ.
Tue, Aug 15, 2:21 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles claimed T172930: Long running thumbnail requests locking up Thumbor instances.
Tue, Aug 15, 2:04 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations
Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

I think that the current per-IP PoolCounter limits are just too generous. A single user can hog up to 32 workers right now. IMHO, what matters for a given user is the size of the queue, but they can afford to wait if there are a lot of thumbnails on the page that need rendering.

Tue, Aug 15, 2:02 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations

Mon, Aug 14

Gilles closed T169072: Due to missing fonts on Thumbor hosts, PNG thumbnail of SVG file showing a Devanagari character only displays a Unicode number box as Resolved.

All SVGs affected during this bug's timeframe have been purged.

Mon, Aug 14, 7:36 PM · Thumbor, Commons
Gilles closed T169072: Due to missing fonts on Thumbor hosts, PNG thumbnail of SVG file showing a Devanagari character only displays a Unicode number box, a subtask of T43371: Thumbnail/imagescaler (tracking), as Resolved.
Mon, Aug 14, 7:36 PM · media-storage, Commons, Tracking
Gilles updated the task description for T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.
Mon, Aug 14, 2:56 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles renamed T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview from Extra 100KB of JS loaded by Kartographer on pageview for authenticated mobile users to Extra 100KB of JS loaded by mobilemaps gadget on pageview for authenticated mobile users.
Mon, Aug 14, 2:55 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.

Wait... extra requests, or changed requests ??? Because i have no idea why icons-moderation would start loading, based on the modules i'm referencing...

Mon, Aug 14, 2:52 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles added a comment to T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.
Mon, Aug 14, 12:28 PM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles updated the task description for T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.
Mon, Aug 14, 1:05 AM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles updated the task description for T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.
Mon, Aug 14, 1:04 AM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles renamed T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview from Extra 100KB of JS loaded on pageview for authenticated mobile users to Extra 100KB of JS loaded by Kartographer on pageview for authenticated mobile users.
Mon, Aug 14, 1:01 AM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles triaged T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview as High priority.
Mon, Aug 14, 1:00 AM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)
Gilles created T173283: Extra 100KB of JS loaded by mobilemaps gadget on pageview.
Mon, Aug 14, 12:59 AM · Readers-Web-Backlog (Tracking), Performance-Team (Radar), Maps-Sprint, Maps (Kartographer)

Thu, Aug 10

Gilles added a comment to T168949: Proper thumbnails of portrait photos not being generated; serious display issues.
Thu, Aug 10, 2:27 PM · Thumbor, Commons
Gilles added a comment to T172930: Long running thumbnail requests locking up Thumbor instances.

I did some load testing yesterday that caused 502s, do you have a list of pages with times?

Thu, Aug 10, 2:18 PM · Patch-For-Review, Performance-Team, User-fgiunchedi, Thumbor, Operations

Wed, Aug 9

Gilles committed rTHMBREXTee0d595cb697: Version bump (authored by Gilles).
Version bump
Wed, Aug 9, 8:16 PM
Gilles committed rTHMBREXT125b7b5e6a3f: Move 3D tests out of test_types (authored by Gilles).
Move 3D tests out of test_types
Wed, Aug 9, 8:12 PM
Gilles committed rTHMBREXT69a3d6d63c9a: Add STL engine (authored by Mark Holmquist <mholmquist@wikimedia.org>).
Add STL engine
Wed, Aug 9, 7:23 PM
Gilles closed D732: Add STL engine by committing rTHMBREXT69a3d6d63c9a: Add STL engine (authored by Mark Holmquist <mholmquist@wikimedia.org>).
Wed, Aug 9, 7:23 PM
Gilles accepted D732: Add STL engine.
Wed, Aug 9, 7:20 PM
Gilles added inline comments to D732: Add STL engine.
Wed, Aug 9, 7:20 PM
Gilles added inline comments to D732: Add STL engine.
Wed, Aug 9, 7:18 PM
Gilles updated the diff for D732: Add STL engine.

Fix reference thumbnails dimensions

Wed, Aug 9, 7:12 PM
Gilles added a comment to D732: Add STL engine.

The reference thumbnails still have incorrect dimensions (square), which will make the tests fail.

Wed, Aug 9, 6:36 PM
Gilles added a comment to T108687: Security review for CodeMirror extension branch master.

What's the status of this security review? It's my understanding that this extension was briefly deployed to all wikis as a beta feature and should be back soon.

Wed, Aug 9, 5:09 PM · Community-Tech, Security-Reviews, MediaWiki-extensions-CodeMirror
Gilles moved T170605: ERR_RESPONSE_HEADERS_MULTIPLE_CONTENT_DISPOSITION from Backlog to Radar on the Thumbor board.
Wed, Aug 9, 5:06 PM · Thumbor, media-storage, Traffic, Operations, Commons
Gilles moved T170810: Update rsvg to 2.40.18 from Backlog to Radar on the Thumbor board.
Wed, Aug 9, 5:06 PM · Wikimedia-SVG-rendering, Thumbor, Commons, media-storage
Gilles added a comment to T170444: On beta commons, thumbnailing of 3D files is broken still.

Yes, now that Thumbor is serving all thumbnail traffic, T161719 is the blocker for this task. @MarkTraceur and I got a solution working locally yesterday, so I expect that it'll be working in beta and possibly production in the next couple of weeks.

Wed, Aug 9, 1:54 PM · Multimedia, Thumbor, Beta-Cluster-Infrastructure

Tue, Aug 8

Gilles added inline comments to D732: Add STL engine.
Tue, Aug 8, 8:06 PM
Gilles added a comment to D732: Add STL engine.

The reference thumbnails are square. So it fails because the aspect ratio is different. I think you need to regenerate the reference thumbnails with the right dimensions. The aspect ratio in the Mediawiki extension is definitely 640 /480.

Tue, Aug 8, 7:52 PM
Gilles added inline comments to D732: Add STL engine.
Tue, Aug 8, 7:49 PM
Gilles added a comment to D732: Add STL engine.

This code in 3d2png is getting in the way: https://github.com/wikimedia/3d2png/blob/master/3d2png.js#L68 because the temp file generated by the thumbor plugins is extension-less.

Tue, Aug 8, 5:46 PM
Gilles added inline comments to D732: Add STL engine.
Tue, Aug 8, 5:23 PM
Gilles added a comment to D732: Add STL engine.

You should be able to test the code on deployment-imagescaler01:

Tue, Aug 8, 4:46 PM
Gilles added inline comments to D732: Add STL engine.
Tue, Aug 8, 4:35 PM
Gilles added a comment to D732: Add STL engine.

Looks right to me at first glance, you can test the hack by adding the same one to our https loader, and adding a test to test_https_loader.py pointing to an STL file hosted on one of our wikis.

Tue, Aug 8, 2:58 PM
Gilles closed T151339: Expose Thumbor internally for ImageTweaks and similar use cases as Declined.

Now that the project is finished, I think that the thumbor instances we have a heavily rigged for our thumbnail traffic (saving all results to swift). It would make more sense to reuse the common infrastructure and deploy thumbor configured differently for those other use cases on other servers. This would also reduce side effects between very different uses of Thumbor. I can provide guidance if needed.

Tue, Aug 8, 2:48 PM · Performance-Team, Thumbor, Beta-Feature, ImageTweaks, Multimedia
Gilles closed T151339: Expose Thumbor internally for ImageTweaks and similar use cases, a subtask of T141317: Provide a beta feature of ImageTweaks, as Declined.
Tue, Aug 8, 2:48 PM · Beta-Feature, ImageTweaks, Multimedia
Gilles closed T152496: Investigate remaining memory leaks in Thumbor as Resolved.

I don't think this is a good time investment. I fixed the leak I could find and couldn't find any more last time I checked. Changing the memory limit seems to have considerably lowered OOMs anyway. I doubt that there's any significant leaking anymore.

Tue, Aug 8, 2:41 PM · Performance-Team, Thumbor
Gilles closed T159358: Get rid of lingering temp files following ImageMagick subprocess update of Thumbor as Resolved.

Ghostscript currently doesn't do a very good job of deleting temporary files if it exits because of an error; you may have to delete them manually from time to time.

Tue, Aug 8, 2:38 PM · Performance-Team, Thumbor

Mon, Aug 7

Gilles added a comment to T172227: Consider adding one or more smaller pregenerated thumbnail buckets for mobile clients.

If you're talking about chaining thumbnails server-side, that's difficult to do without affecting quality due to the conditional sharpening we apply to JPGs. An already sharpened intermediary size is unusable, as the end result would be over-sharpened. This issue is the reason why the last attempt at chaining thumbnails had to be pulled. To respect sharpening, we'd need to store unsharpened intermediary sizes as extra images, which is expensive storage-wise and rendering-wise.

Mon, Aug 7, 9:38 PM · Performance, Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog
Gilles changed the status of T168949: Proper thumbnails of portrait photos not being generated; serious display issues, a subtask of T43371: Thumbnail/imagescaler (tracking), from Stalled to Open.
Mon, Aug 7, 8:26 PM · media-storage, Commons, Tracking
Gilles changed the status of T168949: Proper thumbnails of portrait photos not being generated; serious display issues from Stalled to Open.
Mon, Aug 7, 8:26 PM · Thumbor, Commons
Gilles changed the status of T168949: Proper thumbnails of portrait photos not being generated; serious display issues, a subtask of T121388: Service-based thumbnailing re-architecture in production with Thumbor, from Stalled to Open.
Mon, Aug 7, 8:26 PM · Performance-Team, Thumbor
Gilles closed T121388: Service-based thumbnailing re-architecture in production with Thumbor as Resolved.

Thumbor has been serving all thumbnail traffic for over a month now. I think this task can be closed as the project is completed and successful. Subsequent bugfixes and improvements are tracked with the Thumbor project.

Mon, Aug 7, 8:16 PM · Performance-Team, Thumbor
Gilles changed the status of T169072: Due to missing fonts on Thumbor hosts, PNG thumbnail of SVG file showing a Devanagari character only displays a Unicode number box from Stalled to Open.
Mon, Aug 7, 4:57 PM · Thumbor, Commons
Gilles changed the status of T169072: Due to missing fonts on Thumbor hosts, PNG thumbnail of SVG file showing a Devanagari character only displays a Unicode number box, a subtask of T43371: Thumbnail/imagescaler (tracking), from Stalled to Open.
Mon, Aug 7, 4:57 PM · media-storage, Commons, Tracking
Gilles closed T169198: File purging should also purge the thumbnails of old versions as Resolved.
Mon, Aug 7, 4:54 PM · MW-1.30-release-notes, Thumbor, Commons
Gilles closed T169198: File purging should also purge the thumbnails of old versions, a subtask of T168949: Proper thumbnails of portrait photos not being generated; serious display issues, as Resolved.
Mon, Aug 7, 4:54 PM · Thumbor, Commons
Gilles closed T169198: File purging should also purge the thumbnails of old versions, a subtask of T169072: Due to missing fonts on Thumbor hosts, PNG thumbnail of SVG file showing a Devanagari character only displays a Unicode number box, as Resolved.
Mon, Aug 7, 4:54 PM · Thumbor, Commons
Gilles added inline comments to D732: Add STL engine.
Mon, Aug 7, 2:42 PM
Gilles added a comment to T172227: Consider adding one or more smaller pregenerated thumbnail buckets for mobile clients.

120 is the 5th most requested size and its average file size is obviously small. Interestingly it's also the highest on the list of cache misses. So yes, it would make absolute sense to add 120 to the pre-rendering list. Those thumbnails are already used a lot and would benefit from being pre-rendered, considering how their miss rate is high. As an already popular size, it's a good choice for mobile to leverage it. 220 is also worth considering, as its the size that gets the most absolute hits and is second for misses after 120.

Mon, Aug 7, 1:07 PM · Performance, Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog

Fri, Aug 4

Gilles committed rTHMBREXTca4958128cab: Handle SVG files greater than 10MB (authored by Gilles).
Handle SVG files greater than 10MB
Fri, Aug 4, 12:21 AM
Gilles added a commit to T170352: Large SVG files fail to render in Thumbor (due to lack of use of the --unlimited option in librsvg): rTHMBREXTca4958128cab: Handle SVG files greater than 10MB.
Fri, Aug 4, 12:21 AM · Patch-For-Review, Performance-Team, Thumbor

Jul 22 2017

Gilles renamed T170810: Update rsvg to 2.40.18 from Update rsvg to 2.40.17 to Update rsvg to 2.40.18.
Jul 22 2017, 6:51 AM · Wikimedia-SVG-rendering, Thumbor, Commons, media-storage

Jul 20 2017

Gilles added a comment to T169565: Performance review of "Explore similar".

@EBernhardson sounds good to me, better chances of combining Varnish hits with another feature is definitely the most important optimization

Jul 20 2017, 6:59 PM · Patch-For-Review, Discovery-Search (Current work), Discovery, Performance-Team
Deskana awarded T171093: Performance review of VisualEditor a Love token.
Jul 20 2017, 9:03 AM · VisualEditor-Performance, Performance-Team, VisualEditor

Jul 19 2017

Gilles created P5767 Vagrant Thumbor sync.
Jul 19 2017, 9:20 PM
Gilles moved T171093: Performance review of VisualEditor from Inbox to Potential goals on the Performance-Team board.
Jul 19 2017, 7:11 PM · VisualEditor-Performance, VisualEditor, Performance-Team