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
F18788813: D18576.id.diff
Wed, Oct 15, 6:38 AM
F18768235: D18576.id44611.diff
Wed, Oct 8, 2:30 AM
F18739156: D18576.id.diff
Oct 1 2025, 8:04 PM
F18731047: D18576.diff
Sep 30 2025, 1:56 PM
F18710683: D18576.id44611.diff
Sep 29 2025, 2:29 AM
F18580723: D18576.id.diff
Sep 11 2025, 4:23 AM
F18442700: D18576.id.diff
Aug 31 2025, 5:38 PM
F18418708: D18576.diff
Aug 30 2025, 1:38 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