Page MenuHomePhabricator

Allow config get to work when db is not functional
ClosedPublic

Authored by skroll on Nov 14 2014, 4:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 5:26 AM
Unknown Object (File)
Thu, Apr 11, 8:30 AM
Unknown Object (File)
Sat, Apr 6, 5:07 PM
Unknown Object (File)
Feb 24 2024, 4:07 PM
Unknown Object (File)
Feb 13 2024, 11:02 PM
Unknown Object (File)
Feb 5 2024, 2:32 PM
Unknown Object (File)
Jan 18 2024, 1:01 PM
Unknown Object (File)
Jan 15 2024, 4:00 PM
Subscribers
Tokens
"Like" token, awarded by chad.

Details

Summary

Fixes T6554.

Test Plan

Run bin/config get mysql.host when no db is functional. Should not get an exception, but should see:

{
  "config" : [
    {
      "key"       : "mysql.host",
      "source"    : "local",
      "value"     : null,
      "status"    : "unset",
      "errorInfo" : null
    },
    {
      "key"       : "mysql.host",
      "source"    : "database",
      "value"     : null,
      "status"    : "error",
      "errorInfo" : "Database source is not configured properly"
    }
  ]
}

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

skroll retitled this revision from to Allow config get to work when db is not functional.
skroll updated this object.
skroll edited the test plan for this revision. (Show Details)

I already did, or did it not associate it with my account?

EDIT: Signed it again.

epriestley added a reviewer: epriestley.

Looks like it's good now. Thanks!

This revision is now accepted and ready to land.Nov 14 2014, 5:11 PM
This revision was automatically updated to reflect the committed changes.