[tahoe-lafs-trac-stream] [tahoe-lafs] #1812: parse_abbreviated_size doesn't accept T for terabytes (and other quibbles with the regex it uses)

[tahoe-lafs]

#1812: parse_abbreviated_size doesn't accept T for terabytes (and other quibbles
with the regex it uses)
-------------------------+-------------------------------------------------
     Reporter:           |      Owner:  warner
  davidsarah             |     Status:  new
         Type:  defect   |  Milestone:  1.10.0
     Priority:  minor    |    Version:  1.9.2
    Component:  code     |   Keywords:  reserved_space storage error
   Resolution:           |  usability reviewed
Launchpad Bug:           |
-------------------------+-------------------------------------------------
Changes (by warner):

 * owner:  davidsarah => warner
 * status:  reopened => new


Comment:

 Looks like we should *not* catch the exception, and allow it to be raised,
 since `init_storage()` is called before daemonization, so errors will
 appear on the 'tahoe start' stderr. The only thing that might need
 special-casing is "reserved_space=" (i.e. the default do-not-reserve-any-
 space), just in case that returns an empty string or something
 unparseable. This should be covered in unit tests.

-- 
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1812#comment:14>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage