Del/Drop reputation system support #24

Merged

fyrchik merged 1 commit from carpawell/frostfs-api:del/reputation into master

2023-04-18 10:15:12 +00:00

carpawell commented

2023-04-14 13:58:10 +00:00

Contributor

Closes #22.

carpawell self-assigned this 2023-04-14 13:58:10 +00:00

carpawell force-pushed del/reputation from ff0112f0a4 to 2f8ca6e10a

2023-04-14 13:58:51 +00:00

Compare

carpawell requested review from storage-core-developers 2023-04-14 13:58:57 +00:00

carpawell requested review from storage-core-committers 2023-04-14 13:58:58 +00:00

carpawell requested review from fyrchik 2023-04-14 13:59:01 +00:00

acid-ant approved these changes 2023-04-14 14:30:16 +00:00

aarifullin approved these changes 2023-04-17 06:46:03 +00:00

aarifullin left a comment

Member

Just for my curiosity. The issue says:

FrostFS storage nodes implement variation of EigenTrust iterative algorithm to calculate some metric for every peer. This metric should be distributed to the inner ring nodes so they can remove 'bad' from the network map.

What's an alternative for the deleted EigenTrustAlpha?

Just for my curiosity. The issue says: > FrostFS storage nodes implement variation of EigenTrust iterative algorithm to calculate some metric for every peer. This metric should be distributed to the inner ring nodes so they can remove 'bad' from the network map. What's an alternative for the deleted `EigenTrustAlpha`?

dstepanov-yadro commented

2023-04-17 07:40:49 +00:00

Member

Changelog?

👍 1

carpawell force-pushed del/reputation from 2f8ca6e10a to 059f4d5018

2023-04-17 08:01:23 +00:00

Compare

dstepanov-yadro approved these changes 2023-04-17 08:06:39 +00:00

fyrchik approved these changes 2023-04-17 08:54:05 +00:00

carpawell commented

2023-04-17 12:24:34 +00:00

Author

Contributor

What's an alternative for the deleted EigenTrustAlpha?

@aarifullin, do you mean exactly EigenTrustAlpha? there is no need to have that param (and an alternative to it) at all cause it is only used by the reputation system which is dropped in that PR

if you mean the reputation system in general, there is also no alternative solution cause no public networks yet and no understanding how to use the system. at least for now

if none of the above answers your question, please, clarify it a little

> What's an alternative for the deleted EigenTrustAlpha? @aarifullin, do you mean exactly `EigenTrustAlpha`? there is no need to have that param (and an alternative to it) at all cause it is only used by the reputation system which is dropped in that PR if you mean the reputation system in general, there is also no alternative solution cause no public networks yet and no understanding how to use the system. at least for now if none of the above answers your question, please, clarify it a little

👍 1