add happiness count to check and repair reports #1784
Labels
No Label
0.2.0
0.3.0
0.4.0
0.5.0
0.5.1
0.6.0
0.6.1
0.7.0
0.8.0
0.9.0
1.0.0
1.1.0
1.10.0
1.10.1
1.10.2
1.10a2
1.11.0
1.12.0
1.12.1
1.13.0
1.14.0
1.15.0
1.15.1
1.2.0
1.3.0
1.4.1
1.5.0
1.6.0
1.6.1
1.7.0
1.7.1
1.7β
1.8.0
1.8.1
1.8.2
1.8.3
1.8β
1.9.0
1.9.0-s3branch
1.9.0a1
1.9.0a2
1.9.0b1
1.9.1
1.9.2
1.9.2a1
LeastAuthority.com automation
blocker
cannot reproduce
cloud-branch
code
code-dirnodes
code-encoding
code-frontend
code-frontend-cli
code-frontend-ftp-sftp
code-frontend-magic-folder
code-frontend-web
code-mutable
code-network
code-nodeadmin
code-peerselection
code-storage
contrib
critical
defect
dev-infrastructure
documentation
duplicate
enhancement
fixed
invalid
major
minor
n/a
normal
operational
packaging
somebody else's problem
supercritical
task
trivial
unknown
was already fixed
website
wontfix
worksforme
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Reference: tahoe-lafs/trac-2024-07-25#1784
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
I have a patch for this to go in
1.101.11. Note that the description of #1115 said "add servers-of-happiness to reports", and we agreed to do that, but the patch applied for #1115 only fixed a bug in computingcount-good-share-hosts
.add servers-of-happiness to check reportsto add happiness count to check reportsIn changeset:5532/1.9.2:
In changeset:5886/cloud-backend:
In changeset:b06f8cd8d03a6239:
Copying this from /tahoe-lafs/trac-2024-07-25/issues/6177#comment:27, with minor updates:
... the value of
needs-rebalancing
is computed inconsistently betweenchecker.py
andfilenode.py
. Inchecker.py
it is computed as:In
filenode.py
it is computed aswhere
len(sm)
is equal tocount-shares-good
. I don't understand this latter definition at all, it looks completely wrong. The definition inchecker.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
andchecker.py
to be consistent, and to both use the happiness-based definition above.add happiness count to check reportsto add happiness count to check reports and fix the definition of needs-rebalancingSee also #614
This should probably be split into two tickets: one for adding the happiness outcome to reports and one for fixing
needs-rebalancing
.add happiness count to check reports and fix the definition of needs-rebalancingto add happiness count to check and repair reports, and fix the definition of needs-rebalancingThe
needs-rebalancing
issue has been split to #2105.add happiness count to check and repair reports, and fix the definition of needs-rebalancingto add happiness count to check and repair reports(https://github.com/tahoe-lafs/tahoe-lafs/pull/74) implements this ticket and #2105.
Review included in #2105 since they both are in the same pull request/patch.
In /tahoe-lafs/trac-2024-07-25/commit/0ef593947755a8b81edc73d033219724268faf80:
In /tahoe-lafs/trac-2024-07-25/commit/0ef593947755a8b81edc73d033219724268faf80:
In /tahoe-lafs/trac-2024-07-25/commit/0ef593947755a8b81edc73d033219724268faf80:
In /tahoe-lafs/trac-2024-07-25/commit/0ef593947755a8b81edc73d033219724268faf80:
In /tahoe-lafs/trac-2024-07-25/commit/0ef593947755a8b81edc73d033219724268faf80: