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

possibly out of memory or process/resource limits reached #4116

Closed
4 tasks done
SrMonsterYT opened this issue Feb 3, 2024 · 18 comments · Fixed by #4127
Closed
4 tasks done

possibly out of memory or process/resource limits reached #4116

SrMonsterYT opened this issue Feb 3, 2024 · 18 comments · Fixed by #4127
Labels
🐞 Bug Report A bug that needs to be fixed. 🐌 Performance Problem This issue is related to performance issues, such as memory leaks or drops in timings. ✔ Resolved This Issue has been resolved.

Comments

@SrMonsterYT
Copy link

❗ Checklist

  • I am using the official english version of Slimefun and did not modify the jar.
  • I am using an up to date "DEV" (not "RC") version of Slimefun.
  • I am aware that issues related to Slimefun addons need to be reported on their bug trackers and not here.
  • I searched for similar open issues and could not find an existing bug report on this.

📍 Description

server crashed

error: https://mclo.gs/3rRGEww

I reported it to the ajLeaderboards plugin, and it said that the problem is not with it but with slimefun

" your server is running out of threads, and it appears that Slimefun is using the most threads (162). ajlb is using 70 "

📑 Reproduction Steps

I don't know how it happened but the server crashed

💡 Expected Behavior

that doesn't happen

📷 Screenshots / Videos

No response

📜 Server Log

No response

📂 /error-reports/ folder

No response

💻 Server Software

Purpur

🎮 Minecraft Version

1.20.x

⭐ Slimefun version

https://prnt.sc/PnU70Fb6nTaT

🧭 Other plugins

No response

@SrMonsterYT SrMonsterYT added 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced. 🐞 Bug Report A bug that needs to be fixed. labels Feb 3, 2024
@SrMonsterYT

This comment was marked as off-topic.

@WalshyDev

This comment was marked as off-topic.

@SrMonsterYT

This comment was marked as off-topic.

@Boomer-1 Boomer-1 added invalid This Issue or Pull Request is invalid. and removed 🐞 Bug Report A bug that needs to be fixed. 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced. labels Feb 3, 2024
@Boomer-1
Copy link

Boomer-1 commented Feb 3, 2024

there was no need to open a duplicate issue, instead just provide evidence of what happened.

@Boomer-1 Boomer-1 closed this as completed Feb 3, 2024
@Boomer-1 Boomer-1 added 🐞 Bug Report A bug that needs to be fixed. 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced. and removed invalid This Issue or Pull Request is invalid. labels Feb 3, 2024
@Boomer-1 Boomer-1 reopened this Feb 3, 2024
@Boomer-1

This comment was marked as off-topic.

@SrMonsterYT
Copy link
Author

there was no need to open a duplicate issue, instead just provide evidence of what happened.

only the server crashed before restarting, and this error appeared that the server's memory had run out, but I have 21GB on the server in use.

this spark I did before because the TPS was dropping, so I reported it to griefPrevention, and they pointed me to slimefun

spark: https://spark.lucko.me/krvm0xoPyb

@J3fftw1
Copy link
Contributor

J3fftw1 commented Feb 11, 2024

Few things to mention from your spark report
Networks addon is causing 11% due to the vanilla pusher https://spark.lucko.me/krvm0xoPyb?hl=88775 which of the 11% 5% is caused by griefpreventions module in dough so this issue #4117

isitemsimilar is also doing 0.5% https://spark.lucko.me/krvm0xoPyb?hl=86563 which is quite high for my liking

then there is also 0.66% taken by supreme setting a charge https://spark.lucko.me/krvm0xoPyb?hl=104832
No clue why its that high, it looks like a BS call but that should just be to memory and not to file soo /shrug need to look into it more.

Why your server crashed i cant really tell its hard to find it, we do use a lot of threads since we try to multi thread a lot or in the minecraft world asynchronize a lot

@SrMonsterYT do you have any logs from before the server crashed?
are you with any hosting providor or do you self host?

@J3fftw1 J3fftw1 added 🐌 Performance Problem This issue is related to performance issues, such as memory leaks or drops in timings. and removed 🎯 Needs testing This Issue needs to be tested by our team to see if it can be reproduced. labels Feb 11, 2024
@SrMonsterYT
Copy link
Author

Few things to mention from your spark report Networks addon is causing 11% due to the vanilla pusher https://spark.lucko.me/krvm0xoPyb?hl=88775 which of the 11% 5% is caused by griefpreventions module in dough so this issue #4117

isitemsimilar is also doing 0.5% https://spark.lucko.me/krvm0xoPyb?hl=86563 which is quite high for my liking

then there is also 0.66% taken by supreme setting a charge https://spark.lucko.me/krvm0xoPyb?hl=104832 No clue why its that high, it looks like a BS call but that should just be to memory and not to file soo /shrug need to look into it more.

Why your server crashed i cant really tell its hard to find it, we do use a lot of threads since we try to multi thread a lot or in the minecraft world asynchronize a lot

@SrMonsterYT do you have any logs from before the server crashed? are you with any hosting providor or do you self host?

it's a dedicated server, i9 10k, 128gb, i'm not hosting it's bought from a company, there's nothing before the glitch just that, players do a lot of farming and use the pusher a lot, if there was something to replace the pusher it would be perfect.

@J3fftw1
Copy link
Contributor

J3fftw1 commented Feb 11, 2024

okay so i assume you manage it your self.
There should be some logs in the dmesg that should give some more info about this.
You could always limit the pusher
but i would initially wait for the dough PR to be merged.
This would save a lot from your timings

@SrMonsterYT
Copy link
Author

okay so i assume you manage it your self. There should be some logs in the dmesg that should give some more info about this. You could always limit the pusher but i would initially wait for the dough PR to be merged. This would save a lot from your timings

all the logs are like this, there's nothing else, I even reported it to griefPrevention, and everyone pointed only to slimefun

@J3fftw1
Copy link
Contributor

J3fftw1 commented Feb 11, 2024

I am pretty sure you havent checked the kernel logs.
You can use dmesg to view that.
Slimefun is known to use a lot of threads but that shouldnt matter much.
We need info on why your server crashed and your kernel could give us more info.
You are self hosting this so please check the logs.
Not minecraft server logs, kernel logs.

i mean its easy to point you to somewhere else and say they are the issue but we need more info.
You are the first to report this in ages so it might be something not Slimefun there are 2134 servers running Slimefun and 1 reported this issue.
So it can be something else entirely.

@WalshyDev
Copy link
Member

Please send $ ulimit -a output

@SrMonsterYT
Copy link
Author

Please send $ ulimit -a output

should I type this into the VPS and send you the result?

@J3fftw1
Copy link
Contributor

J3fftw1 commented Feb 11, 2024

yes

@SrMonsterYT
Copy link
Author

yes

fish: Expected a variable name after this $.

@J3fftw1
Copy link
Contributor

J3fftw1 commented Feb 11, 2024

type ulimit -a
$ means its a shell command

@SrMonsterYT
Copy link
Author

type ulimit -a

https://prnt.sc/Ua2hWTRmHLoi

@J3fftw1
Copy link
Contributor

J3fftw1 commented Feb 14, 2024

same issue as #4011

@variananora variananora added the ✔ Resolved This Issue has been resolved. label Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug Report A bug that needs to be fixed. 🐌 Performance Problem This issue is related to performance issues, such as memory leaks or drops in timings. ✔ Resolved This Issue has been resolved.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants