-
Notifications
You must be signed in to change notification settings - Fork 866
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
IPFS Desktop should install go-ipfs as a service #1977
Comments
On installation optionsUnsure if installing Windows Service is worth effort: it won't improve the default installation method (per user, without admin elevation). My rationale:
How to fix it in per user installI think Windows Service is not the solution, but adjusting process priorty could be:
Next stepsIf someone wants to work on this, here are some pointers:
The key is to do this with minimal amount of code that is robust against failures ( |
Well, I think so. One advantage is, that it allows more security around the ipfs daemon through access restrictions, like we have with the hardened service for Linux/Systemd. So the worst case scenario gets much less worse if there's a security flaw in ipfs itself. Regarding CPU/IO spikes, I run my service as background as possible on my Linux box, since I hardly have no CPU time to spare, but giving ipfs a fair share will just increase the overall latency of desktop operations, like open an app etc. This looks like this in systemd:
The effect is certainly noticeable, and I think I wouldn't run ipfs on this box if I couldn't put it in the background like this. |
Is your feature request related to a problem? Please describe.
This feature request stemmed from an issue on Windows where IPFS was randomly going full throttle, presumably to do DHT bootstrapping or the like.
Since IPFS Desktop and go-ipfs are running as a normal user app in the system try, it treats go-ipfs with the default "normal" process priority. When CPU usage gets high, it interferes with most all other user applications that are open.
As a temporary workaround, you can restrict the number of processor cores allowed, or lower the process priority from task manager.
I've seen some Linux users use a much nicer workaround -- running go-ipfs as a service and setting the priority lower by default.
Describe the solution you'd like
Having IPFS Desktop install go-ipfs as a service on Linux, Windows, etc, allows for
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: