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
F14018008: D16502.id39712.diff
Tue, Nov 5, 2:58 AM
F14000330: D16502.id39712.diff
Thu, Oct 24, 9:16 PM
F13992475: D16502.id.diff
Tue, Oct 22, 4:56 PM
F13980995: D16502.id39712.diff
Sat, Oct 19, 1:40 PM
F13966837: D16502.id39717.diff
Oct 16 2024, 9:57 AM
F13965750: D16502.diff
Oct 16 2024, 4:06 AM
Unknown Object (File)
Oct 5 2024, 9:22 PM
Unknown Object (File)
Oct 5 2024, 3:27 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