Page MenuHomePhabricator
Differential D10800

Apply storage adjustments as part of storage upgrade
ClosedPublic
Actions

Authored by epriestley on Nov 6 2014, 9:56 PM.
Tags
None
Referenced Files
F19893967: D10800.id25920.diff
Sat, Mar 28, 11:11 AM
F19890589: D10800.diff
Thu, Mar 26, 2:27 AM
F19868949: D10800.diff
Mar 15 2026, 4:30 AM
F19799875: D10800.id25959.diff
Feb 28 2026, 6:59 AM
F19692127: D10800.id25920.diff
Feb 10 2026, 5:19 AM
F19683002: D10800.id25919.diff
Feb 9 2026, 12:34 AM
F19636403: D10800.id25919.diff
Feb 5 2026, 1:03 PM
F19554608: D10800.id25959.diff
Jan 30 2026, 12:16 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T1191: Comments are truncated at first non-base-plane character
Commits
Restricted Diffusion Commit
rPa17a36869254: Apply storage adjustments as part of storage upgrade
Summary

Fixes T1191. I'll write up the changelog with notes about this and open a feedback task for followups.

When you run storage upgrade, automatically run storage adjust afterward. Provide a flag to disable this.

This brings everyone into the utf8mb4 world.

Test Plan

Ran bin/storage upgrade with various flags. Ran bin/storage adjust.

Diff Detail

Repository
rP Phabricator
Branch
adjust3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2972
Build 2976: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 25919.Nov 6 2014, 9:56 PM
epriestley retitled this revision from to Apply storage adjustments as part of storage upgrade.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T1191: Comments are truncated at first non-base-plane character.
Herald added a subscriber: epriestley. · View Herald TranscriptNov 6 2014, 9:56 PM
Harbormaster completed remote builds in B2971: Diff 25919.Nov 6 2014, 9:57 PM
epriestley updated this revision to Diff 25920.Nov 6 2014, 9:57 PM
  • Make --dryrun win when run as adjust --force --dryrun.
Harbormaster completed remote builds in B2972: Diff 25920.Nov 6 2014, 9:58 PM
epriestley mentioned this in T1191: Comments are truncated at first non-base-plane character.Nov 6 2014, 10:18 PM
btrahan mentioned this in T6350: Diffusion spamming the feed/commits going weird.Nov 6 2014, 10:25 PM
epriestley mentioned this in T6487: Historic Phriction documents do not have mailKeys populated.Nov 7 2014, 12:00 PM
btrahan accepted this revision.Nov 7 2014, 4:54 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Nov 7 2014, 4:54 PM
Closed by commit rPa17a36869254: Apply storage adjustments as part of storage upgrade (authored by epriestley, committed by epriestley). · Explain WhyNov 7 2014, 10:25 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 25920

View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementAdjustWorkflow.php

Loading...
View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementUpgradeWorkflow.php

Loading...
View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php

Loading...
Phabricator · Built by Phacility