search

dashpay / sentinel

DashCore Sentinel Engine
https://www.dash.org
MIT License
32 stars 193 forks source link

Any interest in obsoleting this repo? #77

Closed nmarley closed 11 months ago

nmarley commented 2 years ago

This project never ended up being what it was envisioned as. Currently it just votes on the superblock for DashCore and votes to remove expired proposals, and that functionality could be absorbed into the C++ codebase. It would be less work for MNOs to set this up and it would ensure the superblock votes are coming in (hasn't been an enforced requirement to run Sentinel in a couple years).

I do believe it would take a hard fork cycle or 2 and require changes to the gobject format, using strings instead of numbers for example, but would get rid of this. Not sure if it's worth the effort or not, but wanted to float the idea.

@UdjinM6 @PastaPastaPasta @QuantumExplorer

thephez commented 2 years ago

@nmarley There seems to be agreement around your idea. Not sure what the priority/timeframe would be. Perhaps @PastaPastaPasta will have some idea on that.

nmarley commented 2 years ago

Thanks for the update!

I will look at recent changes to DashCore and see if I can integrate some old Proof-of-Concept code that might help serve as a base for something like this.

PastaPastaPasta commented 2 years ago

I would strongly support getting rid of sentinel!

kxcd commented 2 years ago

Heck, I'm no longer running this on my masternode(s) !

thephez commented 1 year ago

Since dashpay/dash#5525 has been merged, what, if anything, needs to be done with this repository from a de-conflict / compatibility standpoint? @ogabrielides @PastaPastaPasta @UdjinM6 @nmarley

thephez commented 11 months ago

Closing this issue since Dash Core v20 integrated all Sentinel functionality and rendered Sentinel obsolete.