Page MenuHomePhabricator

Make `bin/storage upgrade` and `bin/storage adjust` emit detailed messages if the user has no access to databases
ClosedPublic

Authored by epriestley on Jan 21 2016, 9:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 5, 11:34 AM
Unknown Object (File)
Jan 31 2024, 9:37 AM
Unknown Object (File)
Dec 26 2023, 9:08 AM
Unknown Object (File)
Dec 23 2023, 10:32 PM
Unknown Object (File)
Dec 22 2023, 7:21 AM
Unknown Object (File)
Dec 8 2023, 11:51 AM
Unknown Object (File)
Nov 25 2023, 7:26 AM
Unknown Object (File)
Nov 17 2023, 3:03 PM
Subscribers
None

Details

Summary

Ref T10195. Distinguish between "database does not exist" and "database exists, you just don't have permission to access it".

We can't easily get this information out of INFORMATION_SCHEMA but can just SHOW TABLES IN ... every database that looks like it's missing and then look at the error code.

Test Plan
  • Created a user limited with limited access.
  • Ran bin/storage adjust.
  • Got hopefully more helpful messages about access problems, instead of "Missing" errors.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make `bin/storage upgrade` and `bin/storage adjust` emit detailed messages if the user has no access to databases.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 21 2016, 9:05 PM
This revision was automatically updated to reflect the committed changes.