Page MenuHomePhabricator

Convert `./bin/storage status` to use `PhutilConsoleTable`.
ClosedPublic

Authored by joshuaspence on Jun 17 2014, 5:53 PM.
Tags
None
Referenced Files
F13798858: D9604.id23019.diff
Sat, Sep 14, 8:56 PM
Unknown Object (File)
Thu, Sep 5, 4:42 PM
Unknown Object (File)
Wed, Sep 4, 9:29 PM
Unknown Object (File)
Tue, Sep 3, 9:40 PM
Unknown Object (File)
Wed, Aug 28, 1:44 AM
Unknown Object (File)
Sun, Aug 25, 9:01 PM
Unknown Object (File)
Sun, Aug 25, 1:50 PM
Unknown Object (File)
Thu, Aug 22, 1:53 PM
Subscribers

Details

Summary

Constructing tables manually just isn't fun.

Test Plan
./bin/storage status
phabricator:db.audit                                      Applied db   audit                                                                                       
phabricator:db.calendar                                   Applied db   calendar                                                                                    
phabricator:db.chatlog                                    Applied db   chatlog                                                                                     
phabricator:db.conduit                                    Applied db   conduit                                                                                     
phabricator:db.countdown                                  Applied db   countdown                                                                                   
phabricator:db.daemon                                     Applied db   daemon                                                                                      
phabricator:db.differential                               Applied db   differential                                                                                
phabricator:db.draft                                      Applied db   draft                                                                                       
phabricator:db.drydock                                    Applied db   drydock                                                                                     
phabricator:db.feed                                       Applied db   feed                                                                                        
phabricator:db.file                                       Applied db   file                                                                                        
phabricator:db.flag                                       Applied db   flag                                                                                        
phabricator:db.harbormaster                               Applied db   harbormaster
...

This probably isn't ready to land yet, we should fix PhutilConsoleTable to truncate columns which would otherwise cause overflow.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Convert `./bin/storage status` to use `PhutilConsoleTable`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

(We could likely also convert ./bin/storage probe, but the output format is not a conventional table)

epriestley edited edge metadata.

This probably isn't ready to land yet, we should fix PhutilConsoleTable to truncate columns which would otherwise cause overflow.

I think this is fine -- it's sort of bad to truncate the paths, and the non-truncation behavior is no worse than what we do today.

This revision is now accepted and ready to land.Jun 21 2014, 5:04 PM
joshuaspence edited edge metadata.
joshuaspence updated this revision to Diff 23164.

Closed by commit rPb7b045830307 (authored by @joshuaspence).