Cosmos-sdk: Slashing and Evidence Module Relationship

[alexanderbez]

Summary

With the advent and introduction of the new x/evidence module, there is some natural overlap with the existing x/slashing module. Namely, the handling of evidence. Currently, x/slashing handles the following:

1. Slashing parameters
2. Liveness tracking
3. Evidence handling (equivocation only)
4. Slashing business logic

Note, the actual slashing is done by the x/staking module.

Problem Definition

Modules should be primarily isolated in function and utility (like Linux packages).

Proposal

I think we can reduce or remove completely this overlap by simply having the x/evidence module deal with (3). Then, x/slashing primarily deals with liveness tracking, slashing parameters and slashing business logic!

Thoughts @cwgoes @zmanian @rigelrozanski @fedekunze @AdityaSripal ?

---

For Admin Use

• [x] Not duplicate issue
• [x] Appropriate labels applied
• [x] Appropriate contributors tagged
• [ ] Contributor assigned/self-assigned

[fedekunze]

++

[alexanderbez]

I'll start work on this if there are no objections.

[cwgoes]

Concept ACK, proposal ACK. No objections.