-
-
Notifications
You must be signed in to change notification settings - Fork 831
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
[Bug]: cs2server. Monitor will fail when server is actually on. Then it will send quit and restart server. #4575
Comments
I also came across such a mistake.. Any fix? |
me too , server restart with same error when using 2 instances with the same installation Nov 08 17:34:07.865 cs2server: MONITOR: PASS: Querying port: gsquery: 31.56.0.68:27015 : 1 : OK crontab : ok maybe it was happning before when i had 2 servers with 2 diffrent users and diffrent instalation i used to see steamgamemanagment token key being refreshed 2-3 times a day and it shoudnt because i set the servers to restart only at 6 am every day . so i think monitor failled to monitor and leads the servers to quit and restart . im going to try for monitor once every hour and see if steam game manager token will refresh the token of my servers outside the sheduled time for restart. |
I have run into this issue as well monitoring multiple factorio server instances from a single install. With a single instance everything works fine indefinitely, but as soon as you monitor a second instance, it will fail and restart anywhere between 5 minutes and 2 hours over and over again. I found it interesting this seemed to only affect the first server, the second server never experienced the issue even once over weeks. I added some additional logging to the monitor script to see what the expanded pgrep command is and the value of its output and they are exactly what you would expect for this error to occur. I am reasonably sure at this point that the issue almost exclusively affects unnumbered instances in a multi-instance setup where the pgrep expression for the unnumbered instance can match on the command lines for the pgrep's being run for the monitoring of additional instances. I am testing a modified version of the monitoring module where the problematic pgrep expression has been modified to add a '^' to the beginning to ensure it is explicitly matching the start of command line, I suspect this will resolve the problem. It may be a good idea to repeat this for other applicable pgreps in the codebase if it is found to correct this problem. |
It hasn't quite been two hours yet since the last restart (4:20:17 EST), but I am feeling reasonably confident at this point my change resolves the underlying issue. It should be noted perhaps that my suggested fix might not work in the hopefully uncommon situation where the monitor module is able to see instances across multiple users. |
Witch crontab do you mean with this one ? |
I have only used the user crontab. Over three hours without a restart now. |
Thinking about this some more, while my change does fix this particular issue, the code is still ultimately bugged as it would still trigger on the unnumbered instance for any numbered instance with a tmux launched using the old parameters, so it doesn't work as intended even. |
User story
Need help to fix.
Game
cs2server
Linux distro
Ubuntu 22.04
Command
command: monitor
Further information
Monitor will fail when server is actually on. Then it will send quit and restart server.
Log shows: "PIDS with old type tmux session are running." What is it meaning?
Relevant log output
Steps to reproduce
The text was updated successfully, but these errors were encountered: