Page MenuHomePhabricator
Differential D18576

Correct `bin/storage analyze` internal API for cluster environments
ClosedPublic
Actions

Authored by epriestley on Sep 7 2017, 11:31 PM.
Tags
None
Referenced Files
F15470064: D18576.id44611.diff
Fri, Apr 4, 7:17 PM
F15460485: D18576.id44611.diff
Mon, Mar 31, 11:11 PM
F15448847: D18576.id44608.diff
Fri, Mar 28, 7:10 AM
F15445106: D18576.id.diff
Thu, Mar 27, 12:27 PM
F15441244: D18576.diff
Wed, Mar 26, 5:06 PM
F15437465: D18576.id.diff
Tue, Mar 25, 7:35 PM
F15433758: D18576.diff
Tue, Mar 25, 12:32 AM
F15431860: D18576.diff
Mon, Mar 24, 3:00 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12819: InnoDB FULLTEXT appears to fail catastrophically once it reaches a moderate size
Commits
rP4cae4a3b767f: Correct `bin/storage analyze` internal API for cluster environments
Summary

Ref T12819. This worked right in a non-cluster environment, but bin/storage upgrade iterates over each master in a partitioned cluster environment.

Tweak the API so bin/storage analyze targets a single host but bin/storage upgrade can hit all the masters.

Test Plan

Will run bin/storage upgrade in production again. Ran upgrade and analyze locally, still work fine.

Diff Detail

Repository
rP Phabricator
Branch
analyze2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18391
Build 24763: Run Core Tests
Build 24762: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
infrastructure/
storage/
management/
workflow/
3 lines
7 lines

Diff 44608

View Options

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

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility