Page MenuHomePhabricator

Don't use RENDER_NOW when generating thumbnails for PhotoDNA scans
Closed, ResolvedPublic1 Estimated Story Points

Description

Running the scanning script with the job queue option enabled has caused thumbnail generation to fail when the file has to be transformed. This is because the MediaWiki k8s containers are configured to not allow the thumbnail generation to occur. This caused a large chunk of errors, and so we need to remove the RENDER_NOW flag to prevent rendering the thumbnail in the job.

Event Timeline

Change 991545 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/MediaModeration@master] Remove RENDER_NOW from File::transform call to avoid job thumbnailing

https://gerrit.wikimedia.org/r/991545

Change 991551 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/MediaModeration@wmf/1.42.0-wmf.14] Remove RENDER_NOW from File::transform call to avoid job thumbnailing

https://gerrit.wikimedia.org/r/991551

Change 991545 merged by jenkins-bot:

[mediawiki/extensions/MediaModeration@master] Remove RENDER_NOW from File::transform call to avoid job thumbnailing

https://gerrit.wikimedia.org/r/991545

Change 991551 merged by jenkins-bot:

[mediawiki/extensions/MediaModeration@wmf/1.42.0-wmf.14] Remove RENDER_NOW from File::transform call to avoid job thumbnailing

https://gerrit.wikimedia.org/r/991551

Mentioned in SAL (#wikimedia-operations) [2024-01-18T14:03:58Z] <dreamyjazz@deploy2002> Started scap: Backport for [[gerrit:991551|Remove RENDER_NOW from File::transform call to avoid job thumbnailing (T355309)]]

Mentioned in SAL (#wikimedia-operations) [2024-01-18T14:05:15Z] <dreamyjazz@deploy2002> dreamyjazz: Backport for [[gerrit:991551|Remove RENDER_NOW from File::transform call to avoid job thumbnailing (T355309)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-01-18T14:11:48Z] <dreamyjazz@deploy2002> Finished scap: Backport for [[gerrit:991551|Remove RENDER_NOW from File::transform call to avoid job thumbnailing (T355309)]] (duration: 07m 50s)

QA for this is difficult on a local wiki and requires access to production logstash. I would suggest therefore that QA be skipped, as I have been able to verify that the issue at hand was fixed.

Djackson-ctr subscribed.

I concur with Dreamy_Jazz's comment regarding QA so therefore I'm pushing this ticket to Done.

kostajh claimed this task.