Page MenuHomePhabricator
Differential D16502

Continue on bad database configuration from select scripts
ClosedPublic
Actions

Authored by epriestley on Sep 6 2016, 8:25 PM.
Tags
None
Referenced Files
F19226197: D16502.id.diff
Thu, Dec 18, 1:09 PM
F19218534: D16502.diff
Wed, Dec 17, 9:13 AM
F19208710: D16502.diff
Tue, Dec 16, 11:23 PM
F19081590: D16502.id39717.diff
Tue, Dec 2, 7:40 AM
F19078790: D16502.id39717.diff
Mon, Dec 1, 7:22 PM
F19058061: D16502.diff
Sat, Nov 29, 12:39 AM
F18824181: D16502.id39717.diff
Oct 23 2025, 2:37 PM
F18781815: D16502.id39712.diff
Oct 12 2025, 11:45 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11589: Phabricator configuration with invalid MySQL credentials isn't getting to the help messages
Commits
rP32a580c720e3: (stable) Continue on bad database configuration from select scripts
rP0e8ceeb69088: Continue on bad database configuration from select scripts
Summary

Ref T11589. Provide a way for scripts to say "just continue if database config fails", and use it in bin/config and bin/storage.

Test Plan
  • Broke database config.
  • Ran bin/config, worked fine.
  • Ran bin/storage, got helpful "set up the database" message.
  • Ran bin/repository, got fatal.
  • Ran normal site with valid/invalid config, got proper feedback.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 39712.Sep 6 2016, 8:25 PM
epriestley retitled this revision from to Continue on bad database configuration from select scripts.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11589: Phabricator configuration with invalid MySQL credentials isn't getting to the help messages.
chad accepted this revision.Sep 6 2016, 8:32 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 6 2016, 8:32 PM
Closed by commit rP0e8ceeb69088: Continue on bad database configuration from select scripts (authored by epriestley, committed by epriestley). · Explain WhySep 6 2016, 9:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
scripts/
23 lines
init/
10 lines
12 lines
init/
lib.php
__init_script__.php
11 lines
setup/
2 lines
sql/
2 lines
src/
infrastructure/
env/
20 lines

Diff 39717

View Options

scripts/__init_script__.php

Loading...
View Options

scripts/init/init-script.php

Loading...
View Options

scripts/init/init-setup.php

Loading...
View Options

scripts/init/lib.php

Loading...
View Options

scripts/setup/manage_config.php

Loading...
View Options

scripts/sql/manage_storage.php

Loading...
View Options

src/infrastructure/env/PhabricatorEnv.php

Loading...
Phabricator · Built by Phacility