[tahoe-lafs-trac-stream] [tahoe-lafs] #1784: add happiness count to check reports

[tahoe-lafs]

#1784: add happiness count to check reports
-------------------------+-------------------------------------------------
     Reporter:           |      Owner:  davidsarah
  davidsarah             |     Status:  assigned
         Type:  defect   |  Milestone:  1.11.0
     Priority:  normal   |    Version:  1.9.1
    Component:  code-    |   Keywords:  usability check verify repair
  frontend-web           |  servers-of-happiness
   Resolution:           |
Launchpad Bug:           |
-------------------------+-------------------------------------------------

Comment (by daira):

 Copying this from ticket:1115#comment:27, with minor updates:

 ... the value of {{{needs-rebalancing}}} is computed inconsistently
 between {{{checker.py}}} and {{{filenode.py}}}. In {{{checker.py}}} it is
 computed as:
 {{{
 # The file needs rebalancing if the set of servers that have at least
 # one share is less than the number of uniquely-numbered shares
 # available.
 # TODO: this may be wrong, see ticket #1115 comment:27 and ticket #1784.
 needs_rebalancing = bool(good_share_hosts < len(verifiedshares))
 }}}
 In {{{filenode.py}}} it is computed as
 {{{
 # TODO: this may be wrong, see ticket #1115 comment:27 and ticket #1784.
 needs_rebalancing = bool(len(sm) >= verifycap.total_shares)
 }}}
 where {{{len(sm)}}} is equal to {{{count-shares-good}}}. I don't
 understand this latter definition at all, it looks completely wrong. The
 definition in {{{checker.py}}} is more subtly wrong because it credits
 servers that only have duplicated shares as contributing to existing
 balance. The correct definition should be something like 'iff the
 happiness count is less than the number of uniquely-numbered good shares
 available'.

 I propose to change {{{filenode.py}}} and {{{checker.py}}} to be
 consistent, and to both use the happiness-based definition above.

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