Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync throttle flags #1157

Open
Gingeropolous opened this issue Oct 1, 2016 · 6 comments
Open

Sync throttle flags #1157

Gingeropolous opened this issue Oct 1, 2016 · 6 comments
Labels

Comments

@Gingeropolous
Copy link
Collaborator

Gingeropolous commented Oct 1, 2016

Sorry if I beat you to posting this @fluffypony , but I wanted to expand on the idea.

New Feature - Initial Synchronization (or heavy synch) throttling flag with different levels, or just a general throttling flag.

Goal - gets at decreasing the Cost of Node metric.

Rambling thoughts:

in general, perhaps the "throttle while syncing" should be a flag with different levels. I just had a VPS provider shutdown my service because the initial sync was gobbling the CPU too much and who knows how many other resources. Yeah, I know, this could be addressed with "nice" or whatever, but having it built in might make it function better? I dunno. The bastards could've charged me 15 bucks just to release the server from its banished state. Luckily, they were nice.

So, if a 5 point scale,

0 = no throttle
1 = limit number of incoming connections that can use you to sync during initial sync (default)
2 = no incoming connections during initial sync
3 = no incoming connections keep CPU below 75%
4 = no inc and keep CPU < 50%

Now I have five points.

@moneromooo-monero
Copy link
Collaborator

You have 6 points.

@Gingeropolous
Copy link
Collaborator Author

Now I have 5 points.

@moneromooo-monero
Copy link
Collaborator

I can't argue with that.

@ghost
Copy link

ghost commented Oct 2, 2016

Perhaps your node could run a local function such as 'time to validate x signatures' or 'time to fetch x bytes from peers' and use these as internal metrics against which to drive the CPU usage whilst keeping the initial sync time as low as possible?

@ghost
Copy link

ghost commented Oct 2, 2016

Local *calibration function

@dEBRUYNE-1
Copy link
Contributor

+proposal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants