storage dump"
ClosedPublic
Actions

Authored by epriestley on Dec 18 2021, 7:06 PM.

Details

Reviewers: None
Maniphest Tasks: T13671: Merge "secure003.phacility.net" into "secure001.phacility.net", then migrate to "m4.large" hardware
Commits: rPdec95228a241: Add a "--database <name> ..." flag to "bin/storage dump"

Summary

Ref T13671. Allow "bin/storage dump" to dump a subset of databases, primarily to support merging previously-partitioned databases.

Test Plan

Ran bin/storage dump with and without --database ... flags. Ran --database invalid, --database a --database a to hit error cases.

Diff Detail

Repository

rP Phabricator

Branch

dump1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 25626
Build 35454: Run Core Tests
Build 35453: arc lint + arc unit

Event Timeline

epriestley created this revision.Dec 18 2021, 7:06 PM

Harbormaster completed remote builds in B25625: Diff 51839.Dec 18 2021, 7:07 PM

epriestley requested review of this revision.Dec 18 2021, 7:07 PM

Fix missing variable initialization.

Harbormaster completed remote builds in B25626: Diff 51840.Dec 18 2021, 7:10 PM

This revision was not accepted when it landed; it landed in state Needs Review.Dec 18 2021, 7:11 PM

Closed by commit rPdec95228a241: Add a "--database <name> ..." flag to "bin/storage dump" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPdec95228a241: Add a "--database <name> ..." flag to "bin/storage dump".

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

storage/

management/

PhabricatorStorageManagementAPI.php

11 lines

workflow/

PhabricatorStorageManagementDumpWorkflow.php

61 lines

Diff 51840

View Options

src/infrastructure/storage/management/PhabricatorStorageManagementAPI.php