Page MenuHomePhabricator

No-op the Conpherence thumbnail resizing migration
ClosedPublic

Authored by epriestley on Apr 23 2017, 11:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:51 AM
Unknown Object (File)
Wed, Apr 10, 6:17 AM
Unknown Object (File)
Tue, Apr 2, 1:52 PM
Unknown Object (File)
Mon, Apr 1, 12:37 AM
Unknown Object (File)
Sun, Mar 31, 9:35 AM
Unknown Object (File)
Feb 17 2024, 11:18 AM
Unknown Object (File)
Jan 16 2024, 3:58 PM
Unknown Object (File)
Dec 28 2023, 8:43 PM
Subscribers
None

Details

Summary

Fixes T12628. After later changes to PhabricatorFile, this migration no longer runs if you upgrade through it to a recent HEAD while your data has some room images.

Since this isn't critical and has been available for ~6 months, I just nuked it as a first pass. I can find a more careful approach which lets us continue to run this migration instead if you're hesitant to skip this step, although it may be a little involved.

In 95% of cases we avoid this by updating the storage table as it existed at the time the migraiton ran, but Files are much too complicated for that to be realistic.

Test Plan

Ran bin/storage upgrade -f --apply phabricator:20161005.conpherence.image.2.php, saw it do nothing.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

A slightly cleaner way to degrade this would be to copy the old profile image PHIDs to the new storage (so the images stick around, just without a resolution upgrade), but 20161016.conpherence.imagephids.sql deleted the old storage so writing and testing that patch is fairly involved: I'd have to install a copy from before Oct 2016, add images to rooms, then run the upgrade and verify that the low-resolution images survived the migration.

This revision is now accepted and ready to land.Apr 23 2017, 2:08 PM
This revision was automatically updated to reflect the committed changes.